Commits · ecd6abcb05f4358406773630c371d61b989b1084 · eriksson monteiro / tangled

25 Jan, 2018 40 commits

Roll src/third_party/skia/ be84b720c..eeaa89d1f (1 commit) · ecd6abcb

skia-deps-roller@chromium.org authored Jan 25, 2018

https://skia.googlesource.com/skia.git/+log/be84b720c778..eeaa89d1f406

$ git log be84b720c..eeaa89d1f --date=short --no-merges --format='%ad %ae %s'
2018-01-25 angle-skia-autoroll Roll skia/third_party/externals/angle2/ 2e551f6bc..2d8e432a9 (1 commit)

Created with:
  roll-dep src/third_party/skia


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;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
TBR=robertphillips@chromium.org

Change-Id: I80a9bf8fe39e717d9568755c2dcaff2eaa659694
Reviewed-on: https://chromium-review.googlesource.com/885916
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531879}

ecd6abcb

[scheduler] Disable MSVC optimisations for memory corruption sentinel · 63f18ec5

Alexander Timin authored Jan 25, 2018

R=skyostil@chromium.org

Change-Id: Idf979483eb3e66fbd151d2f8c0834d927b707c97
Reviewed-on: https://chromium-review.googlesource.com/874693Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Commit-Queue: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531878}

63f18ec5

Roll src/third_party/webrtc/ 8ed653d1c..b9b07eaf2 (15 commits) · 4ab310b3

webrtc-autoroll@chromium.org authored Jan 25, 2018

https://webrtc.googlesource.com/src.git/+log/8ed653d1c4ea..b9b07eaf281a

$ git log 8ed653d1c..b9b07eaf2 --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtc
BUG=None,None,None


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


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I62d00a18ff7b923b642ec6a45bc1c5c76bea89bb
Reviewed-on: https://chromium-review.googlesource.com/886143Reviewed-by: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531877}

4ab310b3

[jumbo] add some missing include guards to //gpu headers · 9fa5fa7e

Mostyn Bramley-Moore authored Jan 25, 2018

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: Id8bad8fbcce92bb46a926a7b8fa4e357ad7c052c
Reviewed-on: https://chromium-review.googlesource.com/882321Reviewed-by: Victor Miura <vmiura@chromium.org>
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Cr-Commit-Position: refs/heads/master@{#531876}

9fa5fa7e

[ServiceWorker] Move ServiceWorkerClientInfoPtr through impl of service_worker_client_utils · ae2e0b3d

Han Leon authored Jan 25, 2018

This CL starts to move ServiceWorkerClientInfoPtr through impl of
service_worker_client_utils rather than passing value of
ServiceWorkerClientInfo struct.

BUG=772793

Change-Id: I852efee05dfda6f86b0486912827c29b00071a9d
Reviewed-on: https://chromium-review.googlesource.com/886141
Commit-Queue: Han Leon <leon.han@intel.com>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531875}

ae2e0b3d

[Sensors][DeviceOrientation]Browser-side Feature Policy check · ff0799d2

Mikhail Pozdnyakov authored Jan 25, 2018

The sensor policy-controlled features should be
checked in the browser process, in order to improve
security robustness.

This patch introduces browser-side feature policy check
for both the Generic Sensor and the Device Orientation APIs.

