GitLab

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/22069a18..96fdef6d

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

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

Bug: 435047
Change-Id: I87b417cb779efd69d9b5ab2c7fa5a331b1aea19a
Reviewed-on: https://chromium-review.googlesource.com/702237Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Tomasz Garbus <tgarbus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506698}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/b2e961b1171d..b5807979e8a9

$ git log b2e961b11..b5807979e --date=short --no-merges --format='%ad %ae %s'
2017-10-04 xiaoyin.l Use HTTPS links in CheckAuthorizedAuthor
2017-10-04 dnj [vpython] Roll CIPD version.
2017-10-04 phajdan.jr gclient config: add support for custom vars
2017-10-03 phajdan.jr gclient: make predefined variables native
2017-10-03 mmoss Allow gclient solution URLs to be |None|.
2017-08-28 agable Simplify git-cl-diff for Gerrit
2017-10-03 phajdan.jr gclient flatten: properly quote conditionals
2017-09-28 phajdan.jr gclient: add support for native boolean variables
2017-09-29 phajdan.jr gclient flatten: emit conditions for hooks
2017-09-29 siggi Update fetch config with new Syzygy location.
2017-09-29 agable Remove rietveld commit queue command line client
2017-09-29 agable Remove rietveld git cherry-pick-upload command
2017-09-29 borenet roll-dep: Add comments indicating that --no-log needs to keep working
2017-09-28 mmoss Expose new gclient.sync() arg in gclient.checkout().
2017-09-07 brucedawson Fix autoninja to allow compiling one source file
2017-09-29 agable Remove defunct Rietveld end-to-end tests
2017-09-28 dnj [depot_tools] Add "ensure_bootstrap" script.
2017-09-28 agable bot_update: use patch repo instead of project if it is mapped
2017-09-27 mmoss Add ability to pass extra flags to gclient.sync().
2017-09-27 dnj Roll "vpython".
2017-09-26 agrieve Fix CheckPatchFormatted() when using inherit-review-settings-ok
2017-09-26 mmoss Remove unused gclient.sync() 'with_branch_heads' arg.
2017-09-25 phajdan.jr gclient: evaluate variables before passing them to GN
2017-09-26 phajdan.jr gclient: expose target_os
2017-09-26 yyanagisawa git-cl upload: adjust code-review score on TBR.
2017-09-25 emso Adds progress argument option to fetch for git checkout
2017-09-18 rharrison Add support for PDFium to my_activity.py
2017-09-18 rmistry Add skia_buildbot config
2017-09-14 mmoss Add some differentiating details to gerrit step names.
2017-09-15 ehmaldonado WebRTC: Remove hack in bot_update.py.
2017-09-13 agable git-cl: Parse new change urls containing project paths
2017-09-11 ehmaldonado WebRTC: Update source-of-truth on 'fetch webrtc'
2017-09-13 iannucci Roll vpython.
2017-09-13 athom Add gsutil.vpython spec to recipe bundles
2017-09-13 ehmaldonado Revert "Fix checkout.py issues when p.patchlevel > 1."
2017-09-12 ehmaldonado Fix checkout.py issues when p.patchlevel > 1.
2017-09-12 iannucci Roll led and vpython
2017-09-12 phajdan.jr gclient flatten: improve reporting of DEPS files
2017-09-11 iannucci Add s390x support for cipd client bootstrap script.
2017-09-11 agable git-cl upload: Send mail when starting CQ
2017-09-11 bjaideep gclient: Add aix to host_os list
2017-09-11 agable Add GCE instructions to git cl creds check
2017-09-11 ehmaldonado WebRTC: Update source-of-truth for 'git-cl'
2017-09-07 dcheng Use dateutil.parser for parsing dates.
2017-09-11 tikuta Update ninja to v1.8.2
2017-09-07 dnj [gsutil] run through "vpython" (2)
2017-09-07 mbjorge Revert "[gsutil] run through "vpython""
2017-09-07 dnj [gsutil] run through "vpython"
2017-09-07 phajdan.jr Revert "update_depot_tools: make cipd_bin_setup failures fatal"
2017-09-06 phajdan.jr cipd: also support wget as fetch command
2017-09-06 phajdan.jr update_depot_tools: make cipd_bin_setup failures fatal
2017-09-05 phajdan.jr gclient: fetch arbitrary refs
2017-09-06 phajdan.jr Move downstream recipe trybots to production
2017-09-06 brucedawson Revert "Fix autoninja.bat to not swallow ^^ sequences"
2017-09-06 agable Revert "Capture ctrl-c in presubmit multiprocessing pool"
2017-09-05 brucedawson Fix autoninja.bat to not swallow ^^ sequences
2017-09-06 sebmarchand Add a script to make it easier to build a single source file with ninja
2017-09-05 agable git-cl set_close: Don't fail on branches with no issue
2017-09-05 agable Remove gerrit-over-ssh code from my_activity.py
2017-09-05 agable Capture ctrl-c in presubmit multiprocessing pool
2017-08-24 agable Don't send email from git-cl-description
2017-09-04 whesse Dart: Update the config for the "fetch dart" command.
2017-09-05 tikuta Add Nico and Scott to OWNERS for ninja related things
2017-09-02 dpranke Add a --batch mode to `git-cl owners`.
2017-09-04 tikuta Update ninja to v1.8.0.
2017-09-01 iannucci Update led tool to c9c1865b81113f02fd618259624170f59e2c832e.
2017-08-31 dnj [vpython] Update version.

