GitLab

Previously we added some debugging instrumentation in TouchActionFilter,
when the allowed_touch_action_ is set or reset to non-Auto. The results
that we see is not enough to diagose the crash.

This CL adds more detailed debugging. In particular, whenever it is set
or reset, we add a string.

Bug: 851644
Change-Id: I2d19055c1008659365f9c3f54fcbdb30760b2166
Reviewed-on: https://chromium-review.googlesource.com/1174696Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583087}

https://chromium.googlesource.com/chromiumos/platform/system_api.git/+log/a2a458bd5d69..c0be499563b8

$ git log a2a458bd5..c0be49956 --date=short --no-merges --format='%ad %ae %s'
2018-08-13 kerrnel cryptohome: Add MOUNT_ERROR_REMOVE_FAILED.
2018-08-07 chirantan seneschal: Add storage location field
2018-08-07 mcchou service_contants: Add kErrorAlreadyExists to bluetooth_device namespace
2018-07-25 hendrich shill: Remove PriorityWithinTechnology property
2018-06-18 akhouderchah shill: Added new error strings for Service::ConnectFailure additions
2018-07-13 ljusten dbus/login_manager: Add deletion error string
2018-06-13 lndmrk Add DBus constant permission_broker::kPowerCycleUsbPorts

Created with:
  roll-dep src/third_party/cros_system_api

Bug: 211405
Change-Id: Iffa3919af789c351199aba156ccea181adcb8fdb
Reviewed-on: https://chromium-review.googlesource.com/1175005Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Greg Kerr <kerrnel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583086}

ChromeDriver was not starting successfully on Mac when --whitelisted-ips
option was used. This CL fixes it, while ensuring that ChromeDriver
always listen on the same IPv4 and IPv6 ports

Bug: chromedriver:2541
Change-Id: I2024e3c27ea0fcf262c7b5c54791b86ae6a3e0c1
Reviewed-on: https://chromium-review.googlesource.com/1175059
Commit-Queue: John Chen <johnchen@chromium.org>
Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583085}

This CL may cause a small binary size increase, roughly proportional
to how long it's been since our last AFDO profile roll. For larger
increases (around or exceeding 100KB), please file a bug against
gbiv@chromium.org. Additional context: https://crbug.com/805539

The AutoRoll server is located here: https://afdo-chromium-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.


TBR=gbiv@chromium.org

Change-Id: I132e5ebea28a97bd88b82f0d421ca716468822f4
Reviewed-on: https://chromium-review.googlesource.com/1175213
Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#583084}

Previously, the login screen would crash in both these cases (e.g. if
the powerwash confirmation dialog popped up and the user pressed
left/right arrow).

Change-Id: I5880d72112508de93e77c507239ea28c475d86ae
Reviewed-on: https://chromium-review.googlesource.com/1174907
Commit-Queue: Quan Nguyen <qnnguyen@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583083}

The sanity test looks much more stable:
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/chromeos-amd64-generic-rel-vm-tests

This should be safe to land now.

Bug: 869904
Change-Id: Ic0d7e5537e9212b7f7aaf0250041c5a73353279d
Reviewed-on: https://chromium-review.googlesource.com/1174968Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583082}

This is a bit closer to webui-login's behavior (although currently it doesn't
gracefully handle bubbles extending past the bottom of the screen). This also
prevents the auth error bubble from suddenly jumping to the right if it extends
into the shelf a bit.  This Cl also changes the error bubble bounds calculation
to include all of the black box, rather than just the internal content.

Bug: 873953
Change-Id: I13d670a208418f77232c59b756fb7854e1c88cc2
Reviewed-on: https://chromium-review.googlesource.com/1175196Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Commit-Queue: Quan Nguyen <qnnguyen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583081}

Line length should not exceed 376dip and should have a line height of
24dip.

See bug for spec/demo.

