GitLab

Due to https://chromium-review.googlesource.com/c/chromium/src/+/1296491

See log: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20CFI/10902

TBR=mstensho@chromium.org

Bug: 898540
Change-Id: I2e3e3ead85602b111ca423cb89b1a0ce2813f170
Reviewed-on: https://chromium-review.googlesource.com/c/1298015Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602443}

This is not used at all.

Bug: 898380
Change-Id: I67a86391bb99204932f2bf3f7b680f0908d22801
Reviewed-on: https://chromium-review.googlesource.com/c/1298237Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602442}

This reverts commit 4aa59465.

Reason for revert: High telemetry_perf_unittests failure rate on android-marshmallow-arm64-rel CQ bot

See crbug.com/898646.

Original change's description:
> [perf] Add browse:social:tumblr_infinite_scroll system health story
> 
> Bug: 878390
> Change-Id: Ibdaeafbc6d32970641a5a8562125eb15b0bfedf7
> Reviewed-on: https://chromium-review.googlesource.com/c/1277659
> Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
> Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#602346}

TBR=ulan@chromium.org,perezju@chromium.org

Change-Id: I5a656976e63444aa583d1c35eea62c34da27a551
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 878390
Reviewed-on: https://chromium-review.googlesource.com/c/1298415Reviewed-by: Ben Pastene <bpastene@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602441}

Valid DNS responses need the QR bit set to 1. See
https://tools.ietf.org/html/rfc1035#section-4.1.1

This CL adds verification to the AsyncDNS resolver to weed out invalid
responses.

R=mmenke@chromium.org

Bug: None
Change-Id: Ieda75f13394564f4617a0752d5740c9feccca850
Reviewed-on: https://chromium-review.googlesource.com/c/1292372
Commit-Queue: Asanka Herath <asanka@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602440}

Bug: 898534
Test: manual: no functional/visual change
Change-Id: I32aebe966f4a9fc22f51652ddf797583909ded9f
Reviewed-on: https://chromium-review.googlesource.com/c/1298122
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602439}

TBR=jbudorick

Bug: 893614
Change-Id: I87127f02cfb44c1be8c8c6fa6d80544b121bcc4d
Reviewed-on: https://chromium-review.googlesource.com/c/1298328Reviewed-by: Ben Pastene <bpastene@chromium.org>
Commit-Queue: Ben Pastene <bpastene@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602438}

TouchSelectionControllerClientAura should include set, also removing an
unnecessary semicolon.

Bug: None
Test: Build.
Change-Id: Ie12084e61cce65a71fcdd6a7878cb3cbd0293270
Reviewed-on: https://chromium-review.googlesource.com/c/1297980Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Commit-Queue: David Jacobo <djacobo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602437}

Also removed GetFilterStatus and SetFilterStatus from the Netstack
interface.

CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio
TBR=kmarshall@chromium.org

Change-Id: I17df2c2cb63d27a421fda742d5fd8a3fcbae5036
Reviewed-on: https://chromium-review.googlesource.com/c/1298322
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602436}

This CL sets the fullscreen progress of the toolbar to 1 (meaning
fullscreen isn't enabled) when the toolbar view is created.
It allows the constraints to be correctly positioned the first time
the toolbar is shown.

Bug: 895870
Change-Id: I481342521cd1b396f4d00c9f6dcc2a8a3f4cb1eb
Reviewed-on: https://chromium-review.googlesource.com/c/1297975
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602435}

This is a change to the chromium/src tricium project config,
which would make it so that Tricium jobs are only run for
members of the given group (project-chrome-tryjob-access).

Others could potentially manually run jobs; this would affect
the behavior for analysis run when polling gerrit for new
patchsets. Related code:
https://cs.chromium.org/chromium/infra/go/src/infra/tricium/appengine/gerrit/poll.go?l=401

Bug: 894541
Change-Id: I1da729a7dd64ac2862c8057b32ced35063bfcf6e
Reviewed-on: https://chromium-review.googlesource.com/c/1295389Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602434}

There is no reason to depend on libjingle_xmpp:rtc_task_runner from
//third_party/webrtc_overrides.

It was causing symbol visibility errors in the WebRTC component build
and after looking more carefully into it, it seems that WebRTC doesn't
need it, it is used in //jingle so this CL moves the dependency where
it is needed.

