GitLab

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/a7db6458..3d16d190

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_precise_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=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

Review-Url: https://codereview.chromium.org/2295293002
Cr-Commit-Position: refs/heads/master@{#415739}

This is refactoring to allow MusBrowserCompositorOutputSurface to
provide an ExternalBeginFrameSource that it owns rather than
passing ownership to Display.

This should also make it possible to remove begin frame sources from
OutputSurface in the future after the OutputSurface/CompositorFrameSink
split.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Review-Url: https://codereview.chromium.org/2286273003
Cr-Commit-Position: refs/heads/master@{#415738}

Add steel@ and xiyuan@ as sockets owners.

BUG=None

Review-Url: https://codereview.chromium.org/2296193004
Cr-Commit-Position: refs/heads/master@{#415737}

Makes them owned by the same people that own other Memory.*
histograms.

Also, adds a summary for a TBD histogram and adds units to
process count histograms.

BUG=642626

Review-Url: https://codereview.chromium.org/2300643003
Cr-Commit-Position: refs/heads/master@{#415736}

Revert of Increase maximum size of the HPACK decoder dynamic table to 64 kB. (patchset #1 id:1 of https://codereview.chromium.org/2300683002/ )

Reason for revert:
Suspected patch causing Android Cronet Builder and other bots to fail BidirectionalStreamTest#testSimpleGet. Will re-land if not.

Original issue's description:
> Increase maximum size of the HPACK decoder dynamic table to 64 kB.
>
> * Send out SETTINGS_HEADER_TABLE_SIZE = 64 kB in the initial SETTINGS frame on
>   each HTTP/2 connection.
> * Immediately notify HpackDecoder about the change so that it allows the encoder
>   to update the dynamic table size up to this limit.  It is safe to do so before
>   receiving the SETTINGS ACK, because the new limit is larger than the default 4
>   kB.  In fact, a server following RFC 7540 Section 6.5.3 word-by-word might
>   already use the larger limit before sending an ACK.
>
> BUG=642784
>
> Committed: https://crrev.com/30537e17941018ecffe781c009b39c7a8c4c8f30
> Cr-Commit-Position: refs/heads/master@{#415718}

TBR=rch@chromium.org,bnc@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=642784

Review-Url: https://codereview.chromium.org/2302573002
Cr-Commit-Position: refs/heads/master@{#415735}

This CL:
1) Adds MdTabStrip and MdTab as subclasses of TabStrip and Tab;
2) Uses those classes to implement the Harmony visuals.

BUG=635176

Review-Url: https://codereview.chromium.org/2297193002
Cr-Commit-Position: refs/heads/master@{#415734}

Code leftover from the pre-OOPIF days was causing us to exit early from
the RenderAccessibilityImpl constructor for some child frames that were
already loaded. Everything worked fine if accessibility was already enabled
when loading the frame, but if the frame was already loaded and then
accessibility was enabled, this could cause it to fail to create an
accessibility tree.

The code in RenderAccessibilityImpl is no longer needed because now we
have exactly one accessibility tree per frame.

This wasn't caught by tests because we didn't cover both scenarios, we
always enabled accessibility first.

Added two variants of existing tests that load the page first and then
enable accessibility.

BUG=640231

Review-Url: https://codereview.chromium.org/2299673002
Cr-Commit-Position: refs/heads/master@{#415733}

https://pdfium.googlesource.com/pdfium.git/+log/4d9df422c340..fc9b11358a47

$ git log 4d9df422c..fc9b11358 --date=short --no-merges --format='%ad %ae %s'
2016-08-31 npm Move maxNumDataBytes and maxNumEcBytes calculation to fix crash
2016-08-31 tsepez Make CPDF_ColorState have a CFX_ColorStateData rather than inheriting

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2298603005
Cr-Commit-Position: refs/heads/master@{#415732}

- Remove unused function GetStartupTemperature()
- Move GetHardFaultCountForCurrentProcess() to the anonymous namespace.

This cleanup is a first step toward refactoring startup_metric_utils.

BUG=580207

Review-Url: https://codereview.chromium.org/2283073002
Cr-Commit-Position: refs/heads/master@{#415731}

BUG=642599

Review-Url: https://codereview.chromium.org/2296903003
Cr-Commit-Position: refs/heads/master@{#415730}

BUG=none
TEST=unit tests

Review-Url: https://codereview.chromium.org/2295983002
Cr-Commit-Position: refs/heads/master@{#415729}

This replaces the 1D sliders for X offset and Y offset.

BUG=639095

Review-Url: https://codereview.chromium.org/2292583002
Cr-Commit-Position: refs/heads/master@{#415728}

BUG=642227
TEST=gpu_unittests
R=qiankun.miao@intel.com
CQ_INCLUDE_TRYBOTS=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

Review-Url: https://codereview.chromium.org/2297663005
Cr-Commit-Position: refs/heads/master@{#415727}

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/b14b6aaa26cda052c59eb2908943353e24213bb0 Roll recipe dependencies (nontrivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/6911b067504584996204cf0b2535508ac4207106 Make rebase and sync the default for Gerrit updates. (andybons@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=602906

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2299523005
Cr-Commit-Position: refs/heads/master@{#415726}

First attempt:
https://codereview.chromium.org/2243343004/

Reason for reland:
Now ignores placeholder .so files

The goal is to have a single script that reports all .apk
size related metrics.

* Uses method_count.py to get dex info.
* Already had static initializer logic, but now shows it even without
--so_with_symbols_path.
* Tweaks the static initializer count logic to sum up rather for all
.so files rather than report the same metric repeatedly.

TBR=rnephew@chromium.org,amineer@chromium.org
BUG=609365

Review-Url: https://codereview.chromium.org/2294093006
Cr-Commit-Position: refs/heads/master@{#415725}

context used to be used by PrivacyPreferencesManager.getInstance (removed in
https://codereview.chromium.org/2105183002) method called
from FirstRunGlueImpl.acceptTermsOfService (removed in
https://codereview.chromium.org/2248243002). Both of these
is not true anymore so there is no need passing context
around.

Review-Url: https://codereview.chromium.org/2298243003
Cr-Commit-Position: refs/heads/master@{#415724}

Since there does not appear to be an overlapping way to specify an empty
'backup' string, use .bak and remove the file afterwards in accordance
with the wisdom of stackoverflow.

Review-Url: https://codereview.chromium.org/2295903003
Cr-Commit-Position: refs/heads/master@{#415723}

This is done in preparation for mojoifying this enum; mojo only generates 'enum class' enums.

TBR=jam@chromium.org
BUG=627852
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2248873002
Cr-Commit-Position: refs/heads/master@{#415722}

We need to set the capacity, not the size, to children_.size() because
otherwise the following append() calls will not do the right thing.

R=eae@chromium.org, ikilpatrick@chromium.org, glebl@chromium.org
BUG=635619

Review URL: https://codereview.chromium.org/2295233002 .

Cr-Commit-Position: refs/heads/master@{#415721}

Add setup for new test slave "Android VR Tests'.

Corresponding buildbot/infra related changes can be reviewed at https://codereview.chromium.org/2293803004

BUG=

Review-Url: https://codereview.chromium.org/2255873002
Cr-Commit-Position: refs/heads/master@{#415720}

These flags are on-by-default and no longer passed in as of
https://codereview.chromium.org/2292173002

Review-Url: https://codereview.chromium.org/2270433002
Cr-Commit-Position: refs/heads/master@{#415719}

* Send out SETTINGS_HEADER_TABLE_SIZE = 64 kB in the initial SETTINGS frame on
  each HTTP/2 connection.
* Immediately notify HpackDecoder about the change so that it allows the encoder
  to update the dynamic table size up to this limit.  It is safe to do so before
  receiving the SETTINGS ACK, because the new limit is larger than the default 4
  kB.  In fact, a server following RFC 7540 Section 6.5.3 word-by-word might
  already use the larger limit before sending an ACK.

BUG=642784

Review-Url: https://codereview.chromium.org/2300683002
Cr-Commit-Position: refs/heads/master@{#415718}

Take copies, not referrences, when passing data to another thread in the
WatchingParams.

BUG=none
TEST=unit tests

Review-Url: https://codereview.chromium.org/2289333002
Cr-Commit-Position: refs/heads/master@{#415717}

When a StartupTimeBomb instance deletes itself, it may schedule a future
task to complete the deletion, and this future task must not reference
the destructed instance.

BUG=none
TEST=unit tests

Review-Url: https://codereview.chromium.org/2296793003
Cr-Commit-Position: refs/heads/master@{#415716}

This change introduces a new authenticator class which handles two scenarios:
- Connection policy validation (i.e. do the details of the connection match the
  current policies set on the machine)
- Interactive connection validation (i.e. prompt the user to allow the
  connection to be established)

The first scenario is typically synchronous but the second is async, therefore
this validation mechanism must handle both.  The new authenticator class wraps
another, functional authenticator to provide a level of validation before
allowing the wrapped authenticator to take over.

BUG=617185

Review-Url: https://codereview.chromium.org/2277553002
Cr-Commit-Position: refs/heads/master@{#415715}

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/7b849c1efa09d26a9131400261e9ec25ea42c028 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/9c526af4f649b33e3a1351dd9c9bbe6fdb7122da lightweight builds archiving for mac and win64. (miimnk@google.com)
depot_tools:
  https://crrev.com/f3458f046e0b77a71422c5664c6a00be17845c5d Roll recipe dependencies (MANUAL). (iannucci@chromium.org)
recipe_engine:
  https://crrev.com/09345f9dcf4d2744f66d64766fd349ddf04f04ce Cancel PYTHONUNBUFFERED when unbuffered is explicitly set to False (iannucci@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=604452,635641

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2298163003
Cr-Commit-Position: refs/heads/master@{#415714}

The documentation comment for FontConfigIPC::makeTypeface is now
somewhat confusing because it is about the previous signature of this
method. Previously this method returned SkTypeface* and the ownership of
the reference was ambiguous in the code but specified by the
documentation comment. Now the code specifies the ownership semantics
explicitly by returning a smart pointer, so the comment is no longer
needed.

Review-Url: https://codereview.chromium.org/2295273002
Cr-Commit-Position: refs/heads/master@{#415713}

Revert of Fragment blocks with non-visible overflow as normally when printing. (patchset #1 id:1 of https://codereview.chromium.org/2298193002/ )

Reason for revert:
Suspected cause of "virtual/threaded/fast/scroll-behavior/smooth-scroll/main-thread-scrolling-reason-correctness.html" failures in webkit_tests on "WebKit Linux (dbg)" bot.

example: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20%28dbg%29/builds/9099

Original issue's description:
> Fragment blocks with non-visible overflow as normally when printing.
>
> Splitting scrollbars into multiple fragmentainers is only problematic in
> interactive media. We don't need to impose any such pagination restrictions
> when printing, since printing is non-interactive,
>
> BUG=641983
>
> Committed: https://crrev.com/53d4d7aff8e25eae2ece5bbf5a53eec940871fa8
> Cr-Commit-Position: refs/heads/master@{#415645}

TBR=eae@chromium.org,mstensho@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=641983

Review-Url: https://codereview.chromium.org/2300763002
Cr-Commit-Position: refs/heads/master@{#415712}

It tests whether ResizeObserver is kept alive by:
- direct javascript reference
- observing a live element.

BUG=612962

Review-Url: https://codereview.chromium.org/2272163003
Cr-Commit-Position: refs/heads/master@{#415711}

BUG=

Review-Url: https://codereview.chromium.org/2298303002
Cr-Commit-Position: refs/heads/master@{#415710}

Review-Url: https://codereview.chromium.org/2297273002
Cr-Commit-Position: refs/heads/master@{#415709}

Forward ScrollEvent/MouseWheelEvent to the hosting widget so that
it is processed and reaches the containing ScrollView.

Also not forward touch events to ArcCustomNotificationView since
a View is not supposed to receive touch events. This gets rid of
the NOTREACHED() warning in chrome log.

BUG=b/31115616
BUG=642501

Review-Url: https://codereview.chromium.org/2291193003
Cr-Commit-Position: refs/heads/master@{#415708}

https://pdfium.googlesource.com/pdfium.git/+log/21ce1a68bb9f..4d9df422c340

$ git log 21ce1a68b..4d9df422c --date=short --no-merges --format='%ad %ae %s'
2016-08-31 tsepez Make CPDF_GraphState have a CPDF_GraphStateData instead of inheriting.

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2291343003
Cr-Commit-Position: refs/heads/master@{#415707}

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/dcb82a6eedd3de602c008dca3d69489df036cbc7 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/7b787b8642623e39eafce83595c187660528af9f Revert of It's time to bring subprocess42 to depot_tools. (patchset #1 id:1 of https://codereview.chromium.org/2291223003/ ) (tandrii@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=635641

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2298923002
Cr-Commit-Position: refs/heads/master@{#415706}

BUG=614963

Review-Url: https://codereview.chromium.org/2280283002
Cr-Commit-Position: refs/heads/master@{#415705}

This move allows the BezierEditor to also be used in Sources panel.

BUG=642782

Review-Url: https://codereview.chromium.org/2299473002
Cr-Commit-Position: refs/heads/master@{#415704}

It was never added to histograms.xml

BUG=642287

Review-Url: https://codereview.chromium.org/2298513002
Cr-Commit-Position: refs/heads/master@{#415703}

BUG=639315
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Review-Url: https://codereview.chromium.org/2272993003
Cr-Commit-Position: refs/heads/master@{#415702}

These tests assume that no extension background pages have loaded at the start
of the test, but for some reason DevTools is reporting these earlier than it
used to. It's valid to have these background pages in the window list, so I've
removed the assertions.

BUG=
TBR=stgao@chromium.org

Review-Url: https://codereview.chromium.org/2299723002
Cr-Commit-Position: refs/heads/master@{#415701}

In the current code base, a float number is used when users try to control
the volume slider by using keyboard. It increases/decreases the volume by
0.1 (or 10%) after each press. In the CL2281173002, it accidentally
changed this increment to an integer, so the volume either jumps to silent
or max.

BUG=631775

Review-Url: https://codereview.chromium.org/2297833002
Cr-Commit-Position: refs/heads/master@{#415700}