Bug: b:112597108
Change-Id: Ic0b920d33bc0d571b98f7622a0e9d0fb965c74f1
Reviewed-on: https://chromium-review.googlesource.com/1175057Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Commit-Queue: David Black <dmblack@google.com>
Cr-Commit-Position: refs/heads/master@{#583080}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/db93180205a6..a2f48ac5546b


git log db93180205a6..a2f48ac5546b --date=short --no-merges --format='%ad %ae %s'
2018-08-14 dgarrett@google.com chromeos_config: Disable chrome_binhost_only globally.


Created with:
  gclient setdep -r src/third_party/chromite@a2f48ac5546b

The AutoRoll server is located here: https://chromite-chromium-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.


TBR=chrome-os-gardeners@chromium.org

Change-Id: I1355c118e89be711526f97554a00095b413eb52b
Reviewed-on: https://chromium-review.googlesource.com/1174798Reviewed-by: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#583079}

Previously we thought this was only flaky on devices. It is also flaky
on simulator.

TBR=justincohen, javierrobles

Bug: 873633
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Iddc823fc1034ba26db75da1b42a63a864e58878a
Reviewed-on: https://chromium-review.googlesource.com/1175194Reviewed-by: edchin <edchin@chromium.org>
Commit-Queue: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583078}

This CL removes unused SqlitePersistentCookieStore::Close().

Bug: 801910
Change-Id: Ibba90aaeb5c07215de73e85f33bb5c4067cf9f42
Reviewed-on: https://chromium-review.googlesource.com/1174948Reviewed-by: Maks Orlovich <morlovich@chromium.org>
Commit-Queue: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583077}

Tbr: tbuckley@google.com
Change-Id: Ia9079c26605d5cee901b46a13335985ca4ec9e49
Reviewed-on: https://chromium-review.googlesource.com/1174575
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583076}

Move AppListView::ExcludeWindowFromEventHandling from browser process
to app list code in ash process.

Bug: 854787
Change-Id: I7509fdd8c5cc4ea0db211bdc4c7fe8715fb10660
Reviewed-on: https://chromium-review.googlesource.com/1170036
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583075}

Firebase target depends on a Firebase resources target instead of a
file path. Build configurations that want Firebase enabled should
define a target to bundle GoogleService-Info.plist to application
bundle.

At the same time, unit test and Earl Grey test targets need Firebase
SDK too.

Bug: 848117
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I902437afc93dcef39ccdbc89287638ae9d761afe
Reviewed-on: https://chromium-review.googlesource.com/1162713
Commit-Queue: Peter Lee <pkl@chromium.org>
Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583074}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/5ce57437..4f1630e3

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com

Change-Id: I40321b5e2126f7f905e77c76998acc5246e51e59
Reviewed-on: https://chromium-review.googlesource.com/1175088Reviewed-by: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#583073}

Verified it works on a Paine. This is a regression caused by
https://chromium-review.googlesource.com/c/chromium/src/+/1138861

Bug: 873609
Change-Id: I929fb3be9f0f811d901811aebf34c8c52ea42bf2
Reviewed-on: https://chromium-review.googlesource.com/1175192Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583072}

This reverts commit 3dcaec6e.

Reason for revert: Failing Linux MSan tests

https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20MSan%20Tests/11191

DevTools listening on ws://127.0.0.1:34267/devtools/browser/a12caedc-549b-4d49-b9bb-a0519f91f318
==14126==WARNING: MemorySanitizer: use-of-uninitialized-value
    #0 0x3c2bb23 in ExitFullscreenModeForTab ./../../content/browser/web_contents/web_contents_impl_browsertest.cc:1561:9

Original change's description:
> Security drop fullscreen for any nested WebContents level.
> 
> BUG=873080
> TEST=as in bug
> 
> Change-Id: Icd75715ac42789c84870967b11c9917a972ae086
> Reviewed-on: https://chromium-review.googlesource.com/1173342
> Commit-Queue: Avi Drissman <avi@chromium.org>
> Reviewed-by: Sidney San Martín <sdy@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#582970}

TBR=avi@chromium.org,sdy@chromium.org

