GitLab

https://skia.googlesource.com/skia.git/+log/58a1605d2b9b..502c3ffce8eb


git log 58a1605d2b9b..502c3ffce8eb --date=short --no-merges --format='%ad %ae %s'
2018-07-03 fmalita@chromium.org [skottie] Do not discard SkPath's FillType when interpolating
2018-07-03 weiliangc@chromium.org Use TextureProxy size directly in GrYUVToRGBEffect FragmentProcessor
2018-07-03 skcms-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll skia/third_party/skcms f447e42de725..8342d776db5a (2 commits)
2018-07-03 ruiqimao@google.com Revert "added skeletal animation support to GPU backend"
2018-07-03 robertphillips@google.com Disable NVPR path renderer in DDL mode
2018-07-03 mtklein@chromium.org remove unit tests baked into SkAAClip
2018-07-03 egdaniel@google.com Reland "Remove setting/use of GrPixelConfig in GrBackendTex/RT ctors."


Created with:
  gclient setdep -r src/third_party/skia@502c3ffce8eb

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=ethannicholas@chromium.org

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

This reverts commit a69a2eb1.

Reason for revert: MultiDeviceSetupHostVerifierOperationImplTest.ReceivedInvalidResponse_InvalidResultCode
 keeps failing on linux-chromeos-dbg bot.
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-dbg/6639

Original change's description:
> [CrOS MultiDevice] Add HostVerifierOperation.
> 
> This class represents an operation for completing the verification step
> for the current host device. A HostVerifierOperation instance is meant
> to be used for a single verification attempt; if verification needs to
> be retried, a new instance should be created for the next attempt.
> 
> This class completes the following steps to verify a device:
> (1) Call FindEligibleDevices(). This step sends a message to the host
>     device, which in turn enables background advertising.
> (2) Creates a connection to the device using the BLE listener role.
> (3) Sends an EnableBetterTogetherRequest message to the host device.
> (4) Waits for an EnableBetterTogetherResponse messages to be returned by
>     the host device.
> 
> Bug: 824568
> Change-Id: I520ff9388334467831342af4d0e270cb4e351eeb
> Reviewed-on: https://chromium-review.googlesource.com/1119466
> Reviewed-by: Ryan Hansberry <hansberry@chromium.org>
> Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#572339}

TBR=khorimoto@chromium.org,hansberry@chromium.org

Change-Id: I3f74224c953fbca3932ce178367824342ddf06e8
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 824568
Reviewed-on: https://chromium-review.googlesource.com/1125479Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572407}

As mentioned in https://crrev.com/c/1114413.

Bug: 396230
Change-Id: I2bd1d0b6123bf01f78ea0f88ebf45fc85cbe0f38
Reviewed-on: https://chromium-review.googlesource.com/1124067Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Owen Rodley <orodley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572406}

This reverts commit 18ba607a.

Reason for revert: Speculative revert for consistent failure on linux-chromeos-rel CQ builder.

Original change's description:
> CC: Make LayerTreeHostImpl::OnMemoryPressure work for the desktop as well
> 
> LayerTreeHostImpl::OnMemoryPressure has only worked for low-end devices.
> But it would be good if it works for all devices.
> 
> Bug: 839687
> Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
> Change-Id: I9f670ab08587ad4c5f626eb74fe5d7c9db256150
> Reviewed-on: https://chromium-review.googlesource.com/1107025
> Commit-Queue: Gyuyoung Kim <gyuyoung.kim@lge.com>
> Reviewed-by: Eric Karl <ericrk@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#572216}

TBR=ericrk@chromium.org,gyuyoung.kim@lge.com

Change-Id: I5e8d81f540311373a3d79854310ec59896494cae
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 839687, 860090
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Reviewed-on: https://chromium-review.googlesource.com/1125459Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572405}

Bug: 851163
Change-Id: I2b89682bcec7c24319160fab2cb69d08603d2f60
Reviewed-on: https://chromium-review.googlesource.com/1125322Reviewed-by: Mike Wittman <wittman@chromium.org>
Commit-Queue: Xi Cheng <chengx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572404}

Change-Id: Ida242dcd6a780c207c43262ac05e3955fccf1ce9
Reviewed-on: https://chromium-review.googlesource.com/1125325
Commit-Queue: Jared Saul <jsaul@google.com>
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572403}

On Windows, hovering over the media controls triggers an ET_MOUSE_EXIT
MouseEvent. This is intended to only be caught when the mouse is
not within the video window bounds. On non-Windows, this is a noop.

Bug: 852131
Change-Id: I988d4483510c4eaafc9969edb26de407d77b0149
Reviewed-on: https://chromium-review.googlesource.com/1125139Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572402}

The wording on the message descriptions was incorrect,
suggesting the text is "readable" when it should have
indicated that the text will be read by a screenreader.

