GitLab

It is flaky

TBR=rouslan@chromium.org
NOTRY=true

Bug: 769851
Change-Id: I4ba160fe8d0916729be86807253c408a4d853404
Reviewed-on: https://chromium-review.googlesource.com/690799Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505105}

There is bug in AQ in v1.6.0 which causes very poor video quality in
WebRTC when packetization mode 0 is used. The issue is fixed in v1.7
(https://github.com/cisco/openh264/releases/tag/v1.7.0).

I modified MediaRecorderHandlerTest/EncodeVideoFrames test to make it
pass with new encoder. The test expects encoded frame sizes to be
equal or bigger than the hard coded thresholds. But due to changes in
RC the new encoder uses different QPs. As result there is difference
in frame sizes: the second frame produced by the new encoder is 1
byte smaller than the previous version produced. The test failed
because of that. I changed the threshold from 13 to 12 bytes to make
it work with the new version (it still works with previous version as
well).

Bug: webrtc:8070
Change-Id: I58f82be70b12f26bea306f7788d33db9aca47856
Reviewed-on: https://chromium-review.googlesource.com/674868Reviewed-by: Emircan Uysaler <emircan@chromium.org>
Commit-Queue: Sergey Silkin <ssilkin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505104}

This CL introduces the name source and methods to record it.
Most of the changes are then code to forward the new parameter of the 
recording method.

The updated signature needs rewiring on:
- Android NTP
    chrome/android/
    chrome/browser/android/ntp/
- iOS
    ios/chrome/browser/
- Chrome Desktop
    chrome/browser/resources/
    chrome/browser/search/
    chrome/browser/ui/search
    chrome/common/
    chrome/renderer/resources/extensions/
    chrome/renderer/searchbox/


Bug: 767402
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Id10b344f2280f9859db4c76cafe276e130f5dc53
Reviewed-on: https://chromium-review.googlesource.com/673187Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Friedrich Horschig <fhorschig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505103}

The Fuchsia code is not using the legacy X.509 stack.

Bug: 706445
Change-Id: I6c2cc42bc302c01d38067657317aa1b958442270
Reviewed-on: https://chromium-review.googlesource.com/690376
Commit-Queue: David Benjamin <davidben@chromium.org>
Commit-Queue: Matt Mueller <mattm@chromium.org>
Reviewed-by: Matt Mueller <mattm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505102}

https://skia.googlesource.com/skia.git/+log/3a5e1fee080c..097900648b0b

$ git log 3a5e1fee0..097900648 --date=short --no-merges --format='%ad %ae %s'
2017-09-28 borenet Fix path separators on Windows in zip_utils
2017-09-28 mtklein function santizer is not available on Mac
2017-09-28 brianosman Use wider types to compute bounds area and avoid overflow

Created with:
  roll-dep src/third_party/skia


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


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;master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=jvanverth@chromium.org

Change-Id: I8bb576e4296e83dbd8998e43bc9ce04c7ffc39e9
Reviewed-on: https://chromium-review.googlesource.com/689860Reviewed-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@{#505101}

These tests got disabled on M and above due to flakes. However
now webview tests no longer runs on K cq bot, meaning these tests
have no coverage on cq.

Tried running looping the 22 tests that are enabled here 50 times
locally without any failure. So enable them with RetryOnFailure
and see if there are any flakes on bots.

Bug: 615483
Change-Id: I3bc20a1e1b89f3b925db3943441a5bd00efb9c9b
Reviewed-on: https://chromium-review.googlesource.com/688798Reviewed-by: Paul Miller <paulmiller@chromium.org>
Commit-Queue: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505100}

Passing 'this' (which |object| will be in this case) to
LayoutObject::NextInPreOrder, will turn it into a non-obvious way of
saying FirstChild.
Pass |root| instead of |object| to make the traversal actually pre-
order.

Bug: 769459
Change-Id: I6bc4c487faab053d02bc16826862234f15e2376e
Reviewed-on: https://chromium-review.googlesource.com/690254Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#505099}

Turning on SwiftShader for Layout Tests on Linux.
Added expectations for tests which will require rebaselining.

Attempting to reland:
https://chromium-review.googlesource.com/c/chromium/src/+/651079

TBR=kbr@chromium.org,hubbe@chromium.org
TBR_REASON=A few minor changes to content_browsertests which can be followed up on later

Bug: 726075
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: I2c51376975217a7534b728f23c7881b60dd2f025
Reviewed-on: https://chromium-review.googlesource.com/687276
Commit-Queue: Alexis Hétu <sugoi@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Alexis Hétu <sugoi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505098}

