GitLab

Updates the config to match our full launch group for
Smart Selection on Chrome for Android.

BUG=770767

Change-Id: I80a04f3b66342eba17c17c8b66cda9b25d7b278b
Reviewed-on: https://chromium-review.googlesource.com/809850
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521915}

layer-tree-model.js:
This test dumps the layer tree and associated elements. With root
layer scrolling (RLS), the layers are associated with different
elements (e.g., the iframe scrolling contents layer is "#document"
instead of "iframe#frame") and there are a different number of
layers, but the test passes otherwise.

layer-compositing-reasons.js:
This test dumps the compositing reasons for all layers, directly from
the GraphicsLayer object. With RLS, the root layer has two additional
reasons: overflowScrollingTouch and rootScroller.

Bug: 781428
Change-Id: Ibc1e654474c96d2dd8dfdfc01ec52046989c30cb
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_root_layer_scrolls
Reviewed-on: https://chromium-review.googlesource.com/804620
Commit-Queue: Philip Rogers <pdr@chromium.org>
Reviewed-by: Stefan Zager <szager@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521914}

Especially in order to get the new lint for 'web-platform.test'
(https://github.com/w3c/web-platform-tests/pull/6868)

Chromium custom patch for lint.py is modified to accompany
upstream changes.

Bug: 749879
Change-Id: Ic2fb4655441f1d0e3bccbb7742565cb55645cf8f
Reviewed-on: https://chromium-review.googlesource.com/809745
Commit-Queue: Robert Ma <robertma@chromium.org>
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521913}

This CL changes FrameSinkManagerTest to use CompositorFrameSinkSupport
instead of FrameSinkManagerClient. This will allow for
FrameSinkManagerClient to be deleted in a follow up CL and
FrameSinkManagerImpls internal data structures to be cleaned up.

Also rename SurfaceManagerOrderingParamTest to
FrameSinkManagerOrderingParamTest. This wasn't updated previously.

Bug: 792192
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I0ad3f1e8f8530dccf001fe81c6c9191176dbaddf
Reviewed-on: https://chromium-review.googlesource.com/809614Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521912}

A self-referencing css stylesheet can attempt to preload itself reentrantly.
Ensure a HTMLResourcePreloader doesn't scan the same Resource twice.

Bug: 789198, 790940, 790945
Change-Id: I5a5ca56e3c12978c4a8b7fcbba79ae2a772671f8
Reviewed-on: https://chromium-review.googlesource.com/804496Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521911}

Test redirects with PUT.
Test redirect URL edgecases.

Bug: chromium:787197
Change-Id: I663516828741282f3a6364d71a4fc822a3814045
Reviewed-on: https://chromium-review.googlesource.com/804713
Commit-Queue: Sergei Datsenko <dats@chromium.org>
Reviewed-by: Alex Clarke <alexclarke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521910}

1. alt+left/right

Chrome only fires focus events in this case. This change tries to
recover focus whenever a focus event gets fired on a root web area. The
challenge here is the root node gets re-used. Explicit tracking of the
docurl gets introduced as a result.

Forum discussion:
https://groups.google.com/forum/#!topic/axs-chrome-discuss/u9gcqZxDISM

2. toggle on ChromeVox

In this case, ChromeVox tries to always recover focus. Detected by
checking there is no |currentRange|.

3. ctrl+r

In this case, ChromeVox receives a load complete. Only recover if the
|focus| is not already inside of the page (i.e. |focus| is the top level
root).

restores its range appropriately.

Test: exercise each of the above examples; verify that ChromeVox
Bug: 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I6e427105555b2663dca1100a02b8eb6b00cdc08a
Reviewed-on: https://chromium-review.googlesource.com/806759
Commit-Queue: David Tseng <dtseng@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521909}

It came up in review of
be able to call TypingCommand.cpp's GetSelectionOffsets() method on an arbitrary
SelectionInDOMTree (currently the method takes a LocalFrame and returns the
offsets for the frame's current selection), so I am making that change in this
CL.

https: //chromium-review.googlesource.com/c/chromium/src/+/801979 that we want to
Change-Id: I2e0d580a17819ea7136cd973de6a3aa4685dae9c
Reviewed-on: https://chromium-review.googlesource.com/804416Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: yosin (OOO Dec 11 to Jan 8) <yosin@chromium.org>
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Ryan Landay <rlanday@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521908}

This reverts commit 1aed2aaf.