Change-Id: Ide5ae94e938d1690f4adec639484815aaeeddf5e
Reviewed-on: https://chromium-review.googlesource.com/1124800
Commit-Queue: Orin Jaworski <orinj@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572401}

cookiechange events have been dispatched to ServiceWorkerGlobalScope
since https://crrev.com/c/979334 landed, and can be observed using
addEventListener, as shown by the WPT test at
cookie-store/serviceworker_cookieStore_subscriptions_basic.tentative.https.html

This CL adds the oncookiechange attribute to ServiceWorkerGlobalScope,
allowing the event to be handled by a DOM0-style event handler.

Bug: 729800
Change-Id: Ic27979d94934abf833c116787cdad5243422324b
Reviewed-on: https://chromium-review.googlesource.com/1123766
Commit-Queue: Victor Costan <pwnall@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Joshua Bell <jsbell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572400}

When the network service is enabled, we wouldn't download responses
that had a plugin associated with their mime type. This makes the
PluginResponseInterceptorUrlLoaderThrottle check if the response
is meant to be a download before intercepting it.

Bug: 855539
Change-Id: I8b70abb2a7e24c27b5a6866a0f09c24f59c12e05
Reviewed-on: https://chromium-review.googlesource.com/1119581
Commit-Queue: Robbie McElrath <rmcelrath@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572399}

This patch takes scrolled offset into account when hit testing the
children of a scroll container, so that scrolled contents can be hit
tested correctly.

Bug: 855279
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ic84b40c06302c457cf32cacaa5e79f419dcc3a2b
Reviewed-on: https://chromium-review.googlesource.com/1125312Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572398}

* Add flag for the shelf new UI
* When it's on, center some shelf items (apps, pinned apps)
* Minor dimension and opacity tweaks when the flag is on
* Fix some minor typos here and there

Change-Id: I87b7fa819bac0fff77f5007c4a0cee760a630239
Bug: 805612
Reviewed-on: https://chromium-review.googlesource.com/1116540Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: Alex Newcomer <newcomer@chromium.org>
Commit-Queue: Manu Cornet <manucornet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572397}

When a real IME window is up, AppKit spins a nested runloop that
interacts with the IME window over XPC while we're trying to
invalidate our NSTextInputClient (e.g. because the window has been
deleted). However, this may also cause AppKit to *use* the very
NSTextInputClient that we're trying to invalidate.

Specifically, this seems to happen with phonetic languages such as
Korean and Vietnamese. These use rule-based transforms that do not
commonly pop-up a candidate window. In such cases, a key such as
Enter may simultaneously commit a composition _and_ trigger the omnibox
action, which moves focus away from the omnibox thereby invalidating
the textInputClient_.

-BridgedContentView insertText: was the last hold-out that didn't
have a null check. AttributedSubstringForRangeHelper() and
GetFirstRectForRangeHelper() handle a null client already.

  SystemPreferences -> Keyboard -> Input Sources, then interact with
  the omnibox.

Test: On Mac, add/enable the "2-Set Korean" input method under
Bug: 817097
Change-Id: Ia7d1ebb796b9c98437b563ea405cad785e92ea30
Reviewed-on: https://chromium-review.googlesource.com/1124073Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572396}

If the targetId is not specified, the info for the target of the current
session is returned.

DevTools receives a connection to a target, but does not know the
targetId. Certain commands, particularly Target.attachToTarget, take
a targetId. Using Target.attachToTarget will allow DevTools to create
parallel connections to the page for a better Lighthouse experience.

Change-Id: I777350d6fb486c4e5ed6256ebc0bcc1d738d0ab2
Reviewed-on: https://chromium-review.googlesource.com/1096434
Commit-Queue: Joel Einbinder <einbinder@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572395}

BUG=762641
TBR=derat@google.com

Change-Id: I2fdb769ada863ff26315c8e6c9b0c750e9cc1bda
Reviewed-on: https://chromium-review.googlesource.com/1125170
Commit-Queue: ChromeOS Commit Bot <chromeos-commit-bot@chromium.org>
Reviewed-by: ChromeOS Commit Bot <chromeos-commit-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572394}

https://pdfium.googlesource.com/pdfium.git/+log/95b0293a29b2..94f7b75b4d5d


git log 95b0293a29b2..94f7b75b4d5d --date=short --no-merges --format='%ad %ae %s'
2018-07-03 tsepez@chromium.org Use unowned ptr to Node from outside XFA node tree.
2018-07-03 tsepez@chromium.org Use UnownedPtr<> in cxfa_nodeiteratortemplate.h
2018-07-03 tsepez@chromium.org Use std::vector<float> in CPDF_Function.
2018-07-03 thestig@chromium.org Remove a parameter from CPDF_SyntaxParser::FindTag().
2018-07-03 art-snake@yandex-team.ru Avoid duplicate data buffering in CPDF_SyntaxParser::ReadStream().