Bug: 896154
Change-Id: Icf24838a7b6220d5cabce1878b128caa3fe2fc9e
Reviewed-on: https://chromium-review.googlesource.com/c/1297359
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Henrik Grunell <grunell@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602433}

Remove the |useNewWallpaperPicker| checks, and modify some comments.

Will need follow-up CLs to:
1) Remove the flag in //ash.
2) Take a second pass to check which CSS styles/functions/variables
   are unused.

Bug: 812085
Change-Id: I7faa99c16715360a95feed50b5041d89c8c322a1
Reviewed-on: https://chromium-review.googlesource.com/c/1297308Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602432}

Bug: NONE
Change-Id: I9fe62588a7d68d0350867c30b8cc9821a05d167d
Reviewed-on: https://chromium-review.googlesource.com/c/1296685
Commit-Queue: Tommy Li <tommycli@chromium.org>
Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602431}

Appears calling the syscall over and over can be slow.

Use TLS to stash the result of the syscall.

Benchmarks:

Without change:
taskset -c 0 base_perftests --gtest_filter=SequenceManagerPerfTest.RunTenThousandImmediateTasks_OneQueue*
*RESULT task: run 10000 immediate tasks with one queue SequenceManager with message loop= 23483.009389671362 us/run
*RESULT task: run 10000 immediate tasks with one queue SequenceManager with message pump= 17142.489726027397 us/run
*RESULT task: run 10000 immediate tasks with one queue message loop= 5126.6598360655735 us/run
*RESULT task: run 10000 immediate tasks with one queue single thread in WorkerPool= 9817.776470588235 us/run

With change:
taskset -c 0 base_perftests --gtest_filter=SequenceManagerPerfTest.RunTenThousandImmediateTasks_OneQueue*
*RESULT task: run 10000 immediate tasks with one queue SequenceManager with message loop= 15673.59375 us/run
*RESULT task: run 10000 immediate tasks with one queue SequenceManager with message pump= 8832.218694885361 us/run
*RESULT task: run 10000 immediate tasks with one queue message loop= 5149.919670442842 us/run
*RESULT task: run 10000 immediate tasks with one queue single thread in WorkerPool= 9926.059523809523 us/run

BUG=898294

Change-Id: I414ff18fd0c92a3837b50e20165bf295b4f63082
Reviewed-on: https://chromium-review.googlesource.com/c/1291710Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602430}

      +735,681 bytes normalized apk size
            +5 zip entries file count
       +62,424 bytes main lib size
      +456,644 bytes main dex size
      +281,767 bytes APK size
      +816,041 bytes Estimated installed size (Android Go)
    +2,126,608 bytes Estimated installed size
InstallBreakdown (+2,126,513 bytes):
        +2,730 bytes unwind_cfi (dev and canary only) size
        +1,897 bytes Non-compiled Android resources size
       +62,424 bytes Native code size
          +357 bytes Package metadata size
    +2,043,421 bytes Java code size
       +15,684 bytes Compiled Android resources size
        +2,324 entries fields
        +3,228 entries methods
          +759 entries types
        +1,627 entries strings

Specifics: 
InstallSize: 
Dex: 
Change-Id: I3bf1992956cbdc7384fa5450fb631b0f9ee7f975
Reviewed-on: https://chromium-review.googlesource.com/c/1298114Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
Commit-Queue: Filip Gorski <fgorski@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602429}

This reverts commit 1c6f831f.

Reason for revert: Revert due to FindIt analysis.