Bug: 796894
Bug: 795723
Change-Id: Idc9869d75bca70f3f9ee915101195c7cbee22587
Reviewed-on: https://chromium-review.googlesource.com/850699
Commit-Queue: Mikhail Pozdnyakov <mikhail.pozdnyakov@intel.com>
Reviewed-by: Tim Volodine <timvolodine@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Jun Cai <juncai@chromium.org>
Reviewed-by: Alexander Shalamov <alexander.shalamov@intel.com>
Cr-Commit-Position: refs/heads/master@{#531874}

ff0799d2

telemetry: Disable smoothness.key_silk_cases page using expectations. · e7276154

Sunny Sachanandani authored Jan 25, 2018

key_silk_cases.Page26 was disabled by manually removing it from the
StorySet. This is probably a relic of a time before story expectations
were implemented. This CL disables the page using expectations.

BUG=461127

Change-Id: I055b6bd78667a16b59120211698dd0281419e25e
Reviewed-on: https://chromium-review.googlesource.com/884987Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#531873}

e7276154

Add traffic annotation to WebSocketBasicStream. · cdb199a6

Ramin Halavati authored Jan 25, 2018

Network traffic annotaiton is added to the socket write in
WebSocketBasicStream::WriteEverything().

Bug: 656607
Change-Id: Id9ae38031afb9b8bd6485c1b6db8932a83527fa8
Reviewed-on: https://chromium-review.googlesource.com/877888
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531872}

cdb199a6

[PE] Deflake two virtual/android/rootscroller tests · a53fa339

Xianzhu Wang authored Jan 25, 2018

Single requestAnimationFrame doesn't do the expected work. Double
requestAnimationFrames do. And we have runAfterLayoutAndPaint for
layout tests for more accurate control.

Change-Id: I19ece0ff7887d33736ebece79beb6495e4300bca
Reviewed-on: https://chromium-review.googlesource.com/886142
Commit-Queue: David Bokan <bokan@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531871}

a53fa339

[jumbo] avoid ambiguity between blink:: and blink::XPath:: types · 6fc13430

Mostyn Bramley-Moore authored Jan 25, 2018

Fix courtesy of Fredrik Söderquist <fs@opera.com>

Bug: 421909

Change-Id: I69f4f71eb534ebdb885d34ec645c668e9b2772fd
Reviewed-on: https://chromium-review.googlesource.com/885808
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#531870}

6fc13430

Update V8 to version 6.6.67. · 9e0f6fd0

v8-autoroll authored Jan 25, 2018

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/a9bb5cf9..5bc21ae5

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;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=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org

Change-Id: Ifbe9a2666fde228161a85af26d97734d1e74e1de
Reviewed-on: https://chromium-review.googlesource.com/886382Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531869}

9e0f6fd0

Replace the default timer task runner usages in tests · d3f6ae4d

Hajime Hoshi authored Jan 25, 2018

This is part of efforts to replace non-per-frame task runners with other
appropriate per-frame task runners in the renderer.

Bug: 804694
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ifcbf610a71e87f5570c3c0b5ffcda97fe10bfc39
Reviewed-on: https://chromium-review.googlesource.com/886441Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531868}

d3f6ae4d

Add traffic annotation to Jingle Glue Sockets. · b2940b17

Ramin Halavati authored Jan 25, 2018

Network traffic annotation is added to socket writes of:
jingle/glue/chrome_async_socket.cc
jingle/glue/fake_ssl_client_socket.cc

Bug: 656607
Change-Id: I01e083c9a7154a6fc5071839a4f26007e3717ea4
Reviewed-on: https://chromium-review.googlesource.com/885812Reviewed-by: Nicolas Zea <zea@chromium.org>
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531867}

b2940b17

Use specialized Mirror reconciliation for child accounts on Chrome OS · 0d47ae36

Kush Sinha authored Jan 25, 2018

Child accounts on Chrome OS will be forcibly logged out if account
reconciliation timeout is reached without a successful reconciliation.
Other accounts will maintain their existing behavior.

Bug: 795808
Change-Id: If7dc163e2d0921a7e360aed9673b235d1a2d9c7c
Reviewed-on: https://chromium-review.googlesource.com/883285
Commit-Queue: Kush Sinha <sinhak@chromium.org>
Reviewed-by: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531866}

0d47ae36

Roll src-internal/ 8af6931d8..9c4fd2431 (1 commit) · ef41f9f9

src-internal-roller@chromium.org authored Jan 25, 2018

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/8af6931d8777..9c4fd243124b