Change-Id: I52a9b29f2d17ea6e7db0128815653de624fa5b13
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 873080
Reviewed-on: https://chromium-review.googlesource.com/1174640Reviewed-by: Chris Mumford <cmumford@chromium.org>
Commit-Queue: Chris Mumford <cmumford@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583071}

Now that we display a wider range of web content, some sites are
serving desktop sites with images too large for our memory limits
and so on.

Longer-term, it may be possible to use this UA string for all products.

Bug: internal b/112465626
Change-Id: I5c30b1966f75143eb5d99efabfb9093bf6159e90
Reviewed-on: https://chromium-review.googlesource.com/1175087Reviewed-by: Alex Sakhartchouk <alexst@chromium.org>
Commit-Queue: Luke Halliwell <halliwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583070}

1. Employs a static local variable to store the single instance which is thread safe.
Also prevents destruction which is not thread safe.
2. Took the opportunity to do some cleanup:
Added "const" and "* const" to some places.
Move the COMInitializer to the tools module because there should only be one instance per module and content_browsertests already had one instance.
Used FAILED instead of !SUCCEEDED.
R=sdi@chromium.org, mcasas@chromium.org, aleventhal@chromium.org

Change-Id: Ie4ed3db74342e42c9325a647401b03a44aa0cba2
Reviewed-on: https://chromium-review.googlesource.com/1171562
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Reviewed-by: Miguel Casas <mcasas@chromium.org>
Reviewed-by: Sidney San Martín <sdy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583069}

Devices using certain legacy Empia chips appear to get listed and report valid
formats in both DirectShow and MediaFoundation but only work in DirectShow when
actually trying to open them.

This CL adds a blacklisting mechanism that forces device models known to exhibit
this issue to always get opened with DirectShow instead of MediaFoundation.

Test: capture_unittests.exe --gtest_filter=VideoCaptureDeviceFactoryMFWinTest*
Bug: 792640, 849636, 862395
Change-Id: Ia70d13a45eb49ed185f27630d0e544f3a484b6f2
Reviewed-on: https://chromium-review.googlesource.com/1174902Reviewed-by: Emircan Uysaler <emircan@chromium.org>
Commit-Queue: Christian Fremerey <chfremer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583068}

Crash dumps are unclear and the issue seems resolved on Pixel hardware,
but some earlier stack traces suggest we're getting a null TaskRunner
when attempting to BindToCurrentLoop during CreateCodec(). As such
move binding to a known good thread.

This should be reverted if the issue is not fixed. If it is fixed
there's additional work to be done to understand the consequences
of CreateCodec() being called on an unexpected thread.

BUG=873094
TEST=none

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I2928d7866c1f40de63f526ee7393d2d8b90884f2
Reviewed-on: https://chromium-review.googlesource.com/1175092Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583067}

This caps these devices support for hardware decoding to 1080p.

BUG=873996
TEST=none

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I4b640cf8537a33518d06994d9aaeac772e428b8e
Reviewed-on: https://chromium-review.googlesource.com/1175095
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583066}

Use network quality tracker instead of network quality estimator (NQE)
to obtain network quality in data reduction proxy component.

Network quality tracker can provide network service when network
service is enabled as well as when network service is disabled.

data_reduction_proxy_service.h receives network quality
estimates on UI thread from network quality tracker.
On receiving estimates, it posts them to d_r_p_io_data
which lives on IO thread, and consumes the estimates.

PS#2 is the original CL that got reverted.

Bug: 819244
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Id9f28abaea42a7eb55dd3647c88d13a427a4e5ae
TBR: xunjieli@chromium.org, sky@chromium.org, ryansturm@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/1174698
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583065}

Bug: 866941
Change-Id: I1c5a6a8117eb347dbe838ae1818e726ff1dafcb7
Reviewed-on: https://chromium-review.googlesource.com/1175062Reviewed-by: Esmael El-Moslimany <aee@chromium.org>
Commit-Queue: Scott Chen <scottchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583064}