Original change's description:
> Don't SetVisible on swapped out RenderWidgets, and drop gpu resources.
> 
> When a RenderWidget is created for a RenderViewImpl, it may be
> considered "swapped out". This means the main frame for this
> RenderView's frame tree is not local, the RenderView exists to
> hold some other local subframe, which will be composited and have
> a RenderWidget. Thus the RenderWidget attached to the RenderViewImpl
> is not actually used, there is no output from the remote main frame
> in this tree. I like to refer to these swapped out RenderWidgets as
> zombies.
> 
> During navigations, a RenderViewImpl's main frame may change to or
> from being local, in which case the RenderWidget attached to it changes
> to or from being a zombie.
> 
> When becoming a zombie, the WebFrameWidget attached to the
> RenderViewImpl (wrapping the WebViewImpl) is destroyed, and this
> marks the compositor as SetVisible(false) on the RenderWidget
> (via the WebViewImpl).
> 
> When becoming alive again, a new WebFrameWidget is attached to the
> RenderViewImpl, and the WebFrameWidget marks the compositor as
> SetVisible(true) on the RenderWidget (via the WebViewImpl).
> 
> While a zombie, the WebViewImpl is also told not to allow any
> visibility changes to be passed along to the RenderWidget. This is
> because as the local subframe becomes visible with the Page, we'd
> tell the WebView about this, to tell the Page. But since the
> WebView now has a zombie RenderWidget, it shouldn't also tell that.
> 
> Note when the RenderViewImpl is created and initially swapped out
> it creates a RenderWidget but nothing tells its compositor about the
> zombie state of affairs, so it becomes visible, which makes it
> acquire a Gpu channel and start its scheduler etc. Woops.
> 
> This CL rearranges things a bit.
> - A new RenderWidget does not SetVisible(true) if swapped out.
> - RenderViewImpl directly marks it SetVisible(false) when removing the
> WebFrameWidget.
> - RenderViewImpl *also* removes the LayerTreeFrameSink to drop the
> Gpu channel since we don't need to be fast at becoming visible (the
> RenderWidget is a zombie!)
> - RenderViewImpl directly marks it as SetVisible(true) when getting
> a new WebFrameWidget as the RenderWidget stops being a zombie.
> - WebViewImpl stops calling SetVisible on the RenderWidget when its
> main frame is not visible. In the future the RenderWidget should be
> a member of the main LocalFrame, and thus won't even exist when the
> main frame is not local.
> 
> R=​dcheng@chromium.org, piman@chromium.org
> 
> Change-Id: I87a5d0dfeaaf08cb91c9348b26b4205db55b3a81
> Bug: 894899, 419087
> Reviewed-on: https://chromium-review.googlesource.com/c/1290140
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Antoine Labour <piman@chromium.org>
> Commit-Queue: danakj <danakj@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#602389}

TBR=danakj@chromium.org,dcheng@chromium.org,piman@chromium.org

Change-Id: I2ec1cc5c7bfa723f4fafea7efe01d38ae5019532
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 894899, 419087
Reviewed-on: https://chromium-review.googlesource.com/c/1298330Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602428}

This CL adds Java end to end test for download location change feature.

Bug: 896873
Change-Id: I0dd89dab0f1a4c36726f0795231a3116c00955c2
Reviewed-on: https://chromium-review.googlesource.com/c/1294670
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602427}

Use webstateImpl method for tests instead.
Also remove tab_private includes where it's no longer used

Bug: 620465, 898381
Change-Id: I0db31eecf93eef1f8e88aa4428f16f25ff3a03b7
Reviewed-on: https://chromium-review.googlesource.com/c/1298243Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602426}

When during demo mode setup error occurs the message displayed to the
user contains description of the problem and suggested recovery method.
It helps users as well as support and developers.

Bug: 889604
Test: demo setup unit and browser tests.
Change-Id: Ia2ea4b1d9e1b4667ca9dde139d3a4aaba3a58cd0
Reviewed-on: https://chromium-review.googlesource.com/c/1280311Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Reviewed-by: Rahul Chaturvedi <rkc@chromium.org>
Commit-Queue: Aga Wronska <agawronska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602425}

This CL removes session network stats and historic network stats shown in
net-log bandwidth tab.

Bug: 894946
Change-Id: I8a18f2ae62c17bf877af398c61168b05da315c6c
Reviewed-on: https://chromium-review.googlesource.com/c/1297646Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Commit-Queue: rajendrant <rajendrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602424}

Remove with its users.

Bug: 898379
Change-Id: Ie5a127c17c93b811e230c19b5998d81feee81270
Reviewed-on: https://chromium-review.googlesource.com/c/1298234Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602423}

Not used.

Bug: 898382
Change-Id: I796eac237414917a12944a2ce67b3963f035dfb0
Reviewed-on: https://chromium-review.googlesource.com/c/1297463Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602422}

Biggest non-style/c++11 change is to convert from NotificationRegistrar to
BrowserListObserver.

This should not result in any functional change.

BUG=889097,268984
TEST=covered by tests

Change-Id: I8ea1c89f28c66f312fbcfda925591f96852d2173
Reviewed-on: https://chromium-review.googlesource.com/c/1298123Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602421}

