GitLab

Bug: 776826
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I4c94e829690b37da7029fdf1105731fbf6c2b0b6
Reviewed-on: https://chromium-review.googlesource.com/809607Reviewed-by: Justin Novosad <junov@chromium.org>
Commit-Queue: Olivia Lai <xlai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521892}

The feature has been fully rolled out to stable for more than a month.
We should now be able to start doing feature cleanups.

Bug: 718154
Change-Id: I874c5517003badc37c8fc0e29ca6e96c9a4eaae1
Reviewed-on: https://chromium-review.googlesource.com/794715
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521891}

The proxying for __NR_readlink is successful under test, but
the real-life processes need permission to use it.

Change-Id: Ia02b80f5190c2b48458ad2f48693f9bc093464a2
Reviewed-on: https://chromium-review.googlesource.com/809705Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521890}

This patch checks that |sensor_provider_| has not been reset before
attempting to fetch a fallback sensor after an initial initialization
error. This could happen if the SensorProvider interface is unavailable.

Bug: 788120
Change-Id: I5afd03929e83ae4a35353dcc21f0c48ccbdb0066
Reviewed-on: https://chromium-review.googlesource.com/794519Reviewed-by: Tim Volodine <timvolodine@chromium.org>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521889}

Due to some bug within the translation pipeline, translated grd files
sometimes end up with multiple plural entries that have the same/similar
key, eg: '=1' and 'one'. This change makes sure that the xml output only
has one entry in per key. This is required by aapt2, which fails with an error
otherwise.

Bug: 787488
Change-Id: I6d98c1739a7dbd3872c31cbfca63b0bd5612c79d
Reviewed-on: https://chromium-review.googlesource.com/807148
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521888}

Re-enables VrShellControllerInputTest#testControllerScrolling which was
disabled due to a flake. Was unable to reproduce the flake after
running the test on the swarmed devices with the same gn args 500 times.

Bug: 786200
Change-Id: Iaf8e362381dd2da60ac2e155de99af0d2535e0b4
Reviewed-on: https://chromium-review.googlesource.com/809884Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521887}

Using wpt-import in Chromium 37c44039.

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

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

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

Also moves the tab_stats_data_store tests to unit_tests as they're not
browser tests.

I'm adding some new stats to this class and having the
browser/unit_tests separation will ease the unittesting.

Bug: 720131
Change-Id: I9a2a9c06e0968ef9b537e2eb119934590038d3f0
Reviewed-on: https://chromium-review.googlesource.com/804435
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521885}

Read the new external policy, enforce the selected policies, provide
printers to clients.  Code path is not connected yet.

Bug: 758680
Change-Id: I178d0afd1e0e60d5c0cdaa8adfebb14d1d5d7070
Reviewed-on: https://chromium-review.googlesource.com/677865Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Sean Kau <skau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521884}

Preparation for removing ScriptSourceCode::GetResource().

Bug: 788828
Change-Id: Ic1196997f2b7a0dd6f7febca476d45c4f4128598
Reviewed-on: https://chromium-review.googlesource.com/786891
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521883}

This reverts commit 363560ef.

Reason for revert: The revert broke compile. Will reland and disable the failing test instead.

Original change's description:
> Revert "Run constraints processing for screen capture on the main thread."
> 
> This reverts commit 112c4ff7.
> 
> Reason for revert: This might have broken browser_tests on Mac ASan 64 Tests (1): https://uberchromegw.corp.google.com/i/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/36626
> 
> Original change's description:
> > Run constraints processing for screen capture on the main thread.
> > 
> > Some code paths require access to Blink strings that can only be
> >  accessed on the main thread.
> > A test that triggers the bug is coming up on crrev.com/c/738040.
> > 
> > Bug: 791992
> > Change-Id: I55ae617ae623766d4bd975a19070bde410d8af1d
> > Reviewed-on: https://chromium-review.googlesource.com/809005
> > Reviewed-by: Henrik Boström <hbos@chromium.org>
> > Commit-Queue: Guido Urdaneta <guidou@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#521708}
> 
> TBR=hbos@chromium.org,guidou@chromium.org
> 
> Change-Id: Ibe39bacdedfbf54fac48a6469ec59c4f32381fd1
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 791992
> Reviewed-on: https://chromium-review.googlesource.com/809240
> Reviewed-by: Derek Cheng <imcheng@chromium.org>
> Commit-Queue: Derek Cheng <imcheng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521872}