Created with:
  roll-dep src/third_party/depot_tools
BUG=771277,769369,764087,741001,None,758725,741001


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

Change-Id: Icb9ca53560ec8ee470e60fea5ba7fc23a764fb34
Reviewed-on: https://chromium-review.googlesource.com/701321Reviewed-by: depot-tools-roller . <depot-tools-roller@chromium.org>
Commit-Queue: depot-tools-roller . <depot-tools-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506697}

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

The transition at the bottom starts when the toolbar is in the center of the
visible part of the sheet, and keeps it centered throughout. The top transition
is symmetric to that.

Videos: https://photos.app.goo.gl/psMFW5kRuo0HEfba2

Bug: 759130
Change-Id: I91f518371915ee97fb4b99840e74b7fe8f130a71
Reviewed-on: https://chromium-review.googlesource.com/702259Reviewed-by: Michael van Ouwerkerk <mvanouwerkerk@chromium.org>
Commit-Queue: Bernhard Bauer <bauerb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506695}

Record user actions in Clear Browsing Data to keep track of time period
changes, switching tabs and deleting from specific tabs.

Bug: 725401
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I1973216047177dfd8a677ccf80b0e8352ba2ecc3
Reviewed-on: https://chromium-review.googlesource.com/678955
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506694}

The Calibri Windows font has bitmap strikes for sizes 12, 13, 15, 16,
17, 19. When this font is drawn while using subpixel positioning, letter
spacing becomes wonky: Skia/DWrite are not using multiple subpixel
aliased glyph versions. Usually, with subpixel positioning and
antialiasing 3 or 4 versions of an antialiased glyph exist with
different shading and color filtering. Here, only one antialiased glyph
variant is used, which is placed at full pixel boundaries. For bitmap
strike font sizes Skia/DWrite now rounds the glyph positions within the
text to full pixel positions to place this one glyph variant. The result
is that the letter spacing becomes uneven, whith glyphs moving closer or
further away from each other when the horizontal subpixel offset is
changed [1].

As a mitigation of this problem, disable the usage of embedded bitmap
strikes for Calibri for now. We have not encountered any other non CJK
script fonts that would expose bitmap _and_ vector outline glyphs.

The setEmbeddedBitmapText flag of SkPaint mostly controls what to do
with fonts that have outlines and embedded bitmaps.

[1] https://bugs.chromium.org/p/chromium/issues/detail?id=707713#c5
[2] https://hg.mozilla.org/mozilla-central/rev/c93381b53df3

Bug: 707713
Change-Id: Iae81803ece1ada49e363ff46e0e06df3a26dbcde
Reviewed-on: https://chromium-review.googlesource.com/623672
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506693}

