GitLab

This reverts commit 5abd7bfc.

Reason for revert: Suspected of causing PointerLockBrowserTest flake as detailed on crbug.com/784236.

Original change's description:
> Enable render throttling for OOPIFs
> 
> Throttle OOPIFs when they are off screen or clipped. This adds a
> VisibilityObserver to RemoteFrames with LocalFrame parents, and
> sends an update across processes to the OOPIF when throttling status
> should change.
> 
> This also fixes ScreenOrientation browser tests which has some
> flakiness in OOPIF tests that appears when this patch is applied.
> 
> Bug: 761478
> Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
> Change-Id: I9bfb57058e534e796c23e238664a66a142be3c4a
> Reviewed-on: https://chromium-review.googlesource.com/719601
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
> Reviewed-by: Stefan Zager <szager@chromium.org>
> Commit-Queue: Ken Buchanan <kenrb@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#515875}

TBR=dcheng@chromium.org,kenrb@chromium.org,szager@chromium.org,alexmos@chromium.org

Change-Id: If713f93ff75ef4ea45853052fa05c3095f13e8aa
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 761478
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Reviewed-on: https://chromium-review.googlesource.com/765847Reviewed-by: Colin Blundell <blundell@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515923}

TBR=hajimehoshi@chromium.org

Bug: 780378
Change-Id: Ie3ced9c5c0a1914d9fa329d36209634d2ae55383
Reviewed-on: https://chromium-review.googlesource.com/763269Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515922}

This changes the popup permission to be inherited to incognito profiles.
While other content settings are not inherited due to security or privacy
concerns, inheriting sites allowed to show popups is save.

Bug: 762349
Change-Id: Ia31f6cea46edf8c705302ee4a414a40a8321756e
Reviewed-on: https://chromium-review.googlesource.com/672547Reviewed-by: Raymes Khoury <raymes@chromium.org>
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515921}

This is a follow up to https://chromium-review.googlesource.com/c/chromium/src/+/758869,
which added "ServiceWorker" to some of the constants. That approach seems misleading
since it looks like "ServiceWorkerKey" is a type of key on the same level as a "RegKey"
or "ResKey". Really we want to add "ServiceWorker" to all the constants, which we
can accomplish by making a namespace.

Bug: 746953
Change-Id: I61dffdaecffbeb088906f67907fdc47f0f81a5e1
Reviewed-on: https://chromium-review.googlesource.com/765556Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515920}

Add cleanup handlers that resets "user preferred languages" and "text
track kind user preference", or just reset them outright, to see if this
has any bearing on the problem (works locally.)

Bug: 664844
Change-Id: I6237ace717fbaf6f5203d8a051950763547e191f
Reviewed-on: https://chromium-review.googlesource.com/757102Reviewed-by: srirama chandra sekhar <srirama.m@samsung.com>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#515919}

This patch adds the missing DCHECK(mapping) to the PlatformSensorFusion::Factory
constructor, and fixes implementation of FakePlatformSensorProvider::GetMapping()
which otherwise caused the assertion hit on some test cases.

Change-Id: I6bd2d17c66bcc2bfed766a3a3cbc0ca7c5c846f9
Reviewed-on: https://chromium-review.googlesource.com/765487Reviewed-by: Alexander Shalamov <alexander.shalamov@intel.com>
Commit-Queue: Mikhail Pozdnyakov <mikhail.pozdnyakov@intel.com>
Cr-Commit-Position: refs/heads/master@{#515918}

This removes many uses of waitForVideo() etc. and makes the
control flow of the test clearer. Instead of having the
all-events-occurred handler magically calling reportTestSuccess,
it's laid out explicitly in each test.

Bug: chromium:777857
Change-Id: I034d8950f764df607069b3e4b5ce6baeb46eab9b
Reviewed-on: https://chromium-review.googlesource.com/758848
Commit-Queue: Patrik Höglund <phoglund@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515917}

WebString no longer needs to use WebPrivatePtr.
This CL replaces it with scoped_refptr for cleaner impls.

Bug: 781741
Change-Id: Icacecc2e2a35b59c1d16780625643844b76cbbfb
Reviewed-on: https://chromium-review.googlesource.com/762971
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515916}

Follow up to https://chromium-review.googlesource.com/c/chromium/src/+/721325

Bug: none
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I9caba1853bb8df851ceea954da9a222b82d6f3f3
Reviewed-on: https://chromium-review.googlesource.com/758653Reviewed-by: Moe Ahmadi <mahmadi@chromium.org>
Commit-Queue: Elodie Banel <lod@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515915}