Reason for revert: Causing merge conflicts on a CL that needs to be merged.  (See https://crbug.com/786505.)  Can land again afterward.

Original change's description:
> Move site_isolation_policy from content/common to content/browser.
> 
> The move allows returning
> ContentBrowserClient::GetOriginsRequiringDedicatedProcess from
> SiteIsolationPolicy::GetIsolatedOrigins.
> 
> No expected behavior change.
> 
> Bug: 792162
> Change-Id: Idd79fbddf138c2043ff1811a2d3d58165bcb198b
> Reviewed-on: https://chromium-review.googlesource.com/794303
> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
> Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521893}

TBR=alexmos@chromium.org,lukasza@chromium.org

Change-Id: I419b78c56adb2126eb93367e29bc569a004308f7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 792162
Reviewed-on: https://chromium-review.googlesource.com/810012Reviewed-by: Charlie Reis <creis@chromium.org>
Commit-Queue: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521907}

Change-Id: Ib33feac507cf37c9bd4351b3fede05ea9fea8806
Bug: none
Reviewed-on: https://chromium-review.googlesource.com/797678
Commit-Queue: Adam Langley <agl@chromium.org>
Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521906}

This reverts commit b6e038a7.

Reason for revert: Broke stack_tool_with_logcat_dump

Traceback (most recent call last):
  File "/b/c/b/WebKit_Android__Nexus4_/src/third_party/android_platform/development/scripts/stack", line 247, in <module>
    sys.exit(main(sys.argv[1:]))
  File "/b/c/b/WebKit_Android__Nexus4_/src/third_party/android_platform/development/scripts/stack", line 238, in main
    arch_defined, symbolizer)
  File "/b/c/b/WebKit_Android__Nexus4_/src/third_party/android_platform/development/scripts/stack_core.py", line 192, in ConvertTrace
    ResolveCrashSymbol(list(useful_log), more_info, llvm_symbolizer)
  File "/b/c/b/WebKit_Android__Nexus4_/src/third_party/android_platform/development/scripts/stack_core.py", line 331, in ResolveCrashSymbol
    if pid != -1 and pid in java_stderr_by_pid:
NameError: global name 'java_stderr_by_pid' is not defined

https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/72002

Original change's description:
> Use llvm symbolizer for stack script.
> 
> llvm symbolizer is a more efficient symbolizer than addr2line, objdump,
> etc.
> 
> In this cl, I 1)created a wrapper instance to interact with
> llvm symbolizer, 2)made the stack script to use llvm symbolizer instance,
> and then 3)added llvm symbolizer into isolated inputs.
> 
> Bug: 774267
> Change-Id: I971fb808b97f3a569eb9615f99efa41e3a56f3cb
> Reviewed-on: https://chromium-review.googlesource.com/789376
> Reviewed-by: agrieve <agrieve@chromium.org>
> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> Commit-Queue: Zhiling Huang <hzl@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521820}

TBR=dpranke@chromium.org,agrieve@chromium.org,hzl@chromium.org,bpastene@chromium.org,jbudorick@chromium.org

Change-Id: I166549a6fbfcd04529096cbe65aa5befb767b4ce
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 774267
Reviewed-on: https://chromium-review.googlesource.com/810010Reviewed-by: Ojan Vafai <ojan@chromium.org>
Commit-Queue: Ojan Vafai <ojan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521905}

Only the final URL (after all the redirect hops) should matter for
choosing the renderer process to host a frame after a navigation (this
is definitely true for PlzNavigate but should also be true without
PlzNavigate).  Therefore - this CL removes the
ShouldSwapProcessesForRedirect method - this method is not really
needed now.

Bug: 655214
Change-Id: I18c03d0ebcbb7a0fd673c52bc451186ec909f78b
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Tbr: sky@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/792005
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521904}

When mus does not host viz, aura handles the real AcceleratedWidget. So
in such cases, WindowTreeHostMus needs to wait for the widget to be
avaiable, which is still created by mus and sent to aura over mojom.

BUG=786453, 791619

Change-Id: I335365bfe4793c8fafb8d08eaa45c09c50e17a59
Reviewed-on: https://chromium-review.googlesource.com/807887Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521903}

FeatureSwitch::yield_between_content_script_runs() controls the
experiment to yield between subsequent content script injections. Now,
the metrics Features API (base::Feature[List]) has all the same
capabilities as FeatureSwitch (commandline, runtime, and variation
toggling). Migrate yield_between_content_script_runs to use the Features
API, and remove the FeatureSwitch member.

Note: a followup will remove FeatureSwitch finch support, which will be
unused with the removal of this FeatureSwitch.

Bug: 784476