Callback function classes generated by callback_function.h.tmpl
(e.g. V8FrameRequestCallback) have some common objects and
each of classes doesn't have to have them.
CallbackFunctionBase is a base class for callback function classes and
has objects which used to be common for them.

Bug: 658073
Change-Id: I7a71cf9813b53edbd7b7141ca647508d88c85510
Reviewed-on: https://chromium-review.googlesource.com/697028
Commit-Queue: Sayaka Takayama <stakayama@google.com>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506692}

Bug: 754360
Change-Id: I8dd792787dcf5835eeb279f5ce535cb871f6d1e0
Reviewed-on: https://chromium-review.googlesource.com/660698Reviewed-by: Taiju Tsuiki <tzik@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506691}

Added exception when listener (second parameter in addEventListener)
is not an object to match the specification.

EX: document.getElementById("id").addEventListener('something', 1);

Bug: 712778
Change-Id: I2b1dca04f08ffe67c3a48283caf781ec8b373975
Reviewed-on: https://chromium-review.googlesource.com/672623Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com>
Cr-Commit-Position: refs/heads/master@{#506690}

Change-Id: I68fa3394a4d7ab62df652d8c06e3360618bfb3cc
Reviewed-on: https://chromium-review.googlesource.com/700594
Commit-Queue: Harald Alvestrand <hta@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506689}

Prevents network error screen shown by setting maximum offline timeout

Bug: 770738
Change-Id: I19ae30e1775e758df682a9bab3adb4b2c4037b96
Reviewed-on: https://chromium-review.googlesource.com/700639
Commit-Queue: Roman Sorokin <rsorokin@chromium.org>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506688}

WebCORSPreflightResultCache was originally a thread local thing,
but in the middle of refactoring, it was mistakenly changed
to be a per-process instance.

In terms of the spec, it's fine to be shared among threads, but
WebCORSPreflightResultCache internal implementation isn't
thread-safe to be so.

As a best effort plan, this patch makes it back to be a thread
local to work without any race.

Bug: 736308
Change-Id: I68e3243a9ef26da97f76a0c44785bc1394f5dc75
Reviewed-on: https://chromium-review.googlesource.com/697407Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506687}

There is a Function template in blink's wtf library and it is easy
to get mixed up in it if you try to use the symbol "Function"
elsewhere in blink.

In particular you can easily get an include of
platform/wtf/Functional.h (which exports WTF::Function to the global
scope) in jumbo builds:

Bug: 745732
Change-Id: Ic4f4a02e1aacc5819ce824b94e284bfc6caa964d
Reviewed-on: https://chromium-review.googlesource.com/702236Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#506686}

Before this patch, module scripts were fetched at Low priority, like
as classic async / deferred scripts. But this may not be a good default,
since module scripts tend to have many sub module scripts and fetching
them all at low priority may be too slow. (See the listed bug for more
on the discussion.)

This patch changes the fetch priority of module scripts to High (same
as classic scripts without async / defer). This applies to all module
fetches, includes dynamic import.

Bug: 769151
Change-Id: I24aa22a886e8394969447785eaf1fe0ab2eea818
Reviewed-on: https://chromium-review.googlesource.com/701937Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506685}

TBR=jochen@chromium.org

No-Try: True
Bug: 762486
Change-Id: I5634090ba14770a5f4b2fcfdfdfbc5c800ce6b5a
Reviewed-on: https://chromium-review.googlesource.com/702078Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506684}

Before this patch, "InsertText" command splits SPAN element containing TAB
character when inserting text into it.

For example:
Before: <span>&#9;a|c</span>
Command: insertText "B"
After: <span>&#9;a</span>B|<span>c</span>

This behavior is introduced by the patch[1], which replaces
"class=apple-tab-span" to "style=white-space:pre".

This patch changes |IsTabHTMLSpanElement()| to check "white-space:pre" CSS
property to make "InsertText" command not to split SPAN element containing
TAB character with "white-space: pre".