$ git log 8af6931d8..9c4fd2431 --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src-internal


The AutoRoll server is located here: https://src-internal-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=phajdan.jr@chromium.org

Change-Id: Iace4719648870ad5951663e0da058714afba8ed9
Reviewed-on: https://chromium-review.googlesource.com/885911
Commit-Queue: src-internal-roller . <src-internal-roller@chromium.org>
Reviewed-by: src-internal-roller . <src-internal-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531865}

ef41f9f9

OmniboxPopup don't need toolbar view anymore · db8e1009

Gauthier Ambard authored Jan 25, 2018

This CL changes the OmniboxPopup such as it doesn't need access to the
toolbar view to be presented. Instead it only needs a pointer to its
superview.
The pointer to the view was used to present the popup above or below it
to show the shadow between the omnibox and the popup or not.

This CL changes it by adding the shadow if needed in the popup.

It also removes the white rectangle recovering the place normally
occupied by the toolbar during NTP animation (to avoid seeing the
google logo moving up as the toolbar is hidden). To do that, the
toolbar is directly shown if there are results to be displayed.

Bug: 786940
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I6a8dea25702b4644f0f0c76fc34b01c1deaf6a88
Reviewed-on: https://chromium-review.googlesource.com/881182
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Stepan Khapugin <stkhapugin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531864}

db8e1009

[wasm][streaming] Call Abort without reason when executed in ScriptForbiddenScope · dbd136e4

Andreas Haas authored Jan 25, 2018

It can happen that WebAssembly.compileStreaming gets aborted when we
are not allowed to execute JavaScript code, and therefore are also not
allowed to reject the promise returned by WebAssembly.compileStreaming.
This can happen e.g. when the Chrome tab gets refreshed, which aborts
all downloads.

With this CL we do not pass a reason to Abort if we are not allowed to
execute JavaScript code. On the V8 side we can check the reason passed
to Abort and do not reject the promise if there is no reason passed.

The V8 side change is at crrev.com/c/876091

R=mtrofin@chromium.org

Bug: chromium:803838
Change-Id: I55727f97fac38790cadc34ec5284131e13642f20
Reviewed-on: https://chromium-review.googlesource.com/876103Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
Commit-Queue: Andreas Haas <ahaas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531863}

dbd136e4

Remove old bisect-related scripts · 70d7b68f

Juan Antonio Navarro Perez authored Jan 25, 2018

These do no longer work.

Bug: 804863
Change-Id: Id0bfff21aa9d7fc4338a37bdfcdf9d820ce32ea5
Reviewed-on: https://chromium-review.googlesource.com/881321Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Simon Hatch <simonhatch@chromium.org>
Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531862}

70d7b68f

[LayoutNG] Change how content editable forces legacy layout · 302e6c7c

Yoshifumi Inoue authored Jan 25, 2018

This patch changes how editing forces legacy layout to the one
added for form controls[1] to use legacy layout tree for nested editable, e.g.
  <div contenteditable>
  legacy layout tree
    <div contenteditable=false>
      non-editable block in editable block should be legacy
    </div>
  </div>

This patch continues http://crrev.com/c/880542

[1] http://crrev.com/c/875511 Change form controls not to use LayoutNG

Bug: 714962
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I78d44fc82b3de5e647b67ee6aa7733db62e344b8
Reviewed-on: https://chromium-review.googlesource.com/880530
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531861}

302e6c7c

Add a function to get current virtual time to WebScheduler. · bde15e9c

Yuta Kitamura authored Jan 25, 2018

This patch adds a new function to WebScheduler so that Timer can obtain
the current virtual time without using WebTaskRunner::Monotonically-
IncreasingVirtualTimeSeconds().

WebTaskRunner::MonotonicallyIncreasingVirtualTimeSeconds() is not
deleted in this patch, though, because there are some tests using it
and they deserve a separate patch.