This reverts commit 562c52f7.

Reason for revert: cc_unittests failing on Linux TSan:

https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20TSan%20Tests/24797

[ RUN      ] LayerTreeHostTilesTestPartialInvalidation.PartialRaster_SingleThread_OneCopy
[22487:22487:0814/143420.957209:26522082454:ERROR:pixel_comparator.cc(47)] Number of pixel with an error: 30000
[22487:22487:0814/143420.957442:26522082676:ERROR:pixel_comparator.cc(48)] Error Bounding Box : 0,0 200x200
[22487:22487:0814/143421.140401:26522265659:ERROR:pixel_test_utils.cc(79)] Pixels do not match!
[22487:22487:0814/143421.140586:26522265816:ERROR:pixel_test_utils.cc(80)] Actual: data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAMgAAADICAYAAACtWK6eAAACHUlEQVR4nO3VMRHAMBADwXf4ozUBh0DmWqfYRaDmRmvvOQN8em4PgD8TCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQBAIBIFAEAgEgUAQCASBQFgz+9weAX/lQSAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAIAgEgkAgCASCQCAIBIJAILygpAZjjfRdXgAAAABJRU5ErkJggg==
[22487:22487:0814/143421.140737:26522266017:ERROR:pixel_test_utils.cc(81)] Expected: data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAMgAAADICAIAAAAiOjnJAAACNklEQVR4nO3dsQ0DMQwEQb3hWP3XqQrcgEMvnm/MVMBgwfSutc6CX3vdfQD/SVgkhEVCWCSERUJYJIRFQlgkhEVCWCSERUJYJIRFQlgkhEVCWCSERUJYJIRFQlgkhEVCWCSERUJYJIRFQlgkhEVCWCSERUJYJIRFQlgkhEVCWCSERUJYJIRFQlgkhEVCWCSERUJYJIRFQlgkhEVCWCSERUJYJIRF4n33Ad+ds+8+4TH2nriR62OREBYJYZEQFglhkRAWCWGREBYJYZEQFglhkRAWCWGREBYJYZEQFglhkRAWCWGREBYJYZEQFglhkRAWCWGREBYJYZEQFglhkRAWCWGREBYJYZEQFglhkRAWCWGREBYJYZEQFglhkRAWCWGREBYJYZEQFglhkRAWCWGRuM7Eec6hq6EzzVyj9bFICIuEsEgIi4SwSAiLhLBICIuEsEgIi4SwSAiLhLBICIuEsEgIi4SwSAiLhLBICIuEsEgIi4SwSAiLhLBICIuEsEgIi4SwSAiLhLBICIuEsEgIi4SwSAiLhLBICIuEsEgIi4SwSAiLhLBICIuEsEgIi4SwSAiLhLBIDF1Y5el8LBLCIiEsEsIiISwSwiIhLBLCIiEsEsIiISwSwiIhLBLCIiEsEsIiISwSwiIhLBLCIiEsEsIiISwSwiIhLBLCIiEsEsIiISwSwiIhLBLCIiEsEsIiISwSwiIhLBLCIiEsEsIiISwSwiIhLBLCIiEsEsIiISwSwiIhLBIfCVIMShDYZKEAAAAASUVORK5CYII=
../../cc/test/layer_tree_pixel_test.cc:129: Failure
Value of: MatchesPNGFile(*result_bitmap_, ref_file_path, *pixel_comparator_)
  Actual: false
Expected: true
Stack trace:
#0 0x000001ceb5ff (/b/s/w/ir/out/Release/cc_unittests+0x1ceb5fe)
#1 0x0000017ae1e9 (/b/s/w/ir/out/Release/cc_unittests+0x17ae1e8)
#2 0x0000017bda1f (/b/s/w/ir/out/Release/cc_unittests+0x17bda1e)
#3 0x0000017bd178 (/b/s/w/ir/out/Release/cc_unittests+0x17bd177)
#4 0x000001a362be (/b/s/w/ir/out/Release/cc_unittests+0x1a362bd)
#5 0x000001a3b5ba (/b/s/w/ir/out/Release/cc_unittests+0x1a3b5b9)