TBR=hbos@chromium.org,imcheng@chromium.org,guidou@chromium.org

Change-Id: Ia5cf4ca4e14d3f83aa93c6e9aad71c3348a2467d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 791992
Reviewed-on: https://chromium-review.googlesource.com/809887Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521882}

Sample extension that uses command API to give user alternate
choice of keyboard shortcuts to switch between tabs.
Also moves existing sample command extension into a new
directory.

Bug: None
Change-Id: Ia5c4ae7134c51a1730260a5f23f23c091231ff38
Reviewed-on: https://chromium-review.googlesource.com/749115
Commit-Queue: Crystal Lambert <crystallambert@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521881}

Originally, it was only used when Print Preview is enabled, but the
original user no longer exists, and the only user is the Settings page.
Transitively only build settings::PrintingHandler when printing is
enabled.

BUG=789790

Change-Id: I8e8b9d0df66b1a5ead01abeeaad78982f16e61ec
Reviewed-on: https://chromium-review.googlesource.com/806447Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521880}

Currently we've got some bugs that can lead to crashes on standalone
VR headsets (https://vr.google.com/daydream/standalonevr/), so we
should detect devices that boot into VR, and disable VR there.

This will make sure we just use the fallback 2D-in-VR path that is
supposed to work for all apps :)

This also fixes a bug in scheduling rAF when presentation is rejected
that lead to the rAF loop doubling up every time presentation was
rejected.

Bug: 791090
Change-Id: I7511299935486dd018b558f3489ddd00f314ba8e
Reviewed-on: https://chromium-review.googlesource.com/809194
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521879}

https://skia.googlesource.com/skia.git/+log/06ab3836f779..a3e9271ec41d

$ git log 06ab3836f..a3e9271ec --date=short --no-merges --format='%ad %ae %s'
2017-12-04 csmartdalton CCPR: Eliminate texel buffers
2017-12-05 reed add serial procs to pictures
2017-12-05 scroggo Reland "Hide SkEncodedInfo"
2017-12-05 mtklein ANGLE -> Clang
2017-12-05 csmartdalton Add analytic clip FPs that read from the CCPR atlas
2017-12-05 mtklein remove mips support and bots
2017-12-05 bsalomon Avoid GrEllipseEffect for small radii on devices without 32 bit float.
2017-12-05 brianosman Keep animation running while moving the mouse in viewer (on Windows)
2017-12-05 scroggo Revert "Hide SkEncodedInfo"
2017-12-05 halcanary android_build_app: fixes
2017-12-04 brianosman Fix sample modifier keys for mouse events
2017-12-04 scroggo Hide SkEncodedInfo

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

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

This changes the WPT import script to use the URL
https://build.chromium.org/deprecated/chromium/all_rotations.js
for fetching sheriff rotations.

Bug: 792193
Change-Id: I64bc3396bc921cc7f1e0420f872881fb2b25ebac
Reviewed-on: https://chromium-review.googlesource.com/809616Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521877}

WebStateImpl will cache favicon urls in OnFaviconUrlUpdated and will
call WebStateObserver::FaviconUrlUpdated in OnNavigationFinished for
same document navigations.

Also removed favicon urls caching in WebFaviconDriver introduced in
crrev.com/c/695761. This CL is not exactly a revert of
crrev.com/c/695761, because deprecated NavigationItemCommitted is
replaced with DidFinishNavigation. NavigationItemCommitted is not
called for push/replace state same-document navigaiton, which may
cause other bugs for favicons caching.


Bug: 789581
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I42922ab4d7812380787e350713640efe4d599680
Reviewed-on: https://chromium-review.googlesource.com/797072Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521876}

https://chromium.googlesource.com/catapult.git/+log/37c17ee783f3..09fc536c66c6