Bug: 794845
Change-Id: I21c2f7accc86c7216ee917e5c29bc0ad3ec44b07
Reviewed-on: https://chromium-review.googlesource.com/882703
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531860}

bde15e9c

Re-enable stats WPT tests after WebRTC roll · 4cbc3e4e

Harald Alvestrand authored Jan 25, 2018

Bug: 801642
Change-Id: Ib92b6d857d5add650a757618adb38d7c7ed5c8f7
Reviewed-on: https://chromium-review.googlesource.com/885801Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531859}

4cbc3e4e

Revert "android: Don't parse /proc/self/maps to prefetch the library." · 733c2818

Benoit L authored Jan 25, 2018

This reverts commit ef6f0fb3.

Reason for revert: Regressed startup performance.
See https://chromeperf.appspot.com/report?sid=9df0ec76b0fffc74931cc7bdcf44cd4d540f81bb34ccde3571c2211ec8360ee6&start_rev=531281&end_rev=531574

Original change's description:
> android: Don't parse /proc/self/maps to prefetch the library.
> 
> This commit changes the behavior of the native library prefetcher to
> rely on sentinel symbols rather than parsing /proc/self/maps.
> 
> Behavior changes:
> 1. This will make the code no longer prefetch .data (because we don't
>    look for it in the mappings), neither the sections that are part of
>    the same mapping as .text (for instance, .rodata when using
>    ld.gold). This is intended.
> 2. As a consequence, the UMA metric
>    LibraryLoader.PercentageOfResidentCodeBeforePrefetch will move, as
>    both sides of the ratio will change, and the population will change
>    slightly as well (since code is currently not correctly ordered on
>    ARM64 and x86_64)
> 
> - Removes the reliance on a somewhat brittle parsing of a file
> - Parsing /proc/self/maps costs ~50ms of CPU on a Nexus 5X (on a little
>   core, as this is triggered from an AsyncTask)
> - Allows to only fetch part of the native library (in a forthcoming CL)
> 
> Rationale: 
> Change-Id: I0bb7b28af3c3bd4af5745e2ebcc1fbf283bcc0c1
> Reviewed-on: https://chromium-review.googlesource.com/874470
> Commit-Queue: Benoit L <lizeb@chromium.org>
> Reviewed-by: Egor Pasko <pasko@chromium.org>
> Reviewed-by: agrieve <agrieve@chromium.org>
> Reviewed-by: Matthew Cary <mattcary@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#531515}

TBR=pasko@chromium.org,agrieve@chromium.org,lizeb@chromium.org,mattcary@chromium.org

Change-Id: I11d7cd7898d5f8d8fbebf391eba5fdaca154c08a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/886321Reviewed-by: Benoit L <lizeb@chromium.org>
Commit-Queue: Benoit L <lizeb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531858}

733c2818

Update V8 to version 6.6.66. · 962060a1

v8-autoroll authored Jan 25, 2018

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/50c5a55c..a9bb5cf9

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.

TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org

Change-Id: Ifcb97dcfbe3b4eabb6c8a061ab4d6fa0abde46c8
Reviewed-on: https://chromium-review.googlesource.com/886061Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531857}

962060a1

Add parameters to jpeg_encode_accelerator_unittest · f691b1d9

Sheng-Hao Tsao authored Jan 25, 2018

This CL adds some command line parameters to
jpeg_encode_accelerator_unittest to enable JEAPerf autotest.