https://pdfium.googlesource.com/pdfium.git/+log/5f95f36cac52..79eab23d8f83

$ git log 5f95f36ca..79eab23d8 --date=short --no-merges --format='%ad %ae %s'
2017-09-28 npm Add Length1 in TrueType font streams
2017-09-28 art-snake Implement FakeFileAccess.
2017-09-28 dsinclair Cleanup CFGAS_GEFont methods
2017-09-28 dsinclair Simplify some OS checks to PLATFORM checks
2017-09-28 art-snake Fix load non first pages in linearized document with hints table.
2017-09-28 art-snake Fix infinite loop on form availability check.
2017-09-28 rharrison Remove no-op cast

Created with:
  roll-dep src/third_party/pdfium


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


TBR=dsinclair@chromium.org

Change-Id: I5df7d33b7164e1b10bda6be4d6406aead78177c3
Reviewed-on: https://chromium-review.googlesource.com/690325
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505097}

This change adds a promo dialog and flag to enable it on every
restart. Choosing a setting different from the one the browser is
currently in will cause the browser's activity to restart when the
dialog is closed. The enabled state of the browser when the promo is
in use is driven off of a new shared preference
"chrome_home_user_enabled". The new preference takes priority over
about:flags.

BUG=767738

Change-Id: I4d0b5da6cf47baf338dbbc2fd8ffb85ea9140211
Reviewed-on: https://chromium-review.googlesource.com/683248
Commit-Queue: Matthew Jones <mdjones@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505096}

This is a reland of a240cac3

Fix/Workaround for crbug.com/769759:
- Disables optimization on a function that causes clang to 
  crash for some targets.

Original change's description:
> New NTLMv2 Implementation.
>
> - Supports NTLMv2 with Extended Protection for Authentication (EPA)
>   and Message Integrity Check (MIC).
> - Adds required methods to buffer reader/writers to support NTLMv2
> - Adds optional version/mic fields to all messages
> - Removes protocol level tests from HttpAuthHandlerNtlmPortableTest.
>   A previous CL already implemented all those tests directly against
>   NtlmClient. The tests at that level only existed to validate that
>   the behavior of the old and new implementation were the same.
> - Future CL updates fuzzer to support v2.
> - This CL does not activate NTLMv2. The code still defaults to NTLMv1.
>
> BUG=chromium:22532
>
> Change-Id: I5b6dcbf6cdcf8f671008f60ae49c9bacb4e1d2f3
> Reviewed-on: https://chromium-review.googlesource.com/608620
> Commit-Queue: Zentaro Kavanagh <zentaro@google.com>
> Reviewed-by: Asanka Herath <asanka@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#504808}

Bug: chromium:22532
Change-Id: I3c3ec52b2592c4bf8da4bd79fbfe910567b41260
Reviewed-on: https://chromium-review.googlesource.com/689956Reviewed-by: Asanka Herath <asanka@chromium.org>
Commit-Queue: Zentaro Kavanagh <zentaro@google.com>
Cr-Commit-Position: refs/heads/master@{#505095}

This change:
1) Adds a new class, ContentSettingDomainListView, which draws a titled list
   of domains;
2) Makes ContentSettingBubbleContents use that new class;
3) Adds a new class, BulletView, which draws a marker bullet for an unordered
   list;
4) Makes SadTabView and ExtensionInstallDialogView use BulletView.

Bug: 700196
Change-Id: I2e1999b452277023ca071c0f8a7457fa405e3f00
Reviewed-on: https://chromium-review.googlesource.com/681852
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505094}

ContextMenuJsTest.UnsupportedReferrerPolicy is flaky on iPhone 7
iOS 11 devices, and this is only a speculative fix because I couldn't
reproduce the flake locally.

Bug: 
Change-Id: I99c7228cdcce1129526b33bb892e0ab8aceb3a14
Reviewed-on: https://chromium-review.googlesource.com/690600Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505093}

Bug: 766806
Change-Id: I1a210ded77cc9ffdc0039fca097c6fdba7ff0131
Reviewed-on: https://chromium-review.googlesource.com/677403
Commit-Queue: proberge <proberge@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505092}

WebContents::GetRenderProcessHost is an API that doesn't behave as
expected with out-of-process iframes, as multiple processes can be
associated with a single WebContents.  This CL replaces callers of this
API, so that they go through an equivalent API that explicitly selects
which frame's process is needed.