Test added and passed.

Bug: 774303
Change-Id: I5b8d9a5494aacf8dad3b014788f5c9b5b4ca1f5d
Reviewed-on: https://chromium-review.googlesource.com/734332
Commit-Queue: Henrik Boström <hbos@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515914}

Now ChildThreadImpl needs to pass an IPC listener task runner to create
IPC::SyncChannel, but ChildThreadImpl can't refer schedulers. This CL
enables ChildThreadImpl to refer ChildScheduler by adding ChildScheduler
::IPCListenerTaskRunner function. Now the implementation
(RendererScheduler::IPCListenerTaskRunner) just returns base::
ThreadTaskRunnerHandle::Get().

This CL is just a preparation for flexible scheduling, and doesn't
change any behavior so far.

Bug: 780785
Change-Id: I46965cab95ca6f98dee56299df9ebea1462886d8
Reviewed-on: https://chromium-review.googlesource.com/760336Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515913}

The line position after NGLineBreaker broke the line was rarely
used that HandleOverflow() and Rewind() did not update it.

CL:742041 started using the line position as the inline size of
the line to determine the layout opportunity. This patch updates
the line position on overflow/rewind cases.

Bug: 636993
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ic8799f310ca43efaa372d36ee6cd64b6796bf4fe
Reviewed-on: https://chromium-review.googlesource.com/750442Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515912}

Uses autolayout in LocationBarView to position the omnibox textfield.

Bug: 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: If9f0f9ec65624785617bb3820d56509845f97e5e
Reviewed-on: https://chromium-review.googlesource.com/709215
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Rohit Rao (ping after 24h) <rohitrao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515911}

This fixes a Null-dereference READ error in AXMenuList.

It can get a issue while adding children
in AXObject::UpdateChildrenIfNecessary.
We check HasChildren() before accessing children to fix it.

Bug: 780931
Change-Id: Ia8442643a7ba06114453084d8208d25393dab865
Reviewed-on: https://chromium-review.googlesource.com/753167
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515910}

This adds a COMPONENT directive to //components/os_crypt/OWNERS, which
points to the right crbug.com component for os_crypt. The COMPONENT
directive has been added to OWNERS files recently (an example is
//components/password_manager/OWNERS) and is useful for developers
when filing bugs about known code locations. It is especially useful
for os_crypt, because neither the crbug.com component name nor its
description mentions "os_crypt", so it might be hard to find just by a
query in crbug.com.

Bug: None
Change-Id: I56f1f8289db573118f41a622d09e69aed3029e42
Reviewed-on: https://chromium-review.googlesource.com/765450Reviewed-by: Christos Froussios <cfroussios@chromium.org>
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515909}

The clean-up code touched a profile after the profiles were destroyed. This
patch fixes this issue.

Bug: 783538
Test: none
Change-Id: If010a424e0fbcac2188eb5546c1e4917214dcbc2
Reviewed-on: https://chromium-review.googlesource.com/765635Reviewed-by: Daichi Hirono <hirono@chromium.org>
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515908}

BUG=784278
TBR=dgozman

Change-Id: I79da59f309e7915b80f14cb882679f3788a245e3
Reviewed-on: https://chromium-review.googlesource.com/765640Reviewed-by: Takashi Sakamoto <tasak@google.com>
Commit-Queue: Takashi Sakamoto <tasak@google.com>
Cr-Commit-Position: refs/heads/master@{#515907}

We should not change the visibility of the message center during the notifications are being iterated. This patch makes the code of changing visibility executed asynchronously.

Bug: 782384
Change-Id: I36233312fdd5b3690b9c2e40597e51d8f16128e7
Reviewed-on: https://chromium-review.googlesource.com/762910
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515906}

This is a reland of 4470f02c. The webkit_lint fix had landed in r515892
Original change's description:
> Run network service trybot when its test filters are modified.
>
> Change-Id: I5d28f3512047871e436341f01b9bfd6e0baa2b8e
> Reviewed-on: https://chromium-review.googlesource.com/765575
> Reviewed-by: Ken Rockot <rockot@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#515890}

Change-Id: I73bd0931b2a1ef25db790dde6fa7e8e2fc4458ac
Reviewed-on: https://chromium-review.googlesource.com/765644Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515905}

Change-Id: I33ff7de2b8957516ce8d8b8dc0143264c8a2df69
Reviewed-on: https://chromium-review.googlesource.com/764519Reviewed-by: Colin Blundell <blundell@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515904}