BUG=b:67488323
TEST=Verified that JEAPerf collects metrics successfully on eve.

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: Ie369b17302e6f6587e81126ed35d577e71e00d68
Reviewed-on: https://chromium-review.googlesource.com/874493Reviewed-by: Ricky Liang <jcliang@chromium.org>
Reviewed-by: Wu-Cheng Li <wuchengli@chromium.org>
Commit-Queue: Sheng-hao Tsao <shenghao@google.com>
Cr-Commit-Position: refs/heads/master@{#531856}

f691b1d9

WebUI: Update Polymer paper-fab version 1.2.0 -> 2.0.0 · 3b416c2b

dpapad authored Jan 25, 2018

Also removing paper-material, since the previous paper-fab version was the last
usage of it.

Bug: 738611
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ic1ec8e70d5c5189ead7cb62c9e25f8f8dde5afee
Reviewed-on: https://chromium-review.googlesource.com/883565Reviewed-by: Scott Chen <scottchen@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531855}

3b416c2b

Make LayoutObject::EnclosingBlockFlowFragment() strict · dd9f5030

Yoshifumi Inoue authored Jan 25, 2018

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Bug: 714962
Change-Id: I14e724e06247c59e2f80a8a4db7dc1ef6e5433c7
Reviewed-on: https://chromium-review.googlesource.com/885542Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531854}

dd9f5030

DevTools: split persistence bindings from automapping bindings · 4859b2d9

Andrey Lushnikov authored Jan 25, 2018

This patch introduces AutomappingBinding, that has additional
information related to the automapping process.

For now, the automappingBinding has only "exact" boolean flag.
In future, AutomappingBinding will be created for both successful
and failed bindings, surfacing binding errors.

R=dgozman
Bug=805740

Change-Id: I6245a8b26ae278635add9ef7a46e60886d59bd29
Reviewed-on: https://chromium-review.googlesource.com/884989
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531853}

4859b2d9

Roll src/third_party/catapult/ e99797e78..62619cb4a (1 commit) · d8100dc0

catapult-deps-roller@chromium.org authored Jan 25, 2018

https://chromium.googlesource.com/catapult.git/+log/e99797e78121..62619cb4a9ae

$ git log e99797e78..62619cb4a --date=short --no-merges --format='%ad %ae %s'
2018-01-23 sadrul tabs: Add a TBM metric for tab_switching.

Created with:
  roll-dep src/third_party/catapult
BUG=802104


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

d8100dc0

WebUI: Update Polymer paper-button version 1.0.13 -> 2.0.0 · bb42e883

dpapad authored Jan 25, 2018

Also had to update paper-styles version 1.1.4 -> 1.3.1, and removed
the deprecated and unused shadow-layout.html.

Bug: 738611
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I43fc909502ac321e3a6a512ca6b00c93a98abab2
Reviewed-on: https://chromium-review.googlesource.com/882446Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531851}

bb42e883

Revert "Introduce services/network/public/cpp/features.h" · 9030bb3e

Takashi Sakamoto authored Jan 25, 2018

This reverts commit 1f1519bd.

Because this causes Mac Retina Debug bot failure, e.g.
https://ci.chromium.org/buildbot/chromium.gpu/Mac%20Retina%20Debug%20%28AMD%29/18470

[18936:47619:0124/230505.322884:FATAL:feature_list.cc(328)] Check failed: CheckFeatureIdentity(feature). NetworkService
0 libbase.dylib 0x00000001029659be base::debug::StackTrace::StackTrace(unsigned long) + 174
1 libbase.dylib 0x0000000102965a7d base::debug::StackTrace::StackTrace(unsigned long) + 29
2 libbase.dylib 0x0000000102963efc base::debug::StackTrace::StackTrace() + 28
3 libbase.dylib 0x00000001029fd0ef logging::LogMessage::~LogMessage() + 479
4 libbase.dylib 0x00000001029fae25 logging::LogMessage::~LogMessage() + 21
5 libbase.dylib 0x000000010297598c base::FeatureList::IsFeatureEnabled(base::Feature const&) + 748
6 libbase.dylib 0x0000000102975687 base::FeatureList::IsEnabled(base::Feature const&) + 71
7 libcontent.dylib 0x000000011fb99e44 content::GetNetworkServiceImpl() + 180
8 libchrome_dll.dylib 0x0000000106dd0831 IOThread::ConstructSystemRequestContext() + 8257
9 libchrome_dll.dylib 0x0000000106dcce1a IOThread::Init() + 10666
10 libcontent.dylib 0x000000011ef901e6 content::BrowserThreadImpl::Init() + 854
11 libcontent.dylib 0x000000011ef8cfd7 content::BrowserProcessSubThread::Init() + 231
12 libbase.dylib 0x0000000102c5daa3 base::Thread::ThreadMain() + 2419
13 libbase.dylib 0x0000000102c350da base::(anonymous namespace)::ThreadFunc(void*) + 682
14 libsystem_pthread.dylib 0x00007fff924ed93b _pthread_body + 180

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