$ git log 37c17ee78..09fc536c6 --date=short --no-merges --format='%ad %ae %s'
2017-12-04 rnephew [Telemetry/TADA] Give StoryExpectations the ability to decode TADA expectation files
2017-12-05 simonhatch Pinpoint - Grab histogram summary metric equivalent.
2017-12-05 rnephew [Telemetry] Add expectations path to telemetry config object.
2017-12-05 rnephew [Telemetry/TADA] Fix bug where story name cannot be url.

Created with:
  roll-dep src/third_party/catapult


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

Make test and existing code changes as needed to incorporate native
layer into the new UI. No behavior changes.

Bug: 773928
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I660a4707a2a54d75e5ca7dde2b9ec10496c5055b
Reviewed-on: https://chromium-review.googlesource.com/807585Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521874}

Change-Id: Ib087b7ea70bae6bad8c7512cdc7f6c5d84b58661
Reviewed-on: https://chromium-review.googlesource.com/809292Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521873}

This reverts commit 112c4ff7.

Reason for revert: This might have broken browser_tests on Mac ASan 64 Tests (1): https://uberchromegw.corp.google.com/i/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/36626

Original change's description:
> Run constraints processing for screen capture on the main thread.
> 
> Some code paths require access to Blink strings that can only be
>  accessed on the main thread.
> A test that triggers the bug is coming up on crrev.com/c/738040.
> 
> Bug: 791992
> Change-Id: I55ae617ae623766d4bd975a19070bde410d8af1d
> Reviewed-on: https://chromium-review.googlesource.com/809005
> Reviewed-by: Henrik Boström <hbos@chromium.org>
> Commit-Queue: Guido Urdaneta <guidou@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521708}

TBR=hbos@chromium.org,guidou@chromium.org

Change-Id: Ibe39bacdedfbf54fac48a6469ec59c4f32381fd1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 791992
Reviewed-on: https://chromium-review.googlesource.com/809240Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521872}

The ui service DCHECKs if the root_location and location don't
match. This is always the case for events coming in from the platform,
but RemoteEventInjector circumvents that and could supply different
locations. This makes EventInjector set the locations to
match. Non-mus path does this by way of
LocatedEvent::UpdateForRootTransform.

BUG=786453
TEST=none

Change-Id: If70378c18d354b29290d0545cfa6bf2f546a60f7
Reviewed-on: https://chromium-review.googlesource.com/809610Reviewed-by: Elliot Glaysher <erg@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521871}

In this configuration, StackTrace may produce an empty result,
which causes the is_set() assertion to fail.

Bug: 791223, 598854
Change-Id: I4c4cabf2eb1176def431cd5798a2c80808bf8074
Reviewed-on: https://chromium-review.googlesource.com/809102Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521870}

Rendering console logs with lots of text can stall DevTools. In this CL,
very long text will get truncated. A 'Show N more' button reveals the
full text and a 'Copy' button copies the full text to clipboard.

Screenshot: https://imgur.com/a/0dHJI

Bug: 658525
Change-Id: Ic27e4836b8aaf9069f7ff0643b39ded4a83ce91f
Reviewed-on: https://chromium-review.googlesource.com/597012
Commit-Queue: Erik Luo <luoe@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521869}

This change makes it possible to decouple the actions of capturing an
animation frame from a canvas and sending it to the compositor.
This is a prerequisite for mergin OffscreenCanvasResourceProdiver
with CanvasResourceProvider.

Bug: 788439, 776801
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Iaf4563069ede53b1bba8bde5ea8b693d4c1a2b73
Reviewed-on: https://chromium-review.googlesource.com/803945
Commit-Queue: Justin Novosad <junov@chromium.org>
Reviewed-by: Olivia Lai <xlai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521868}

Replace "WebApk.Permission.ChromePermissionDenied" UMA by "WebApk.Permission.ChromePermissionDenied2"

It previously logs all Chrome permission denied. Fix to only include WebApk.

Bug: 780934
Change-Id: I32e2d9009423365e1f10ae68dc53ff870e52ea06
Reviewed-on: https://chromium-review.googlesource.com/797431Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Xi Han <hanxi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521867}

AudioContextTest.cpp tests the audio latency and is not involved
any AudioWorklet operation. Removing AudioWorkletThread addresses
the data race found in the bug.