Created with:
  gclient setdep -r src/third_party/pdfium@94f7b75b4d5d

The AutoRoll server is located here: https://pdfium-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=dsinclair@chromium.org

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

- Handle subsamples.
- Use ID3D11VideoContext1 for SubmitDecoderBuffers1.
- Pass a decryption context to DecoderBeginFrame().
- Change GetAddressOf in D3D11VideoDecoderImpl::Initialize to
  ReleaseAndGetAddressOf so that it doesn't leak when initialized
  multiple times.

Bug: 799310
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: I802f3afff630811f3f2a497557ad7af566ff3f4a
Reviewed-on: https://chromium-review.googlesource.com/1123694
Commit-Queue: Rintaro Kuroiwa <rkuroiwa@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572392}

This avoids clearing some longer-lived Skia caches, hopefully avoiding
an UMA latency regression we experienced when the idle clearing was
initially added.

This re-land uses a more direct Skia API, hopefully avoiding the
flakiness seen in issue 805575.

Note for sheriffs: We expect a small regression in measured memory.
This is intentional, and comes with an improvement to input latency.
This regression shouldn't have a large impact on real-world scenarios,
as we are preventing the clearing of a cache that will be immediately
re-created on any page raster.

TBR=brettw@chromium.org for test-only DEPS change (already ran by bsalomon@ in previous patch, this is re-land+updates)

Bug: 664181, 805575
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
Change-Id: Id4c9a091a543c31f928e0214558695de81a30e9d
Reviewed-on: https://chromium-review.googlesource.com/1123169
Commit-Queue: Eric Karl <ericrk@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572391}

https://crrev.com/c/1120677 "Don't create a Surface for WebXR SharedBuffer
mode" had a logic error that led to the GPU process connection being recreated
unconditionally on re-entering VR, resulting in an inconsistent state where it
tried to reuse mailbox names belonging to the previous connection.

Change-Id: Ic005c7884a46b3d1b3da986a55e9d733f09d68d3
Reviewed-on: https://chromium-review.googlesource.com/1125273Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Klaus Weidner <klausw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572390}

Bug: 845037
Change-Id: I60574af1513793a0872c4d02c20c829f0adda9df
Reviewed-on: https://chromium-review.googlesource.com/1117769
Commit-Queue: Quan Nguyen <qnnguyen@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572389}

Adds integration tests to check the navigation bar color on OMR1.

BUG=818210

Change-Id: I3f5f56eccb8795022cf3a3765efa8e3c55bdcba5
Reviewed-on: https://chromium-review.googlesource.com/1125274
Commit-Queue: Theresa <twellington@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572388}

The (?) tooltip was removed in
https://chromium-review.googlesource.com/c/chromium/src/+/1107778 but
the comment was not updated.

Change-Id: I2403df4e5672c26b306da628cc46c2e5faa8d63a
Reviewed-on: https://chromium-review.googlesource.com/1125207Reviewed-by: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Jared Saul <jsaul@google.com>
Cr-Commit-Position: refs/heads/master@{#572387}

Add title and description that appears on UI dialog for BLE registration
and sign-in flow.

Bug: 849323
Change-Id: I373af67363bddcdfc613282bef095dee58f8521a
Reviewed-on: https://chromium-review.googlesource.com/1105598
Commit-Queue: Jun Choi <hongjunchoi@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Reviewed-by: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572386}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/670e331c4d87..a1689df8745a


git log 670e331c4d87..a1689df8745a --date=short --no-merges --format='%ad %ae %s'
2018-07-03 dgarrett@google.com cros_sdk_lib: Increase cleanup timeout.


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

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.



BUG=chromium:859984
TBR=chrome-os-gardeners@chromium.org

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

Change-Id: I1f3d9d5ad10e175809b6e304019d6179ca0b0d43
Reviewed-on: https://chromium-review.googlesource.com/1125302Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572384}

We had enough time with CHECK to ensure this is not needed.

A couple of tests which did browser-initiated navigation do
not dump the policy anymore.

reload-policy-delegate was doing browser-initiated reload,
and does not test anything meaningful for a while. I changed it
to check performance.navigation.type which is pretty similar
is also available for browser-initiated navigation.

Bug: 789577
Change-Id: If89041e4ae9b509e82d724f171a1faa731099965
Reviewed-on: https://chromium-review.googlesource.com/1119380Reviewed-by: Camille Lamy <clamy@chromium.org>
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572383}

This allows testing of both navigation manager implementations on bots.
Temporarily disabled currently failing tests.

Bug: 851119
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I3ed3d60479aa180101571ce56e78b66c95d5d4f4
Reviewed-on: https://chromium-review.googlesource.com/1110999
Commit-Queue: Danyao Wang <danyao@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572382}