Change-Id: I780695e90b83b2037b7f601efa42f0871da24303
Reviewed-on: https://chromium-review.googlesource.com/797893
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521902}

This is a reland of 0ea77a78
Original change's description:
> [DevTools] Switch shared worker inspection from routing id to SharedWorkerHost
>
> This is a preparation to replace routing id + legacy IPC with Mojo interface
> retrievable from SharedWorkerHost.
>
> Temporarily duplicated some code from WorkerDevToolsAgentHost, which
> makes it possible to migrate shared and service workers separately.
>
> Removed a test for WORKER_STATE manipulations altogether with the state itself.
>
> Bug: 776009
> Change-Id: Id884f4f5009eb70148251c94cae9629d5a1f6e8e
> Reviewed-on: https://chromium-review.googlesource.com/783110
> Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
> Reviewed-by: Tsuyoshi Horo <horo@chromium.org>
> Reviewed-by: Alexei Filippov <alph@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521277}

TBR=alph@chromium.org, horo@chromium.org

Bug: 776009
Change-Id: I0c0bbf56e5faf44fbf4cb7e12ea2b507c840fed2
Reviewed-on: https://chromium-review.googlesource.com/809466Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Tsuyoshi Horo <horo@chromium.org>
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521901}

https://chromium.googlesource.com/catapult.git/+log/37c17ee783f3..09fc536c66c6

$ git log 37c17ee78..09fc536c6 --date=short --no-merges --format='%ad %ae %s'
2017-12-04 rnephew [Telemetry/TADA] Give StoryExpectations the ability to decode TADA expectation files
2017-12-05 simonhatch Pinpoint - Grab histogram summary metric equivalent.
2017-12-05 rnephew [Telemetry] Add expectations path to telemetry config object.
2017-12-05 rnephew [Telemetry/TADA] Fix bug where story name cannot be url.

Created with:
  roll-dep src/third_party/catapult


The AutoRoll server is located here: https://catapult-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: Id1667dc459991b7aa9f3c21f83ddc20a27c7eec1
Reviewed-on: https://chromium-review.googlesource.com/809748
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521900}

Bug: 763980
Change-Id: Ic7f5e4c37293767c3fb38c345880f2a07e053385
Reviewed-on: https://chromium-review.googlesource.com/808747Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521899}

This reverts commit b6e038a7.

Reason for revert: Broke stack_tool_with_logcat_dump on WebKit Android (Nexus4): https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/72002

Original change's description:
> Use llvm symbolizer for stack script.
> 
> llvm symbolizer is a more efficient symbolizer than addr2line, objdump,
> etc.
> 
> In this cl, I 1)created a wrapper instance to interact with
> llvm symbolizer, 2)made the stack script to use llvm symbolizer instance,
> and then 3)added llvm symbolizer into isolated inputs.
> 
> Bug: 774267
> Change-Id: I971fb808b97f3a569eb9615f99efa41e3a56f3cb
> Reviewed-on: https://chromium-review.googlesource.com/789376
> Reviewed-by: agrieve <agrieve@chromium.org>
> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> Commit-Queue: Zhiling Huang <hzl@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521820}

TBR=dpranke@chromium.org,agrieve@chromium.org,hzl@chromium.org,bpastene@chromium.org,jbudorick@chromium.org

Change-Id: Ieebbcb9527dafd25a0fc74d2826a31162495c86e
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 774267
Reviewed-on: https://chromium-review.googlesource.com/809941Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521898}

Fixes regression on non-ozone platforms and introduced by
  73d609f3
  https://chromium-review.googlesource.com/787290

Note that the format is only used for sanity check. All the logic is
done automatically in the vaapi driver's implementation of vaPutSurface.

Bug: 785201
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Idc0bdf184874bf2c238e28da6f42f4e3572e9743
Reviewed-on: https://chromium-review.googlesource.com/807928Reviewed-by: Dan Sanders <sandersd@chromium.org>
Commit-Queue: Julien Isorce <julien.isorce@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521897}

Bug: 757243
Change-Id: I4e82b842301cc824f6113229bf99aab5add15b74
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/809939Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521896}

Bug: 725683
Change-Id: Ifb0ced19dc80664810821edb2434381a8e81179f
Reviewed-on: https://chromium-review.googlesource.com/791941Reviewed-by: Gayane Petrosyan <gayane@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Xiaoqian Dai <xdai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521895}

The proposed metrics can help us making compositing related decisions in
the future. They could also help locating regression reasons.