Locally confirmed that TSAN with this patch does not crash with
the data race any more.

Bug: 792140
Change-Id: I297533a176c42fb1af5025fdcd77a1cf4e68ee56
Reviewed-on: https://chromium-review.googlesource.com/809825Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Raymond Toy <rtoy@chromium.org>
Commit-Queue: Hongchan Choi <hongchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521866}

On low-end Android devices in-proc GPU is started on demand, when
CompositorImpl::SetVisible(true) is called for the first time.

Due to the fact that native / Java messages on Android are not strictly
ordered (see crbug.com/788008) sometimes GPU startup is delayed for too
long, stalling the renderer (see the bug).

We already have code to start GPU early (from BrowserThreadsStarted),
but that code includes explicit !UsingInProcessGpu() check.

However, the check (which was there from 2012), seems obsolete now. As
sievers@ and boliu@ discussed in crbug.com/450396#c4 the check was there
to avoid a race setting ChildProcess::current, but became obsolete once
ChildProcess::current switched to TLS.

This CL removes !UsingInProcessGpu() check, causing in-proc GPU to start
early.

Bug: 787647
Change-Id: Ifb6764e8b479003412674ce667dd6c3ea08dcb43
Reviewed-on: https://chromium-review.googlesource.com/804144Reviewed-by: Antoine Labour <piman@chromium.org>
Commit-Queue: Dmitry Skiba <dskiba@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521865}

This is a reland of e62d46f6

The original change was reverted due to failures in the new unit tests, which assume
box fragment creation for inline elements <b>, <i> or <u>, which however is not true.

This reland ensures box fragment creation by adding border to those elements.

Original change's description:
> [LayoutNG] Introduce NGInlineFragmentTraversal in place of NGInlineFragmentIterator
>
> This patch introduces a utility class NGInlineFragmentTraversal, so that we can
> conveniently retrieve:
> - ancestors of a given physical fragment in a container
> - descendants of a given physical fragment
> - physical fragments in a container with the given layout object
>
> The utility class supersedes the existing NGInlineFragmentIterator class. Follow
> up patches will convert clients of NGInlineFragmentIterator to use the traversal
> class instead.
>
> Bug: 636993
> Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
> Change-Id: I7f3f553df18291df50f60075ab84294dd394d86c
> Reviewed-on: https://chromium-review.googlesource.com/807758
> Reviewed-by: Koji Ishii <kojii@chromium.org>
> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521758}

Tbr: eae@chromium.org, kojii@chromium.org, yoichio@chromium.org, yosin@chromium.org
Bug: 636993
Change-Id: I616a89ac0ca1360681e47bbc45317ffca5d99d90
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Reviewed-on: https://chromium-review.googlesource.com/809302Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521864}

This patch makes that when you select empty lines
on a contenteditable element, the selection is painted
similar to what happens on a textarea.

There are 2 changes to make this happen.
On one side, newlines are no longer marked as
InlineBox::KnownToHaveNoOverflow().
On the other side, InlineFlowBox::ComputeOverflow() has been modified
in order to extend the overflow rect for newlines.

With this the behavior of selection in contenteditable elements
and textareas is more consistent.

BUG=789508

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I55f33e8c688a92e760e3d359c2cf54deacd005af
Reviewed-on: https://chromium-review.googlesource.com/793036Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: yosin (OOO Dec 11 to Jan 8) <yosin@chromium.org>
Commit-Queue: Manuel Rego Casasnovas <rego@igalia.com>
Cr-Commit-Position: refs/heads/master@{#521863}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/b88605f7..7af7216b

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: If31e6aa7c3984e6be8324f3101055e00206f1958
Reviewed-on: https://chromium-review.googlesource.com/809193Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521862}

Records infobar displayed when a survey is accepted.
Calls closeInfoBar instead of onCloseButtonClicked
 when accepting a survey to prevent logging the new action.

Bug: 791752
Change-Id: I488720882832ee20946cc1a56937d234c4cce620
Reviewed-on: https://chromium-review.googlesource.com/806367Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Daniel Park <danielpark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521861}

Update the New Tab Bubble Promo String to "Add tabs and switch between
pages".

Bug: 779656
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ia42386ecc45e45ccf93105d48ff5bdd898cf4058
Reviewed-on: https://chromium-review.googlesource.com/809844Reviewed-by: edchin <edchin@chromium.org>
Commit-Queue: Gregory Chatzinoff <gchatz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521860}