[  FAILED  ] LayerTreeHostTilesTestPartialInvalidation.PartialRaster_SingleThread_OneCopy (4296 ms)

Original change's description:
> Roll src/third_party/skia 32c7d4dfcdca..d19fed1e291d (2 commits)
> 
> https://skia.googlesource.com/skia.git/+log/32c7d4dfcdca..d19fed1e291d
> 
> 
> git log 32c7d4dfcdca..d19fed1e291d --date=short --no-merges --format='%ad %ae %s'
> 2018-08-14 fmalita@chromium.org [skottie] Eliminate some temp SkString allocations
> 2018-08-14 mtklein@google.com pass sprite opacity to color xform when blitting.
> 
> 
> Created with:
>   gclient setdep -r src/third_party/skia@d19fed1e291d
> 
> 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;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
> TBR=kjlubick@chromium.org
> 
> Change-Id: Ib608389718b76c94782e3415bcc82dc607902e82
> Reviewed-on: https://chromium-review.googlesource.com/1174793
> Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
> Cr-Commit-Position: refs/heads/master@{#583019}

TBR=kjlubick@chromium.org,skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com

Change-Id: I0e6bc9c3c8c052a0ae17eb292cad8bb5e579faf2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Reviewed-on: https://chromium-review.googlesource.com/1174639Reviewed-by: Chris Mumford <cmumford@chromium.org>
Commit-Queue: Chris Mumford <cmumford@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583063}

Final cleanup after https://crrev.com/581931

Bug: internal b/73383411
Test: manual
Change-Id: I9407ebbaa7b189bf3b71aa1700451198ec5c5810
Reviewed-on: https://chromium-review.googlesource.com/1169586Reviewed-by: Alex Sakhartchouk <alexst@chromium.org>
Commit-Queue: Kevin Schoedel <kpschoedel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583062}

Port a couple related browser tests to ash unit tests.

Change-Id: Id1fcf3b823b3f470dc93d651c681738053cc3383
Reviewed-on: https://chromium-review.googlesource.com/1173152Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583061}

Bug: 874102
Cq-Include-Trybots: luci.chromium.try:closure_compilation
Change-Id: I64ae721146a6ac631800bd0bbd1ca6cca9f522f8
Reviewed-on: https://chromium-review.googlesource.com/1172741
Commit-Queue: David Tseng <dtseng@chromium.org>
Reviewed-by: Toni Baržić <tbarzic@chromium.org>
Reviewed-by: Katie Dektar <katie@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583060}

Previously, "OK Google"
Now, "Ok Google"

This string is used in Chrome OS settings.

Bug: b:112326023
Change-Id: Ic69713f93518241ade199034bce0a0c1063e89e1
Reviewed-on: https://chromium-review.googlesource.com/1171219Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Xiao Yang <yanxiao@google.com>
Cr-Commit-Position: refs/heads/master@{#583059}

"Answering on another device":
  To display when disambiguating "Ok Google" and we lose devie
  selection to another surface.

"What's on my screen":
  show on suggestion chip in Assistant UI to initiate a contextual search

Bug: b:112464186
Change-Id: Ia61f12d3c39f5005486824ad7982736719dfd14b
Reviewed-on: https://chromium-review.googlesource.com/1173263Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Commit-Queue: Xiao Yang <yanxiao@google.com>
Cr-Commit-Position: refs/heads/master@{#583058}

Bug: 854704
Change-Id: If3326d678e53b3ccff89bcd2ba98d02a0611b605
Reviewed-on: https://chromium-review.googlesource.com/1172739Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583057}

SafeStack [1] introduces a secondary thread stack called the unsafe
stack that also needs to be scanned for object references. Introduce
matching logic in oilpan to scan the unsafe stack for heap references.

[1] https://clang.llvm.org/docs/SafeStack.html

Bug: 864705
Change-Id: I376077bd985e2077aa3771101c1822e1570c7807
Reviewed-on: https://chromium-review.googlesource.com/1169772Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
Commit-Queue: Vlad Tsyrklevich <vtsyrklevich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583056}

This patch adds a flag indicating whether an iframe element is occluded
or has visual effects applied in the parent document; and adds the flag
to the IPC message containing the iframe's intersection with the
parent's viewport.

BUG=869049
R=kenrb@chromium.org,chrishtr@chromium.org

Change-Id: I0891a3e6740dbce39f0cf0120353feb8c9fca173
Reviewed-on: https://chromium-review.googlesource.com/1167960Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Stefan Zager <szager@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583055}

Bug: 851609
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: I2f261550d21f80e4b2143c0253a3867f11091885
Reviewed-on: https://chromium-review.googlesource.com/1175054
Commit-Queue: Eric Roman <eroman@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583054}