https://chromium.googlesource.com/catapult.git/+log/388187b9746c..f4c21da08850

$ git log 388187b97..f4c21da08 --date=short --no-merges --format='%ad %ae %s'
2017-11-10 benjhayden Fix more references to results2.

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: I6a0dfb2430652f6b98701e2c7f54a7a83e5f327d
Reviewed-on: https://chromium-review.googlesource.com/765216
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515903}

Bug: b/69008819
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I2d7c9c77f19964227ba46c4e2c54b01c8124f346
Reviewed-on: https://chromium-review.googlesource.com/764795
Commit-Queue: Nick Harper <nharper@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515902}

content/browser/renderer_host/render_process_host_impl.cc also has a
ForwardRequest function in the anonymous namespace, which can collide
with the function in content/browser/renderer_interface_binders.cc

TBR=jam@chromium.org

Bug: 779444,746953
Change-Id: Iaa2c9f22ff0945b4a35fef689f2b60896d048fea
Reviewed-on: https://chromium-review.googlesource.com/765489
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515901}

r515443 swapped out the custom higlighting in the Profile Chooser for ink drops
instead. However, this introduced a regression where the buttons would no longer
highlight on focus. Fix by turning on highlight on focus mode for ink drops.

Bug: 784235
Change-Id: Ib45cbbc82644279496bbab42e97406de96b4db7c
Reviewed-on: https://chromium-review.googlesource.com/765612
Commit-Queue: Trent Apted <tapted@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515900}

https://chromium.googlesource.com/catapult.git/+log/aee17a9d0e6d..388187b9746c

$ git log aee17a9d0..388187b97 --date=short --no-merges --format='%ad %ae %s'
2017-11-10 benjhayden Move makeTestModel out of the global namespace in webview_startup_metric_test.

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: I74b33b34d0b574803ca09a74b8dc85972bdd4446
Reviewed-on: https://chromium-review.googlesource.com/764993
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515899}

Bug: 783587
Change-Id: I4ffd2a49e0b8f4ae93074b867b079b9562ecf80e
Reviewed-on: https://chromium-review.googlesource.com/765609Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515898}

This patch moves content::ResourceRequestCompletionStatus from
content/public/common/ to services/network/public/cpp, and renames
it to URLLoaderStatus so to be consistent with mojom mapped type name.

TBR: jam@chromium.org, mkwst@chromium.org
Bug: 736308, 748491
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I7608eba2a3c0419b1cbc1cf08313ac71eeb2e9c3
Reviewed-on: https://chromium-review.googlesource.com/756822
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515897}

This reverts commit 4470f02c.