Revert "Reland "Enable video capture service by default on platforms where it is rolling out with M61.""

This reverts commit 0fdb7a80.

Reason for revert: broke webkit_layout_tests: https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak/builds/12888

Original change's description:
> Reland "Enable video capture service by default on platforms where it is rolling out with M61."
> 
> This is a reland of 1e2e99ef
> 
> The reason for the revert was a Webkit layout test failing with this CL. With
> https://chromium-review.googlesource.com/c/chromium/src/+/806442 landed to fix that test,
> this CL should now be able to reland without modification.
> 
> Original change's description:
> > Enable video capture service by default on platforms where it is rolling
> > out with M61.
> >
> > Bug: 721812
> > Change-Id: I1de0f625a1fb106e2dea664675e94267e66fc2fd
> > Reviewed-on: https://chromium-review.googlesource.com/719416
> > Commit-Queue: Christian Fremerey <chfremer@chromium.org>
> > Reviewed-by: Emircan Uysaler <emircan@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#517761}
> 
> TBR=emircan@chromium.org
> 
> Bug: 721812
> Change-Id: If69cdb660b0b97a28084b6bd0610234d5abb232f
> Reviewed-on: https://chromium-review.googlesource.com/809530
> Reviewed-by: Christian Fremerey <chfremer@chromium.org>
> Commit-Queue: Christian Fremerey <chfremer@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521800}

TBR=emircan@chromium.org,chfremer@chromium.org

Change-Id: Iee20ddb4dabf156cd3709151792cc374b0b50194
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 721812
Reviewed-on: https://chromium-review.googlesource.com/809770Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521859}

This is a reland of 52dda118
Original change's description:
> [Media Router] Better sink availability tracking
> 
> - When sink availability changes for an MRP, start/stop observing sinks only for that MRP
> - Make WiredDisplayMRP report sink availability updates
> 
> Bug: 789277,777650
> Change-Id: I3d1a087ca39a4830403d6d2f0f00bf1411388f33
> Reviewed-on: https://chromium-review.googlesource.com/794106
> Reviewed-by: Derek Cheng <imcheng@chromium.org>
> Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521550}

Bug: 789277, 777650
Change-Id: I0a2edc4af121fa675548f9d545bb6048b05e2f13
Reviewed-on: https://chromium-review.googlesource.com/809333Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Takumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521858}

This CL adds support to filter code coverage results by list of
directories or files so that the script can run faster without having
to generate a report for every executed file.

Bug: 789691
Change-Id: I2e954872b7f1113cd04b8e609ffdabf0578db762
Reviewed-on: https://chromium-review.googlesource.com/809295
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Abhishek Arya <inferno@chromium.org>
Reviewed-by: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521857}

Previously the targets to be compiled (but not run) were specified
explicitly, effectively duplicating the logic already present in the GN
build configuration files.

Bug: 707031
Change-Id: I8c6715f9f862a04a49c81030ba9d0212f0468ec9
Reviewed-on: https://chromium-review.googlesource.com/802189
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521856}

This is more consistent.

Bug: 784495
Change-Id: I39bdff1cf126d6d3abf245620ce3c803d382db34
Reviewed-on: https://chromium-review.googlesource.com/806644
Commit-Queue: Jacob Dufault <jdufault@chromium.org>
Reviewed-by: Achuith Bhandarkar <achuith@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521855}

This change requests focusing of the first item after selecting
an item. Generally that should focus the newly connecting item as
soon as the list is updated.

Bug: 739633
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I728658d1bbc916c56793960c576e47fe5bc91354
Reviewed-on: https://chromium-review.googlesource.com/806440Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521854}

Bug: 754889
Change-Id: I60bc810728bed4759dac3a406f0eae36a1e11820
Reviewed-on: https://chromium-review.googlesource.com/794491Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Commit-Queue: Steven Holte <holte@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521853}