9030bb3e

Remove content deps of AudioSyncReader and AudioInputSyncWriter. · 13ddfd32

Max Morin authored Jan 25, 2018

They will be moved to media in a future CL since they need to be used
from the audio service. A side-effect of this CL is that logs from the
SyncReader/Writer are prefixed with the stream_id, like other logs are.

Bug: 803102
Change-Id: Iae2876033e4172066d5a0a3b1322092f870dc32d
Reviewed-on: https://chromium-review.googlesource.com/878627Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Olga Sharonova <olka@chromium.org>
Commit-Queue: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531849}

13ddfd32

Sanitize OWNERS for Blink Sydney Style and Animations team · 3a75923c

Naina Raisinghani authored Jan 25, 2018

Bug: 
Change-Id: I5857b11ae92b7dabc24fce281d19c2a40854fe2c
Reviewed-on: https://chromium-review.googlesource.com/885685
Commit-Queue: nainar <nainar@chromium.org>
Reviewed-by: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531848}

3a75923c

Revert "Oilpan: Rename to CrossThreadPersistentRegions" · 895ec9ea

Keishi Hattori authored Jan 25, 2018

This reverts commit 62bb75d3.

Reason for revert: Decided we don't need this change

Original change's description:
> Oilpan: Rename to CrossThreadPersistentRegions
> 
> Rename CrossThreadPersistentRegion to CrossThreadPersistentRegions as we will be adding a second PersistentRegion for weak persistents:
> 
> BUG=757440
> 
> Change-Id: I751747470c5ddc1481270e181cdd1edcf9ae5291
> Reviewed-on: https://chromium-review.googlesource.com/882626
> Reviewed-by: Kentaro Hara <haraken@chromium.org>
> Commit-Queue: Keishi Hattori <keishi@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#531830}

TBR=haraken@chromium.org,keishi@chromium.org,hpayer@chromium.org,mlippautz@chromium.org

Change-Id: Ia1ac956e678dab92d6daab0852d0d9eafc3aa08a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 757440
Reviewed-on: https://chromium-review.googlesource.com/886162Reviewed-by: Keishi Hattori <keishi@chromium.org>
Commit-Queue: Keishi Hattori <keishi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531847}

895ec9ea

Blink: Make ThreadCondition only work with Mutex. · eea5ec40

Victor Costan authored Jan 25, 2018

The interaction between ThreadCondition (condition variables) and
recursive (a.k.a. re-entrant) mutexes is difficult to reason about.
Details in the "Why can the holder of a Lock not reacquire it?" section
of https://www.chromium.org/developers/lock-and-condition-variable

Bug: 805814
Change-Id: Ie0b5e026a43768108011ca8f2f6fd3fe0e1f4b03
Reviewed-on: https://chromium-review.googlesource.com/885862
Commit-Queue: Victor Costan <pwnall@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531846}

eea5ec40

Roll src/third_party/skia/ b6f702595..be84b720c (9 commits) · 02fe30d4

skia-deps-roller@chromium.org authored Jan 25, 2018

https://skia.googlesource.com/skia.git/+log/b6f7025955d6..be84b720c778