This CL was started by using an ad-hoc clang-plugin that replaced
existing callsites of WebContents::GetRenderProcessHost() with a call to
something like wc->GetMainFrame()->GetProcess().  This was followed-up
by manually adding an include of render_frame_host.h and git cl format
and git cl lint and then a self-review with small tweaks (e.g. sometimes
using a wc->GetRenderViewHost()->GetProcess() is more appropriate).

This CL was uploaded by git cl split.

R=achuith@chromium.org

Bug: 666525
Change-Id: If9e66ef88a2d29360ddf0455de2f1489d324c895
Reviewed-on: https://chromium-review.googlesource.com/689106Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505091}

Originally, the coverage.profdata file is always generated as
out/Coverage-iphonesimulator/coverage.profdata, and one problem with
this setup is that the coverage.profdata can be easily overwritten,
which makes it hard to reuse.

This CL fixes the issue by generating the file in a user specified
output directory.

Bug: 757515
Change-Id: I555d8bac887c01eab99ff31b2860d52aeada9f26
Reviewed-on: https://chromium-review.googlesource.com/688299
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505090}

Add a test for https://crbug.com/766149. The issue happens when
webview's embedders are trying navigate to javascript-urls while the
main document is loading. It causes the main document to stop loading.

Navigation to a javascript-url is not a "real" navigation and should not
prevent the main document from loading.

See the others CLs in this series:
  [1/2] https://chromium-review.googlesource.com/c/684736.
  [2/2] This patch.

BUG=766149

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Change-Id: If2e4d2661ad3ab975a53d9e5646a1d3e8956772d
Reviewed-on: https://chromium-review.googlesource.com/684834
Commit-Queue: Nasko Oskov <nasko@chromium.org>
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505089}

WebContents::GetRenderProcessHost is an API that doesn't behave as
expected with out-of-process iframes, as multiple processes can be
associated with a single WebContents.  This CL replaces callers of this
API, so that they go through an equivalent API that explicitly selects
which frame's process is needed.

This CL was started by using an ad-hoc clang-plugin that replaced
existing callsites of WebContents::GetRenderProcessHost() with a call to
something like wc->GetMainFrame()->GetProcess().  This was followed-up
by manually adding an include of render_frame_host.h and git cl format
and git cl lint and then a self-review with small tweaks (e.g. sometimes
using a wc->GetRenderViewHost()->GetProcess() is more appropriate).

This CL was uploaded by git cl split.

R=dpapad@chromium.org

Bug: 666525
Change-Id: I7f56ede89ac9d6c72debb7fa018317539bcf8fc9
Reviewed-on: https://chromium-review.googlesource.com/689101Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505088}

WebContents::GetRenderProcessHost is an API that doesn't behave as
expected with out-of-process iframes, as multiple processes can be
associated with a single WebContents.  This CL replaces callers of this
API, so that they go through an equivalent API that explicitly selects
which frame's process is needed.

This CL was started by using an ad-hoc clang-plugin that replaced
existing callsites of WebContents::GetRenderProcessHost() with a call to
something like wc->GetMainFrame()->GetProcess().  This was followed-up
by manually adding an include of render_frame_host.h and git cl format
and git cl lint and then a self-review with small tweaks (e.g. sometimes
using a wc->GetRenderViewHost()->GetProcess() is more appropriate).

This CL was uploaded by git cl split.

R=lfg@chromium.org

Bug: 666525
Change-Id: Ie278447778d4b066b1588ecd1c0748221aa14522
Reviewed-on: https://chromium-review.googlesource.com/689025
Commit-Queue: Lucas Gadani <lfg@chromium.org>
Reviewed-by: Lucas Gadani <lfg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505087}

-Introduce a new browserservices package for classes that will be shared
across customtabs, browseractions and trustedwebactivities.
- Move CustomTabsContentHandler to the new package with a rename
- Extract the related static calls in CustomTabActivity to a utility
class inside the new package.
- Also move CustomTabsIntentDataProvider,
PostMessageHandler, OriginVerifier to the same package.

BUG=769129

Change-Id: I66829c5bb01338cb16089ab3baefe7ec1c79ade6
Reviewed-on: https://chromium-review.googlesource.com/685943
Commit-Queue: Yusuf Ozuysal <yusufo@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505086}

Fix isolation of traffic_annotation_auditor_unittests again.

R=msramek