Bug: 776863
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I7cec76d6112b99fb1d02cff3a8f2ae46dd06f2c3
Reviewed-on: https://chromium-review.googlesource.com/806690
Commit-Queue: Yi Gu <yigu@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521894}

The move allows returning
ContentBrowserClient::GetOriginsRequiringDedicatedProcess from
SiteIsolationPolicy::GetIsolatedOrigins.

No expected behavior change.

Bug: 792162
Change-Id: Idd79fbddf138c2043ff1811a2d3d58165bcb198b
Reviewed-on: https://chromium-review.googlesource.com/794303
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521893}

Bug: 776826
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I4c94e829690b37da7029fdf1105731fbf6c2b0b6
Reviewed-on: https://chromium-review.googlesource.com/809607Reviewed-by: Justin Novosad <junov@chromium.org>
Commit-Queue: Olivia Lai <xlai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521892}

The feature has been fully rolled out to stable for more than a month.
We should now be able to start doing feature cleanups.

Bug: 718154
Change-Id: I874c5517003badc37c8fc0e29ca6e96c9a4eaae1
Reviewed-on: https://chromium-review.googlesource.com/794715
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521891}

The proxying for __NR_readlink is successful under test, but
the real-life processes need permission to use it.

Change-Id: Ia02b80f5190c2b48458ad2f48693f9bc093464a2
Reviewed-on: https://chromium-review.googlesource.com/809705Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521890}

This patch checks that |sensor_provider_| has not been reset before
attempting to fetch a fallback sensor after an initial initialization
error. This could happen if the SensorProvider interface is unavailable.

Bug: 788120
Change-Id: I5afd03929e83ae4a35353dcc21f0c48ccbdb0066
Reviewed-on: https://chromium-review.googlesource.com/794519Reviewed-by: Tim Volodine <timvolodine@chromium.org>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521889}

Due to some bug within the translation pipeline, translated grd files
sometimes end up with multiple plural entries that have the same/similar
key, eg: '=1' and 'one'. This change makes sure that the xml output only
has one entry in per key. This is required by aapt2, which fails with an error
otherwise.

Bug: 787488
Change-Id: I6d98c1739a7dbd3872c31cbfca63b0bd5612c79d
Reviewed-on: https://chromium-review.googlesource.com/807148
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521888}

Re-enables VrShellControllerInputTest#testControllerScrolling which was
disabled due to a flake. Was unable to reproduce the flake after
running the test on the swarmed devices with the same gn args 500 times.

Bug: 786200
Change-Id: Iaf8e362381dd2da60ac2e155de99af0d2535e0b4
Reviewed-on: https://chromium-review.googlesource.com/809884Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521887}

Using wpt-import in Chromium 37c44039.

Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/6950

Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

TBR=qyearsley

No-Export: true
Change-Id: Ib62ee0569b97dcbdd5ced54961b91c9dbb8c2bb3
Reviewed-on: https://chromium-review.googlesource.com/809645
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521886}

Also moves the tab_stats_data_store tests to unit_tests as they're not
browser tests.

I'm adding some new stats to this class and having the
browser/unit_tests separation will ease the unittesting.

Bug: 720131
Change-Id: I9a2a9c06e0968ef9b537e2eb119934590038d3f0
Reviewed-on: https://chromium-review.googlesource.com/804435
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521885}

Read the new external policy, enforce the selected policies, provide
printers to clients.  Code path is not connected yet.

Bug: 758680
Change-Id: I178d0afd1e0e60d5c0cdaa8adfebb14d1d5d7070
Reviewed-on: https://chromium-review.googlesource.com/677865Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Sean Kau <skau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521884}

Preparation for removing ScriptSourceCode::GetResource().

Bug: 788828
Change-Id: Ic1196997f2b7a0dd6f7febca476d45c4f4128598
Reviewed-on: https://chromium-review.googlesource.com/786891
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521883}

This reverts commit 363560ef.

Reason for revert: The revert broke compile. Will reland and disable the failing test instead.

Original change's description:
> Revert "Run constraints processing for screen capture on the main thread."
> 
> This reverts commit 112c4ff7.
> 
> Reason for revert: This might have broken browser_tests on Mac ASan 64 Tests (1): https://uberchromegw.corp.google.com/i/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/36626
> 
> Original change's description:
> > Run constraints processing for screen capture on the main thread.
> > 
> > Some code paths require access to Blink strings that can only be
> >  accessed on the main thread.
> > A test that triggers the bug is coming up on crrev.com/c/738040.
> > 
> > Bug: 791992
> > Change-Id: I55ae617ae623766d4bd975a19070bde410d8af1d
> > Reviewed-on: https://chromium-review.googlesource.com/809005
> > Reviewed-by: Henrik Boström <hbos@chromium.org>
> > Commit-Queue: Guido Urdaneta <guidou@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#521708}
> 
> TBR=hbos@chromium.org,guidou@chromium.org
> 
> Change-Id: Ibe39bacdedfbf54fac48a6469ec59c4f32381fd1
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 791992
> Reviewed-on: https://chromium-review.googlesource.com/809240
> Reviewed-by: Derek Cheng <imcheng@chromium.org>
> Commit-Queue: Derek Cheng <imcheng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521872}