Note: "apple-tab-span" CSS class is MacOS specific feature[2]. On MacOS,
UITextView puts following HTML into pasteboard:

<style>.Apple-Tab-Span { white-space: pre; }</style>
<span class=Apple-Tab-Span>&#9;</span>

[1] http://crrev.com/2718543003 Remove EditingAppleTabSpan class handling
[2] https://www.cocoanetics.com/2013/06/apple-tab-span/

Bug: 741826
Change-Id: I42d8d388e306b613fbe8ffa614b9e974db254c22
Reviewed-on: https://chromium-review.googlesource.com/697044
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506683}

BUG=b:65560282
TEST=Ran trybot.

Change-Id: I8766ca2a207212375c6afa29fee2cbfd5ced48b6
Reviewed-on: https://chromium-review.googlesource.com/700237
Commit-Queue: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506682}

Bug: 763844
Change-Id: Icfaf76fd6be189bc16b0d428e4ddcd5060e28139
Reviewed-on: https://chromium-review.googlesource.com/700056Reviewed-by: Yuta Kitamura <yutak@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506681}

Intrinsic sizing for multicol used to be defined in the intrinsic and
extrinsic sizing CSS spec, but it must have been removed at some point.

All we have for inspiration now is the implementation in the legacy engine,
which landed with https://chromiumcodereview.appspot.com/23763002 back in 2013.
That, and common sense. :) The principles of min/max width calculation are,
after all, quite simple:

1. Figure out how much width we need to prevent avoidable overflow (min)
2. Figure out how much width we need to prevent avoidable soft wrapping (max)

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Id465449369d0227883e08228ce33346eb5b5cdba
Reviewed-on: https://chromium-review.googlesource.com/700460
Commit-Queue: Morten Stenshorne <mstensho@opera.com>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506680}

The profile_ member in TriggeredProfileResetter is not read by 
any code so let us remove it for platforms where it is not used.
This warning appeared in jumbo build
experiments (jumbo builds give the compiler more information which
triggers warnings that were not there before).

Bug: 770625
Change-Id: I4fd04d3147b01d0cf853b34c01ac7aa03fbd7d14
Reviewed-on: https://chromium-review.googlesource.com/695102
Commit-Queue: Daniel Bratell <bratell@opera.com>
Reviewed-by: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506679}

Recently, Mojo wire format change was introduced
for native structure.
https://chromium.googlesource.com/chromium/src/+/fd907634b5ea956b34bd1672c084089aad2150cd
It causes a protocol mismatch between ARC and Chrome.
Unfortunately, rolled libmojo is too old, so we cannot
use values.mojom now. As its workaround,
this CL stringifies the JSON value if necessary,
and use it for the communication.

Note that this is breaking change. Though, the protocol is already
broken, so we don't much care about it.

BUG=767313
TEST=Ran CTS.

Change-Id: Iaaf28065aa7cde0ce17d7231da0571312f23772d
Reviewed-on: https://chromium-review.googlesource.com/691495
Commit-Queue: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Kazuhiro Inaba <kinaba@chromium.org>
Reviewed-by: Mattias Nissler <mnissler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506678}

Currently the stylesheet_contents_fuzzer, which fuzzs CSS stylesheet
parsing, has no dictionary, which means most of the inputs are garbage
and will be ignored at the parsing level. This patch adds some common
strings seen in CSS stylesheets, which would hopefully produce more
valid tokens that can expose more bugs at the parsing level.

Bug: 752194
Change-Id: Ibf596a0a78c5b1bfb042f2c9e952ebdc07790a42
Reviewed-on: https://chromium-review.googlesource.com/696441Reviewed-by: meade_UTC10 <meade@chromium.org>
Reviewed-by: Max Moroz <mmoroz@chromium.org>
Reviewed-by: Oliver Chang <ochang@chromium.org>
Commit-Queue: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506677}

This CL moves ownership of ISensorManager to sensor thread, so that
ISensorManager is initialized when new sensor is created or in use,
and destroyed when there are no sensors. Before this CL, ISensorManager
could have been reset from IO thread.