Bug: 
Change-Id: Ic756bc7371efe4c354d21a8a20ed43b7567539bf
Reviewed-on: https://chromium-review.googlesource.com/690395Reviewed-by: Martin Šrámek <msramek@chromium.org>
Commit-Queue: Aleksandr Derbenev <alex-ac@yandex-team.ru>
Cr-Commit-Position: refs/heads/master@{#505085}

Bug: 767853
Change-Id: I3af50b2eed7e842849ec1b0818be4e21b53e59f1
Reviewed-on: https://chromium-review.googlesource.com/690699
Commit-Queue: Moe Ahmadi <mahmadi@chromium.org>
Commit-Queue: Marc-Antoine Courteau <macourteau@chromium.org>
Reviewed-by: Marc-Antoine Courteau <macourteau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505084}

WebContents::GetRenderProcessHost is an API that doesn't behave as
expected with out-of-process iframes, as multiple processes can be
associated with a single WebContents.  This CL replaces callers of this
API, so that they go through an equivalent API that explicitly selects
which frame's process is needed.

This CL was started by using an ad-hoc clang-plugin that replaced
existing callsites of WebContents::GetRenderProcessHost() with a call to
something like wc->GetMainFrame()->GetProcess().  This was followed-up
by manually adding an include of render_frame_host.h and git cl format
and git cl lint and then a self-review with small tweaks (e.g. sometimes
using a wc->GetRenderViewHost()->GetProcess() is more appropriate).

This CL was uploaded by git cl split.

R=estark@chromium.org

Bug: 666525
Change-Id: Ia5cfdc5682fdfa8f419de5341dc2868292adb74a
Reviewed-on: https://chromium-review.googlesource.com/689110Reviewed-by: Emily Stark <estark@chromium.org>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505083}

Chrome was setting RLIMIT_DATA under the assumption that RLIMIT_DATA
only covered the brk() heap. Starting with
https://github.com/torvalds/linux/commit/84638335900f1995495838fe1bd4870c43ec1f67,
RLIMIT_DATA also accounts for mmap()-ed memory. This means that on systems with
newer kernels the current limit is too low, and Chrome OOMs on large webpages.

Increase the limit to 8GB on 64-bit systems. This will make spraying a little
easier, but we need a usable browser.

BUG=752185
TEST=cat /proc/<pid>/limits:
TEST=Max data size        8589934592         8589934592        bytes
R=thomasanderson@chromium.org

Change-Id: I8b34aad1823fa7f94fcb40703d72c96dc900e769
Reviewed-on: https://chromium-review.googlesource.com/641992Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Chris Palmer <palmer@chromium.org>
Commit-Queue: Jorge Lucangeli Obes <jorgelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505082}

We should avoid creating render surface if we are animating opacity with
the same value of 1.0f. The opacity value not equal to 1.0f has already
been handled in the code. Therefore, in this cl, we modify the IsThreaded
to not create cc::Animation if it is animating with same start and target
values.

Bug: 764575
Test: local with flag ui-show-composited-layer-borders=renderpass
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I3b6044382d395c831e6e1abf627ead570d9715c1
Reviewed-on: https://chromium-review.googlesource.com/663535Reviewed-by: Ali Juma <ajuma@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Commit-Queue: Tao Wu <wutao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505081}

Bug: 664578
Change-Id: I47437bd37bd3581d070c72c8fe977cc457d81214
Reviewed-on: https://chromium-review.googlesource.com/686875Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Commit-Queue: Nicolas Dossou-Gbété <dgn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505080}

This is to permit less-confusing introduction of kSimpleEntryTotalFileCount
which also counts the _s file.

Bug: 636400
Change-Id: Id4bd67d2f2db4ab2c20a064ec49bdaea8c105e97
Reviewed-on: https://chromium-review.googlesource.com/690482Reviewed-by: Josh Karlin <jkarlin@chromium.org>
Commit-Queue: Maks Orlovich <morlovich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505079}

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/69862

TBR=schenney@chromium.org
NOTRY=true

Bug: 755405
Change-Id: I86fa77bad65ad9290b598fd6d5e74bb806b030c7
Reviewed-on: https://chromium-review.googlesource.com/690516Reviewed-by: Xida Chen <xidachen@chromium.org>
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505078}

WebContents::GetRenderProcessHost is an API that doesn't behave as
expected with out-of-process iframes, as multiple processes can be
associated with a single WebContents.  This CL replaces callers of this
API, so that they go through an equivalent API that explicitly selects
which frame's process is needed.

This CL was started by using an ad-hoc clang-plugin that replaced
existing callsites of WebContents::GetRenderProcessHost() with a call to
something like wc->GetMainFrame()->GetProcess().  This was followed-up
by manually adding an include of render_frame_host.h and git cl format
and git cl lint and then a self-review with small tweaks (e.g. sometimes
using a wc->GetRenderViewHost()->GetProcess() is more appropriate).