Reason for revert: 
webkit_lint failing on multiple builders. e.g.
https://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests/builds/50152
Running ['/usr/bin/python', '/b/c/b/Mac10_9_Tests/src/third_party/WebKit/Tools/Scripts/lint-test-expectations', '--json', '/var/folders/9x/6c6sv3cj4j53wzpzthbp4ksm0000gm/T/tmpOSCHu4'] in None (env: None)
third_party/WebKit/LayoutTests/FlagExpectations/PRESUBMIT.py:4 expecting "[", "#", or end of line instead of "script" """Presubmit script for //third_party/WebKit/LayoutTests/FlagExpectations.
third_party/WebKit/LayoutTests/FlagExpectations/PRESUBMIT.py:4 Missing expectations. """Presubmit script for //third_party/WebKit/LayoutTests/FlagExpectations.
third_party/WebKit/LayoutTests/FlagExpectations/PRESUBMIT.py:4 Path does not exist. """Presubmit script for //third_party/WebKit/LayoutTests/FlagExpectations.

Original change's description:
> Run network service trybot when its test filters are modified.
> 
> Change-Id: I5d28f3512047871e436341f01b9bfd6e0baa2b8e
> Reviewed-on: https://chromium-review.googlesource.com/765575
> Reviewed-by: Ken Rockot <rockot@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#515890}

TBR=jam@chromium.org,rockot@chromium.org

Change-Id: I1203ce740e9260ba827fc7aeeba404d0ef76b88a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/765627Reviewed-by: Takashi Sakamoto <tasak@google.com>
Commit-Queue: Takashi Sakamoto <tasak@google.com>
Cr-Commit-Position: refs/heads/master@{#515896}

It should check BUILD.gn files instead of GYP files.

Change-Id: I83c9d454dd876ce1808dc7c3439b8c6238d4ebf9
Reviewed-on: https://chromium-review.googlesource.com/762150
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515895}

Replace Element.styleMap with Element.attributeStyleMap.
Element.sytleMap is still behind runtime feature of CSSTypedOM.
so it does not need to notify at blink-dev groups.

https://drafts.css-houdini.org/css-typed-om-1/#inline-stylepropertymap

Bug: 774933
Change-Id: I25c3dd401702505981b77fe38aa4798ea8ea9bd6
Reviewed-on: https://chromium-review.googlesource.com/765553
Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com>
Reviewed-by: Darren Shen <shend@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515894}

Using wpt-import in Chromium cec56100.
With Chromium commits locally applied on WPT:
23705631 "service worker: Upstream sandboxed iframe test to WPT."


Build: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer/builds/5168

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

No-Export: true
Change-Id: Ib1b33f618624a01d64a7899974152ec3d0461711
Reviewed-on: https://chromium-review.googlesource.com/764680
Commit-Queue: Robert Ma <robertma@chromium.org>
Reviewed-by: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515893}

This is a followup to r515890

Change-Id: Ib083a3f72e84728700fe88c439d93acdc43a9ef8
Reviewed-on: https://chromium-review.googlesource.com/765215
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515892}

This patch:
- Moves CSSTranslation tests to a subfolder containing all the other
  CSSStylevalue subclasses.
- Adds the appropriate meta and link tags.
- Clean up the test code a bit.

Notably, the spec changed so that Z can be a percentage.

Note: Following the WPT guidelines, our test structure closely follows
the spec structure. Therefore, serialization and normalization of
CSSTranslation will be done in a separate file as it is a separate
section in the spec.

Spec: https://drafts.css-houdini.org/css-typed-om-1/#csstranslation
WPT Guidelines: http://web-platform-tests.org/writing-tests/general-guidelines.html

There are failing tests where we are passing in a CSSMathValue/calc
value. We currently don't support CSSMathValues as arguments.

Bug: 774887
Change-Id: I571d6dd529a60381c310df74fddead39c61e2cd9
Reviewed-on: https://chromium-review.googlesource.com/765552
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: meade_UTC10 <meade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515891}

Change-Id: I5d28f3512047871e436341f01b9bfd6e0baa2b8e
Reviewed-on: https://chromium-review.googlesource.com/765575Reviewed-by: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515890}

We add a test for identifier normalization that covers [1]. Note that
we're only testing that an <identifier> normalizes to a CSSKeywordValue
(hence the simple test). Testing of which properties normalizes to
CSSKeywordValue will be done in a separate test, as it is in a separate
part of the spec [2].

[1] https://drafts.css-houdini.org/css-typed-om-1/#normalize-ident
[2] https://drafts.css-houdini.org/css-typed-om-1/#stylevalue-normalization

Bug: 774887
Change-Id: Idc2c0bb1279389e1e7fd46373bea13ac9b5dd105
Reviewed-on: https://chromium-review.googlesource.com/765550
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: Bugs Nash <bugsnash@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515889}

Moved the generated apply* methods from StyleBuilderFunctions.cpp
to the property classes for 'auto' properties which share a non
custom template:
- clip
- column-count
- column-gap
- column-width
- z-index

Also fixed includes so that headers required for apply* functions
are only included when those functions are implemented, not
when they are to be declared only.

Note that this patch creates duplicated logic in
make_css_property_headers.py and make_style_builder.py.
Since it is planned to remove style builder as part of this project
I have not bothered factoring this logic out.

Diff of generated files:
https://gist.github.com/1bad2de3801d6d75d77e9b08c7b128f8/revisions

Bug: 751354
Change-Id: I75d9a85a50771541f8c8be64c62c4dfa0796cd8a
Reviewed-on: https://chromium-review.googlesource.com/760079Reviewed-by: Renée Wright <rjwright@chromium.org>
Commit-Queue: Bugs Nash <bugsnash@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515888}

TBR=xingliu@chromium.org
NOTRY=true

Change-Id: I73c923f8486b611f347e914fe17545175db63970
Reviewed-on: https://chromium-review.googlesource.com/765229Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515887}

Bug: 782005
Change-Id: Icd1888daf9873719bce2b4d37d7d66e31c24da66
Reviewed-on: https://chromium-review.googlesource.com/763031Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515886}

Bug: 688116, 532855, 486308
Change-Id: I0cda49c9072099fa706988c6a1aa789456af2c1e
Reviewed-on: https://chromium-review.googlesource.com/765564Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515885}

Bug: 591099
Change-Id: I8c006a0669e056c9a779a282e30cbea6ae106eb4
Reviewed-on: https://chromium-review.googlesource.com/763581
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515884}