Bug: 765527
Change-Id: I579f06dfd4cec7b678fe0080fe99cc6b182de27c
Reviewed-on: https://chromium-review.googlesource.com/699759Reviewed-by: Mikhail Pozdnyakov <mikhail.pozdnyakov@intel.com>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Alexander Shalamov <alexander.shalamov@intel.com>
Cr-Commit-Position: refs/heads/master@{#506676}

Bug: 700245
Change-Id: Id316cb680cecc5755ab52026ccff790e12d1d133
Reviewed-on: https://chromium-review.googlesource.com/702019Reviewed-by: oysteine <oysteine@chromium.org>
Commit-Queue: Siddhartha S <ssid@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506675}

When FetchEvent.respondWith() is called with a rejecting promise in a service
worker and if OnFetchEventFinished() of ServiceWorkerSubresourceLoader is called
before OnResponse(), OnFallback() passes |url_loader_client_| to the network
loader factory, and the renderer process crashes while calling
OnReceiveResponse().

  OnFetchEventFinished()
   => OnFallback()
     => std::move(url_loader_client_)

  OnResponse()
   => StartResponse()
    => DeliverErrorResponse()
     => CommitResponseHeaders()
      => url_loader_client_->OnReceiveResponse() (* crash here)


OnFetchEventFinished() should not call OnFallback() when |status| is
ERROR_EVENT_WAITUNTIL_REJECTED. And also, we don't need to fallback to the
network because it's for main resource load when FetchEvent failed.

Bug: 715640
Change-Id: I78d31c118ad9169740e3db4c561aa25e2dbae83e
Reviewed-on: https://chromium-review.googlesource.com/700276Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506674}

https://chromium.googlesource.com/catapult.git/+log/63a614b75273..f8ebe0eae6cb

$ git log 63a614b75..f8ebe0eae --date=short --no-merges --format='%ad %ae %s'
2017-10-03 benjhayden Refactor histogram-set-view API.

Created with:
  roll-dep src/third_party/catapult


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.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I06ff71f6464041fbbcdd9cd5d789c28501f8900b
Reviewed-on: https://chromium-review.googlesource.com/701665
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506673}

seek

Add UseCounter to check if setting playbackRate causes a compensatory
seek forcing a change in start_time_. This is as we want to spec this
behaviour on an API level.

Bug: 
Change-Id: Ic6ab06bb482b4e086d2559904bd4f6f4ccdd52eb
Reviewed-on: https://chromium-review.googlesource.com/700057
Commit-Queue: nainar <nainar@chromium.org>
Reviewed-by: dstockwell <dstockwell@chromium.org>
Reviewed-by: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506672}

Using wpt-import in Chromium 276f6dd9.

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

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

TBR=qyearsley@chromium.org

No-Export: true
Change-Id: Ic70765d2b8a02216acdbfa784e4342770ab7d4f7
Reviewed-on: https://chromium-review.googlesource.com/701800
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@{#506671}

For cleanup, this CL moves InstalledScriptsManager access from
WorkerThread::InitializeOnWorkerThread() to
ServiceWorkerGlobalScope::EvaluateClassicScript() because only ServiceWorker
needs to access the manager and placing it in EvaluateClassicScript() looks more
natural like importScripts().

<Motivation>

My final goal is to remove script evaluation code from WorkerThread and leave it
up to each owner of WorkerThread so that WorkerThead is no longer concerned with
how to evaluate scripts (i.e., "classic", "module", lazy eval for worklets).

For example, for DedicatedWorker, DedicatedWorkerMessagingProxy may post a task
to call WorkerGlobalScope::EvaluateClassicScript() or
WorkerGlobalScope::ImportModuleScript() (to be added later) based on
"WorkerType"[1] after initializing DedicatedWorkerThread.

[1] https://html.spec.whatwg.org/multipage/workers.html#workertype

Bug: 680046, 753350, 755054
Change-Id: I546cbda16f09da66c3f94c319e498055e1f9d05b
Reviewed-on: https://chromium-review.googlesource.com/701854
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506670}