Update the expected behavior for OnDestroyedCompositorFrameSink
and OnInvalidatedFrameSinkId:
OnDestroyedCompositorFrameSink(element): Set the |element| to not
connected, move all its children to |element_root| and
remove |element| from its parents.
OnInvalidatedFrameSinkId(element): free the memory of |element|
and follow the same steps as OnDestroyedCompositorFrameSink.

TEST=Manual, doesn't crash after connected.

Change-Id: I06eb18fe14a017d50fc8209cc20251da9ed7f0a5
Reviewed-on: https://chromium-review.googlesource.com/c/1294411Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: kylechar <kylechar@chromium.org>
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: Yi Xu <yiyix@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602420}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/8bc0f79a..7bd88028

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: Ie3639e3a07bc6e252d1bd6b74d59c84a1858384c
Reviewed-on: https://chromium-review.googlesource.com/c/1298224Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#602419}

Bug: 806868
Change-Id: I9aa48df17b16259fd9635e226c6bb9ea347e0fda
Reviewed-on: https://chromium-review.googlesource.com/c/1296595
Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Stephane Zermatten <szermatt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602418}

Bug: 808287, 898088
Change-Id: I7aa9dd73aa405e32a46118dfe8474c7849471caf
Reviewed-on: https://chromium-review.googlesource.com/c/1297460Reviewed-by: Nathan Parker <nparker@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602417}

No more needed 'NullString' identifier because
'[TreatNullAs=NullString]' doesn't belong per spec.

Bug: 497307
Change-Id: I06356118e3b061fb5801fd1704dd470d7f13407e
Reviewed-on: https://chromium-review.googlesource.com/c/1293046
Commit-Queue: Jinho Bang <jinho.bang@samsung.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602416}

Bug: 870611
Change-Id: I311f2dcdafe25fea71ce0494424602e0e1989bb7
Reviewed-on: https://chromium-review.googlesource.com/c/1298174Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602415}

Bug: 883456
Change-Id: I990f680aeb7cb7c18571abc63c09573bcc6f3b1c
Reviewed-on: https://chromium-review.googlesource.com/c/1297486
Commit-Queue: Alexandre Frechette <frechette@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602414}

(Probably could use some improvement to make file:/// work more
Fuchsia-like, but this at least makes the file compile.)

Bug: 808287
Change-Id: Id4476f6ec9d2f37006df439a72855587bb18b4c3
Reviewed-on: https://chromium-review.googlesource.com/c/1298178Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Scott Graham <scottmg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602413}

Using wpt-import in Chromium c294aaed.
With Chromium commits locally applied on WPT:
f79c6fe7 "Reimplementing getDatabaseNames() as databases() for indexeddb."
89bcea96 "[run_web_tests] Remove extra baselines after crrev.com/c/1277654"
36e64bc7 "Updating tests for Window onerror is not triggered bug fix."


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

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

NOAUTOREVERT=true
TBR=kyleju

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

https://chromium-review.googlesource.com/c/chromium/src/+/1253206 made WebApk.apk and
NewSplashWebApk.apk use the same AndroidManifest.xml template. In doing so, the CL
caused the NewSplashWebApk.apk-specific-code in AndroidManifest.xml to be uploaded to
the WebAPK server.

This CL changes the "webapk_tmpl" GN template to run pystache twice.
Pass #1 to generate AndroidManifest.xml to be uploaded to the WebAPK server. Fields
        which cannot be customized by the WebAPK server on a per-site basis are filled
        in.
Pass #2 fills in the remaining fields to generate AndroidManifest.xml for the test APK
        (WebApk.apk, NewSplahWebAPk.apk). These APKs are for local testing and are not
        uploaded to the WebAPK server.

BUG=817263

Change-Id: I3d4849d122195ba2e3d82bfb7bb5661dd765e3f7
Reviewed-on: https://chromium-review.googlesource.com/c/1279953
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602411}

https://skia.googlesource.com/skia.git/+log/472639d9f12f..2d123ad73f85


git log 472639d9f12f..2d123ad73f85 --date=short --no-merges --format='%ad %ae %s'
2018-10-24 herb@google.com Remove unique glyph IDs from glyph runs
2018-10-24 jvanverth@google.com Reland "Some more clean-up of YUVA code"
2018-10-24 bsalomon@google.com Make GrOp base class responsible for merging bounds.