This makes the code reusable for other CodeMirrorTextEditors that
might want the same autocomplete experience we have in the Console.

Change-Id: I8660249c08d920d86d2b82524b7bd50d9b5deb05
Reviewed-on: https://chromium-review.googlesource.com/1092080
Commit-Queue: Joel Einbinder <einbinder@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Erik Luo <luoe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572381}

https://chromium.googlesource.com/catapult.git/+log/10e0a3798e47..3eb704babdf7


git log 10e0a3798e47..3eb704babdf7 --date=short --no-merges --format='%ad %ae %s'
2018-07-03 charliea@chromium.org Remove BattOr support from systrace


Created with:
  gclient setdep -r src/third_party/catapult@3eb704babdf7

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=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

BUG=chromium:859514
TBR=sullivan@chromium.org

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

Bug: 846424
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I214fa0e5a1e6604e27d6066278c7d647a583a4e8
Reviewed-on: https://chromium-review.googlesource.com/1108445
Commit-Queue: Sahel Sharify <sahel@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572379}

Recent SDK builds have a broken 'pm archive' command, so builds have not
actually been rebuilding the package archives, and things have only been
"working" because our GN rules are broken in such a way that ninja
doesn't notice the brokenness.

TBR: thakis
Bug: 859955
Change-Id: Ic71ccee289d4cee532f539e19f4d9baa63408e0a
Reviewed-on: https://chromium-review.googlesource.com/1125214Reviewed-by: Wez <wez@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572378}

Use a common GrContext across all RasterDecoders. This ensures that we
get a unified GrProgramCache for compiled shaders.

R=backer@chromium.org, ericrk@chromium.org

Bug: 854416
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: I653486d2a20f4bb0112c6cfe66f3922c222ad338
Reviewed-on: https://chromium-review.googlesource.com/1114287
Commit-Queue: Khushal <khushalsagar@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Eric Karl <ericrk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572377}

This is mainly a performance optimization because we don't need
FinishCycle() if PaintController has nothing changed.

By the way it can also avoid the immediate bad effect (but not the
root cause) of bug 859294.

Bug: 859294
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I3a4d26253fef2b04adc805ef94b93c5479c88759
Reviewed-on: https://chromium-review.googlesource.com/1124980
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572376}

The isNodeJS() method validates whether target
has only SDK.Target.Capability.JS set. This
patch makes it more explicit and resolves the
existing TODO in the isNodeJS() method.

The creator of the target must now explicitly
mark the target as Node.js by calling
markAsNodeJS() on the target object, thus
avoiding the dependency on SDK.Target.Capability.JS.

Change-Id: I4da9af58ddc42eba2b6b91f7b1b81b411180256b
Reviewed-on: https://chromium-review.googlesource.com/1116761
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572375}

This CL adds the option to query histogram deltas since the previous call,
rather than cumulative counts since launch.

Bug: 855745
Change-Id: I4c2965613d1e1ca39975bbeda3af2c5004c1f990
Reviewed-on: https://chromium-review.googlesource.com/1118846
Commit-Queue: Bryce Thomas <bryct@amazon.com>
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572374}

We currently have metrics for task durations by queue type in background
after five and ten minutes, and this adds these metrics by task type. These
metrics will be useful for determining which exact tasks are running in the
background after tabs are frozen.

Bug: 858841
Change-Id: I518167c3ff7029f383492d088cfbcb1d90e0a557
Reviewed-on: https://chromium-review.googlesource.com/1119375Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Commit-Queue: Scott Haseley <shaseley@google.com>
Cr-Commit-Position: refs/heads/master@{#572373}

Change-Id: I21f5d472325103e2970a54197736259ea8177dd1
Reviewed-on: https://chromium-review.googlesource.com/1125287Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Jeffrey Yasskin <jyasskin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572372}

Bug: 836908
Change-Id: I184d54721a995fc059038c4c046c3aecffdce0c5
Reviewed-on: https://chromium-review.googlesource.com/1121109Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Reviewed-by: Jared Saul <jsaul@google.com>
Commit-Queue: Siyu An <siyua@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572371}

Though the accelerator is now handled by ConfirmQuitBubbleController, we still
need the accelerator entry for showing the accelerator subtext in the menu on
the Exit entry.

BUG=859830
R=pkasting

Change-Id: Ia5f40482b009619d0cb44d5b05b9592b7fb13700
Reviewed-on: https://chromium-review.googlesource.com/1125082Reviewed-by: Peter Kasting <pkasting@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572370}

Pass a Service pointer with Getters instead of pointers for each interface

Bug: None
Test: Manual Test
Change-Id: I4b32aabbabda94b74f699cb7093708ead1bc6fa6
Reviewed-on: https://chromium-review.googlesource.com/1123371
Commit-Queue: Yue Li <updowndota@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572369}