This fixes a problem where navigating after tab reparenting could cause
WebappActivity's TabObserver to interact with destroyed activity and
crash Chrome due to NPE.

Bug: 771174
Change-Id: I0231ff3c50ebebd8341e5826e3fb6e1a12b69627
Reviewed-on: https://chromium-review.googlesource.com/702075Reviewed-by: Dominick Ng <dominickn@chromium.org>
Commit-Queue: Piotr Swigon <piotrs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506669}

Destroying PaintLayer would cause its CompositedLayerMapping and
composited layers to destroyed immediately. The layer destructor
automatically detach it from the layer tree, but auxiliary layer
pointers such as scroll/clip parent could dangle if compositing
update were omitted. This CL forces a one time compositing update
when a PaintLayer is removed.

BUG=769185

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I97946ac5047a5b1f1bc925a6c42509ed6f915655
Reviewed-on: https://chromium-review.googlesource.com/699192
Commit-Queue: Tien-Ren Chen <trchen@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506668}

Previously we would call SeekTask for every
completed read. This often cause us to seek
to somewhere and change our mind a very
short time later. By delaying our seeks and
batching them together, we can reduce the
number of times this happens.

I've tested this on a a bunch of badly muxed
videos, and generally got improvements. For
my primary test video, the number of connections
went from ~110 to ~70. As far as I can tell,
this has no impact on well-muxed videos.

Bug: 765313
Change-Id: I1cfdd803e9cda8fd45fae30ec2c0343f08a79edd
Reviewed-on: https://chromium-review.googlesource.com/690978
Commit-Queue: Fredrik Hubinette <hubbe@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506667}

I noticed this during my mash-related ImeController refactoring while
updating both locations.

Bug: 
Change-Id: I2fea8d237a325f64f619378b3db4776c4a47feb4
Reviewed-on: https://chromium-review.googlesource.com/700136
Commit-Queue: Blake O'Hare <blakeo@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506666}

GN should generate projects for Visual Studio 2017 by default since this
is the default compiler now.

Bug: 683729
Change-Id: I560983cf4c228a785e0ed2648567c7e0f07584d4
Reviewed-on: https://chromium-review.googlesource.com/700258Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Tomasz Moniuszko <tmoniuszko@opera.com>
Cr-Commit-Position: refs/heads/master@{#506665}

This feature is now ramping up to 100% in all channels. Flipping default to ensure it remains enabled in future clank releases.

A cleanup CL will follow up, to get rid of this flag entirely.

Bug: 718475
Change-Id: I96fbe835db9d6189cae557cf534217f12f5c45b8
Reviewed-on: https://chromium-review.googlesource.com/700004
Commit-Queue: Luis Barguñó <lbargu@google.com>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506664}

Change-Id: I639f16620a84a2b2aff827c38dd901bba73d37d0
Reviewed-on: https://chromium-review.googlesource.com/702095
Commit-Queue: Michael McGreevy <mcgreevy@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506663}

EncryptedMedia is more consistent with the rest of the code.

Bug: 689802
Change-Id: I990212b8e3972ee1c2581ed1237d15321628fab5
Reviewed-on: https://chromium-review.googlesource.com/596617Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Reviewed-by: Ian Clelland <iclelland@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: David Dorwin <ddorwin@chromium.org>
Commit-Queue: Raymes Khoury <raymes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506662}

https://skia.googlesource.com/skia.git/+log/e5ccb29e1eca..1ea81e766e1a

$ git log e5ccb29e1..1ea81e766 --date=short --no-merges --format='%ad %ae %s'
2017-10-05 angle-deps-roller Roll skia/third_party/externals/angle2/ c1abf9178..ce8602ab7 (1 commit)
2017-10-05 mtklein Revert "Don't execute onFlush op lists until after GPU data is uploaded"

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

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

Comes with a test that catches the bug.

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: Ib2e7434108d0e9825c812e37d5dd2b51f55f57ac
Reviewed-on: https://chromium-review.googlesource.com/702094Reviewed-by: Frank Liberato <liberato@chromium.org>
Commit-Queue: Chris Watkins <watk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506660}