Created with:
  gclient setdep -r src/third_party/skia@2d123ad73f85

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

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-chromeos-compile-dbg;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
TBR=brianosman@chromium.org

Change-Id: Ia06d6e9d8d70930b9cba6883d99b852163f79080
Reviewed-on: https://chromium-review.googlesource.com/c/1298154Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#602410}

PreconnectManager is designed with restriction on the maximum number of
simultaneous dns requests. There are instances of violation of this
constraint.

Each dns job has an alternative proxy resolve job that can finish first,
cancel pending dns job and schedule a new one. However, this cancelling
is asynchronous and wouldn't have an effect if a dns job was already
scheduled in the network stack.

This CL makes the PreconnectManager wait for dns job being cancelled
before scheduling a new one.

Bug: 883806
Change-Id: I8e90ad0dce55b569a4c77b202435f70a6b776231
Reviewed-on: https://chromium-review.googlesource.com/c/1292880Reviewed-by: Benoit L <lizeb@chromium.org>
Commit-Queue: Alexandr Ilin <alexilin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602409}

This is deprecated in favor of explicitly checking with v8::Maybe<bool>::To.

Bug: 670615
Change-Id: I71e8f3c1f643a7eabfaf25359581c15565513302
Reviewed-on: https://chromium-review.googlesource.com/c/1298049Reviewed-by: Fernando Serboncini <fserb@chromium.org>
Commit-Queue: Jeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602408}

Bug: 774054
Change-Id: I42ce8c29699701aec0df0de806ca5077bf49a056
Reviewed-on: https://chromium-review.googlesource.com/c/1297144Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602407}

This reverts commit e9928022.

Reason for revert: Speculative fixes have landed in skia.

Original change's description:
> Revert "cc: Cache text blobs in the GPU service."
> 
> This reverts commit 57df0676.
> 
> Reason for revert: Seems to be causing a crash on Mac.
> 
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac%20FYI%2010.14%20Release%20%28Intel%29/327
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac%20FYI%20Experimental%20Release%20%28Intel%29/6072
> 
> Crash in __ZN13GrCCPathCache5evictEP18GrCCPathCacheEntry
> 
> Bug: 897507
> 
> Original change's description:
> > cc: Cache text blobs in the GPU service.
> > 
> > Since skia's internal caching is keyed on text blobs, its important to
> > reuse the same object. Make sure we cache text blobs in the
> > ServiceTransferCache to get a cache hit in skia.
> > 
> > R=​enne@chromium.org
> > 
> > Bug: 894200
> > Change-Id: I8808d649ca3520211d034e07e7d4670ffb39094c
> > Reviewed-on: https://chromium-review.googlesource.com/c/1292722
> > Commit-Queue: Khushal <khushalsagar@chromium.org>
> > Reviewed-by: enne <enne@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#601404}
> 
> TBR=enne@chromium.org,khushalsagar@chromium.org
> 
> # Not skipping CQ checks because original CL landed > 1 day ago.
> 
> Bug: 894200
> Change-Id: I5e0a151c5a539aa8a9b6cac5f9bcf710ba2f405a
> Reviewed-on: https://chromium-review.googlesource.com/c/1292384
> Reviewed-by: Jamie Madill <jmadill@chromium.org>
> Commit-Queue: Jamie Madill <jmadill@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#601451}

TBR=jmadill@chromium.org,enne@chromium.org,khushalsagar@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 897507, 894200
Change-Id: I345adbbfaff91ab201d68e73e861dd4ccce41775
Reviewed-on: https://chromium-review.googlesource.com/c/1298235Reviewed-by: Khushal <khushalsagar@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Khushal <khushalsagar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602406}

This change will only allow blob URLs loading into WKWebView. blob URL
downloads are still unsupported.

Bug: 847073
Change-Id: I09a963c5b427eb7e3f8530f5b999c34da0fcfa7a
Reviewed-on: https://chromium-review.googlesource.com/c/1297390Reviewed-by: Danyao Wang <danyao@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602405}

This special serialization behavior belongs in the css directory. Also, in
a subsequent CL (for css-properties-values-api), it will be necessary to
expose the alpha rounding part of the serialization, and I don't want to
add this function to platform/color.h.

Change-Id: I4512e695016b6495c6b9197b521cda6431642962
Reviewed-on: https://chromium-review.googlesource.com/c/1293614Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Anders Ruud <andruud@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602404}