TBR=hbos@chromium.org,imcheng@chromium.org,guidou@chromium.org

Change-Id: Ia5cf4ca4e14d3f83aa93c6e9aad71c3348a2467d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 791992
Reviewed-on: https://chromium-review.googlesource.com/809887Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521882}

Sample extension that uses command API to give user alternate
choice of keyboard shortcuts to switch between tabs.
Also moves existing sample command extension into a new
directory.

Bug: None
Change-Id: Ia5c4ae7134c51a1730260a5f23f23c091231ff38
Reviewed-on: https://chromium-review.googlesource.com/749115
Commit-Queue: Crystal Lambert <crystallambert@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521881}

Originally, it was only used when Print Preview is enabled, but the
original user no longer exists, and the only user is the Settings page.
Transitively only build settings::PrintingHandler when printing is
enabled.

BUG=789790

Change-Id: I8e8b9d0df66b1a5ead01abeeaad78982f16e61ec
Reviewed-on: https://chromium-review.googlesource.com/806447Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521880}

Currently we've got some bugs that can lead to crashes on standalone
VR headsets (https://vr.google.com/daydream/standalonevr/), so we
should detect devices that boot into VR, and disable VR there.

This will make sure we just use the fallback 2D-in-VR path that is
supposed to work for all apps :)

This also fixes a bug in scheduling rAF when presentation is rejected
that lead to the rAF loop doubling up every time presentation was
rejected.

Bug: 791090
Change-Id: I7511299935486dd018b558f3489ddd00f314ba8e
Reviewed-on: https://chromium-review.googlesource.com/809194
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521879}

https://skia.googlesource.com/skia.git/+log/06ab3836f779..a3e9271ec41d

$ git log 06ab3836f..a3e9271ec --date=short --no-merges --format='%ad %ae %s'
2017-12-04 csmartdalton CCPR: Eliminate texel buffers
2017-12-05 reed add serial procs to pictures
2017-12-05 scroggo Reland "Hide SkEncodedInfo"
2017-12-05 mtklein ANGLE -> Clang
2017-12-05 csmartdalton Add analytic clip FPs that read from the CCPR atlas
2017-12-05 mtklein remove mips support and bots
2017-12-05 bsalomon Avoid GrEllipseEffect for small radii on devices without 32 bit float.
2017-12-05 brianosman Keep animation running while moving the mouse in viewer (on Windows)
2017-12-05 scroggo Revert "Hide SkEncodedInfo"
2017-12-05 halcanary android_build_app: fixes
2017-12-04 brianosman Fix sample modifier keys for mouse events
2017-12-04 scroggo Hide SkEncodedInfo

Created with:
  roll-dep src/third_party/skia


The AutoRoll server is located here: https://autoroll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=rmistry@chromium.org

Change-Id: I457e7c515acd46b4cc0d4ee7a974b12d815248d0
Reviewed-on: https://chromium-review.googlesource.com/809806Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521878}

This changes the WPT import script to use the URL
https://build.chromium.org/deprecated/chromium/all_rotations.js
for fetching sheriff rotations.

Bug: 792193
Change-Id: I64bc3396bc921cc7f1e0420f872881fb2b25ebac
Reviewed-on: https://chromium-review.googlesource.com/809616Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521877}

WebStateImpl will cache favicon urls in OnFaviconUrlUpdated and will
call WebStateObserver::FaviconUrlUpdated in OnNavigationFinished for
same document navigations.

Also removed favicon urls caching in WebFaviconDriver introduced in
crrev.com/c/695761. This CL is not exactly a revert of
crrev.com/c/695761, because deprecated NavigationItemCommitted is
replaced with DidFinishNavigation. NavigationItemCommitted is not
called for push/replace state same-document navigaiton, which may
cause other bugs for favicons caching.


Bug: 789581
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I42922ab4d7812380787e350713640efe4d599680
Reviewed-on: https://chromium-review.googlesource.com/797072Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521876}