This fully enables support for context menu in iFrames.

Bug: 873660, 873662
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I3345f56f6812ef619c6ed5b7bcabd34e09de410f
Reviewed-on: https://chromium-review.googlesource.com/1174052
Commit-Queue: Mike Dougherty <michaeldo@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583053}

For those subfolders of //chrome/ without violations, add them to
gn check targets. Otherwise, add number of errors in comments.

Bug: 367595
Change-Id: I8a2dc703f9b415e47f6fa3c1910b6338dd02ae69
Reviewed-on: https://chromium-review.googlesource.com/1174095Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583052}

Currently, the |FidoRequestHandlerBase::AddAuthenticator| method
synchronously invokes the virtual |DispatchRequest| method, which
subclasses implement to process the request and invoke the response
handling callback. Most authenticators are instantiated by a
|FidoDiscovery| which does some asynchronous discovery work and then
calls |AddAuthenticator|.

Platform authenticators, on the other hand, are instantiated by
|MaybeAddPlatformAuthenticator|, which gets invoked synchronously with
request handler instantiation. This makes it possible to have a
synchronous code path from request handler instantation to response
callback invocation (i.e.  hairpinning). In those cases,
|AuthenticatorImpl| is unable to handle the response (see e.g.
https://cs.chromium.org/chromium/src/content/browser/webauth/authenticator_impl.cc?sq=package:chromium&dr&g=0&l=749).

This change makes |DispatchRequest| invocation asynchronous, which
eliminates any possibility of response callback hairpinning.

Bug: 678128
Change-Id: I6bf273775885abac1fa38fe12c964e930407cc7e
Reviewed-on: https://chromium-review.googlesource.com/1172917
Commit-Queue: Martin Kreichgauer <martinkr@google.com>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Reviewed-by: Jun Choi <hongjunchoi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583051}

TBR=kbr@chromium.org

Bug: 753483, 873914
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_layout_tests_slimming_paint_v2;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I958694f9db571b66c40baf144025d87acf824dc2
Reviewed-on: https://chromium-review.googlesource.com/1173169
Commit-Queue: Fernando Serboncini <fserb@chromium.org>
Reviewed-by: Fernando Serboncini <fserb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583050}

Specifically, adding new test suites webui_polymer2_browser_tests and
webui_polymer2_interactive_ui_tests that run a given list of tests and
explicitly excludes currently failing tests.

This will allow gradually getting to a state where all tests are passing
with the flag on.

Bug: 862472
Change-Id: Ia19ae36a4b2b63499b970e14c742b7719e118f83
Reviewed-on: https://chromium-review.googlesource.com/1132320
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583049}

This was removed in r577553 but I unintentionally reintroduced it
in r578809 (incorrect rebase).

This regressed https://crbug.com/860801#c18

R=kylechar@chromium.org

Bug: 860801
Change-Id: Ia42d24f915d271832b59c481ebe77ee1bc6fd11d
Reviewed-on: https://chromium-review.googlesource.com/1175060Reviewed-by: kylechar <kylechar@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583048}