$ git log b6f702595..be84b720c --date=short --no-merges --format='%ad %ae %s'
2018-01-25 angle-skia-autoroll Roll skia/third_party/externals/angle2/ 68981eb56..2e551f6bc (1 commit)
2018-01-24 reed compose and combine maskfilters
2018-01-24 scroggo Remove unused field/methods
2018-01-24 fmalita [skottie] Cache the last keyframe
2018-01-24 stani Revert "Increase depth threshold for SkClipStack::isRRect."
2018-01-24 fmalita [skottie] Animator scrubbing
2018-01-24 benjaminwagner [bazel] Add third_party to skia_all_hdrs
2018-01-24 angle-skia-autoroll Roll skia/third_party/externals/angle2/ 01b21b194..68981eb56 (2 commits)
2018-01-09 mtklein Start of cross process SkScalerContext.

Created with:
roll-dep src/third_party/skia

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.

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

02fe30d4

Support pool for action_foreach · 4c48a351

Takuto Ikuta authored Jan 25, 2018

This is follow up of pool support in action
https://codereview.chromium.org/2926013002

Using action pool can remove some overhead of many running process.
Pool support of action_foreach gives better control for some python generator step when using goma.
e.g. https://codereview.chromium.org/2726103005

Bug: 695864
Change-Id: Ibd0bbaffc59513db42119138520aee3505762eee
Reviewed-on: https://chromium-review.googlesource.com/882625Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@google.com>
Cr-Commit-Position: refs/heads/master@{#531844}

4c48a351

service worker: Remove non-PlzNavigate code around SWProviderHost::web_contents_getter() · c8e779e6

Matt Falkenhagen authored Jan 25, 2018

The non-PlzNavigate code path is no longer supported.

R=shimazu

Bug: 789577
Change-Id: Id0b61bd9042769d3afdcb78df6913632d833d831
Reviewed-on: https://chromium-review.googlesource.com/885687Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531843}

c8e779e6

[ServiceWorker] Apply OnceCallback through impl of service_worker_client_utils · 4b9e63a7

Han Leon authored Jan 25, 2018

BUG=

Change-Id: I6b5b28230390de1f3b8a105c1f750fc394dfe923
Reviewed-on: https://chromium-review.googlesource.com/886004Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Han Leon <leon.han@intel.com>
Cr-Commit-Position: refs/heads/master@{#531842}

4b9e63a7

Allow base::{Time,TimeDelta,TimeTicks} in Blink. · a7634160

Yuta Kitamura authored Jan 25, 2018

As of crrev.com/531516, all of the WTF counterparts of those base
classes are now simple aliases to the base ones. Now, we can start
using the base classes in Blink.

Bug: 763980, 794845
Change-Id: Ica8d81549ed6d7fd4f56ed03d22562c973e8a00e
Reviewed-on: https://chromium-review.googlesource.com/886041Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531841}

a7634160

Use root_out_dir for fdebug-prefix-map for better cache utilization on goma · 63f179b1

Takuto Ikuta authored Jan 25, 2018

rebase_path("//.") is resolved to the path like "$HOME/chromium/src".
But it is necessary to be like "$HOME/chromium/src/out/Release" for higher cache hit rate when using goma.

This patch allows us to share goma backend cache where we build in different out directory like
out/feature_a and out/feature_b if compile requests has only difference of cwd and -fdebug-prefix-map flag.

If we use same args.gn with linux_chromium_rel_ng, this patch allows us to use cached result generated by buildbot
even if we do not clone chromium to path like '/b/c/b/linux/src/' and 'out/Release'.

Note that some generated files having directive like `#include "out/Release/..."` is not normalized, but there is a few such tasks.

Change-Id: I1232415aeedc98dc203e9d7582839bcc1cdbefc6
Reviewed-on: https://chromium-review.googlesource.com/882712
Commit-Queue: Takuto Ikuta <tikuta@google.com>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Shinya Kawanaka <shinyak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#531840}

63f179b1