This CL was uploaded by git cl split.

R=tbarzic@chromium.org

Bug: 666525
Change-Id: I9411427ca99d25dd1081871d3db6400e8d16a418
Reviewed-on: https://chromium-review.googlesource.com/689131Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505077}

R=cros-printing-dev@chromium.org, skau@chromium.org

The file printer_backend_proxy_chromeos.cc was moved to a different file in a
recent CL so I have updated the docs.

Bug: 
Change-Id: Ic5c834470e70f3be0cf4e2424957cdcca9a2e03d
Reviewed-on: https://chromium-review.googlesource.com/688504Reviewed-by: Sean Kau <skau@chromium.org>
Commit-Queue: David Valleau <valleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505076}

TBR=schenney@chromium.org
NOTRY=true

Bug: 769729
Change-Id: I51576dffb92b28cfff21143517433c3b2e9ac00e
Reviewed-on: https://chromium-review.googlesource.com/690745Reviewed-by: Xida Chen <xidachen@chromium.org>
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505075}

It's not currently launched or planned to launch on Linux, and the
config on Android is changing.

Bug: 768874
Change-Id: I2664ab4ff54a155a2cd18a2047412d762404debb
Reviewed-on: https://chromium-review.googlesource.com/685578
Commit-Queue: Miriam Gershenson <mgersh@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505074}

1. Disable composited layers for frame scrollbars in PaintLayerCompositor
   for SPv2.
2. Issue DrawingRecorder for scrollbar backgrounds.

This is mainly to reduce spv2 test failures. We still need a solution of
scrollbar painting during composited scrolling for spv2.

Bug: 757938
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: If5885cfb9a7e3b5dacbac0d26b0b59dd89f42a79
Reviewed-on: https://chromium-review.googlesource.com/685267
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505073}

Extend the DCHECK to also cover SVG layout objects. The IsBox() check
should cover the SVG root, and if it is zero-sized any descendant should
not be considered to have a "non-empty" layout size. Overflow is
disregarded (as it seems to be in general in this method.)

Bug: 769459
Change-Id: Iea15b86032094a15608b614f5baa1057c5d6a686
Reviewed-on: https://chromium-review.googlesource.com/689997Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#505072}

It has been very flaky. Please re-enable it with fix

TBR=nednguyen@chromium.org
NOTRY=true

Bug: 769809
Change-Id: I7077fbfa752ca5c046ae63475d974896f42e1225
Reviewed-on: https://chromium-review.googlesource.com/690515Reviewed-by: Xida Chen <xidachen@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505071}

If the state field is unpopulated, inserts a blank element at the
beginning of the list of states, as the first element is selected by
default when the UIPickerView is shown.

R=mahmadi

Bug: 767972
Change-Id: Iaa0c4bc29b863d8c521f3e9f0b826eb7ccef408d
Reviewed-on: https://chromium-review.googlesource.com/689857Reviewed-by: Moe Ahmadi <mahmadi@chromium.org>
Commit-Queue: Marc-Antoine Courteau <macourteau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505070}

This patch fixes inline blocks with orthogonal writing mode:
1. Not to crash in ResolveInlineLength.
2. Uses the correct side of margins for the inline block.

Bug: 636993
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I83abddd8e910985e6f9d9fba9f93954ef038e41d
Reviewed-on: https://chromium-review.googlesource.com/688523Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505069}

Change the accessibility rating text to the way Play Store is using.

Bug: 756991
Change-Id: I812faf2e74233ba7f7d1407bea19e00bfbf538b9
Reviewed-on: https://chromium-review.googlesource.com/667785
Commit-Queue: Jiaquan He <hejq@google.com>
Reviewed-by: Vadim Tryshev <vadimt@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505068}

Test: ash_unittests WindowSelectorTest.ExitOverviewWithKey
Bug: 764995
Change-Id: Ice98b3451a11a526972c1c3cfc0a6a0a4d7399da
Reviewed-on: https://chromium-review.googlesource.com/688021Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Valery Arkhangorodsky <varkha@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505067}

On Mac, currently when an entry is selected in the device chooser, the
paired text is shown as light green. This CL changes its color to be
white to increase the contrast.

I uploaded a screenshot on the issue page.

Bug: 711563
Change-Id: I4842bfe5e1ea04fcc0a868b8d6f16b8552bd2d20
Reviewed-on: https://chromium-review.googlesource.com/688753Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Jun Cai <juncai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#505066}