GitLab

Screenshots: https://imgur.com/a/EwqTzly

R=hcarmona@chromium.org
BUG=936524

Change-Id: I5ed3a43dc4273c1179eddca56a8138cf394c10a6
Reviewed-on: https://chromium-review.googlesource.com/c/1496246Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Dan Beam <dbeam@chromium.org>
Auto-Submit: Dan Beam <dbeam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636676}

Currently, starting up crostini apps while switching profiles causes
two issues. First, windows other then the terminal are owned by
whichever profile is active when they become visible, not the profile
that owns crostini. Second, if a window becomes visible while it's
owning profile is inactive, then its icon will still be added to the
shelf. On switching to the correct profile, the icon is added
again. If the app is then closed, only one of these shelf entries is
removed, causing crash if the other is used.

As we don't currently support using crostini from non-primary
profiles, this CL fixes the first issue by associating all crostini
app windows with the primary profile, and prevents app icons from
being added to the shelf unless the primary profile is active.

Bug: 846213, 896996, 909590
Change-Id: Ibc888d22eb9567d822ce3e7d1af3603afb02a98e
Reviewed-on: https://chromium-review.googlesource.com/c/1491873Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Fergus Dall <sidereal@google.com>
Cr-Commit-Position: refs/heads/master@{#636675}

Add a PRESUBMIT check to ensure new |unsigned short| use don't
come in.

Bug: 929986
Change-Id: Ib4031362495ceedd94bc7e33fcbb913aea1e6ee2
Reviewed-on: https://chromium-review.googlesource.com/c/1493642Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Darwin Huang <huangdarwin@chromium.org>
Commit-Queue: Miyoung Shin <myid.shin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636674}

Before this CL, we always treated endScrollOffset as inclusive, against
the spec. The spec was recently updated to treat endScrollOffset as
inclusive but only when it is equal to max-offset:
https://github.com/WICG/scroll-animations/pull/37

Bug: 934989
Change-Id: I7cd9cf4619e804a54cef33ef8d3ec7395166bace
Reviewed-on: https://chromium-review.googlesource.com/c/1483682
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636673}

We've found visual corruption on this GPU / driver combination.
Blacklisting very restrictively for now. If we find this issue in future
drivers we can change the blacklist to >=.

Bug: 890688
Change-Id: I996fb8b9c7f83230aa9cd0f5fd39038d3f686eb2
Reviewed-on: https://chromium-review.googlesource.com/c/1496536
Commit-Queue: Eric Karl <ericrk@chromium.org>
Commit-Queue: Khushal <khushalsagar@chromium.org>
Auto-Submit: Eric Karl <ericrk@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636672}

Bug: 937069
Change-Id: Ic00c4ea761b16c685ca200df03e0c856765d450f
Reviewed-on: https://chromium-review.googlesource.com/c/1496276Reviewed-by: Robbie McElrath <rmcelrath@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636671}

https://skia.googlesource.com/skia.git/+log/b2565b18d03e..906430d4b40c


git log b2565b18d03e..906430d4b40c --date=short --no-merges --format='%ad %ae %s'
2019-02-28 csmartdalton@google.com Make GrMesh compatible with instanced draws that have no attribs
2019-02-28 reed@google.com Move responsibility to onMakeContext for device colorspace decision
2019-02-28 kjlubick@google.com [canvaskit] Add support for external fonts
2019-02-28 csmartdalton@google.com Revert "Drop unused shader support for sample variables"
2019-02-28 brianosman@google.com Particles: Removed emitters, added more full-featured position affectors
2019-02-28 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader c6eb41bebc27..a1924739029e (2 commits)
2019-02-28 recipe-roller@chromium.org Roll recipe dependencies (trivial).
2019-02-28 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader bc63cf9c511b..c6eb41bebc27 (3 commits)
2019-02-28 jvanverth@google.com Fix a couple of Mac windowing issues.
2019-02-28 bsalomon@google.com Remove support for SkDeferredDisplayListRecorder::DelayReleaseCallback::kNo.
2019-02-28 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader 201f136c9907..bc63cf9c511b (1 commits)
2019-02-28 thakis@chromium.org skia: Fix -Wextra-semi warnings in non-dcheck builds.
2019-02-28 michaelludwig@google.com Reland "Add general quad API to SkGpuDevice"
2019-02-28 brianosman@google.com Fix scaling error in SkLights::makeColorSpace
2019-02-28 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ../src 61d99852..1dc32aec (2053 commits)
2019-02-28 benjaminwagner@google.com Remove OS condition from isolate files
2019-02-28 kjlubick@google.com [canvaskit][pathkit] Remove flaky/low-utility Debug tests


Created with:
  gclient setdep -r src/third_party/skia@906430d4b40c

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

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

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

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel

BUG=chromium:926235
TBR=scroggo@chromium.org

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

Bug: 935683
Change-Id: Ie8c77f6a896810eb8e04032857570dd449dce02e
Reviewed-on: https://chromium-review.googlesource.com/c/1490952Reviewed-by: Jordy Greenblatt <jordynass@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636669}

Summary of changes:
 - Move the clear activities button under the history/stream tabs
 - Add extension name and icon to activity log page
 - Add headings above activity log items

Refactoring of styles may be done in a follow up CL

Screenshots included in:
https://bugs.chromium.org/p/chromium/issues/detail?id=932768#c5

Bug: 932768
Change-Id: I249cee13e1b5c7e4bd319b5b788a0c05718ba8da
Reviewed-on: https://chromium-review.googlesource.com/c/1483738Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636668}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/96fc7ac1..16c76cb9

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=luci.chromium.try:linux-blink-rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com

Change-Id: Ie7c9ac6c88ad2abc84a83c97f8666563c79bc8b1
Reviewed-on: https://chromium-review.googlesource.com/c/1496247Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#636667}

R=cmasso@chromium.org

Change-Id: I2e90056db5b017b34eeba901904371854bd41ff4
Reviewed-on: https://chromium-review.googlesource.com/c/1496241Reviewed-by: Mustafa Emre Acer <meacer@chromium.org>
Commit-Queue: Krishna Govind <govind@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636666}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/5117888302cf..a6d41e2396bd


git log 5117888302cf..a6d41e2396bd --date=short --no-merges --format='%ad %ae %s'
2019-03-01 vadimsh@chromium.org [cipd] Update to 2.2.18.


Created with:
  gclient setdep -r src/third_party/depot_tools@a6d41e2396bd

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

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

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



BUG=chromium:936911
TBR=agable@chromium.org

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

The story is already disabled on other version of system health and is
causing the shard to hang indefinitely.

NOTRY=true
TBR=crouleau@chromium.org
CC=perezju@chromium.org

Bug: 934270
Change-Id: Ia5b6328fa1311009cb8da0c6267a3585df5ad46b
Reviewed-on: https://chromium-review.googlesource.com/c/1496540Reviewed-by: Charlie Andrews <charliea@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636664}

RulesetMatcher is a class in the extensions layer but it is unit-tested at the
chrome layer. Fix this by moving the tests to the extensions layer by removing
dependencies to chrome's extension loading code. This also means we don't
check the extension loading in these tests, but that is taken care of in the
browser tests.

BUG=930961

Change-Id: Ic971bce981a65cda8928a8e3d9e3b49f960b4aef
Reviewed-on: https://chromium-review.googlesource.com/c/1496314Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Karan Bhatia <karandeepb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636663}

This is a mechanical execution of tools/git/mass-rename.py.

BUG=937018

Change-Id: Idf6f45d4507106e17137c577496e909b3aafc765
Reviewed-on: https://chromium-review.googlesource.com/c/1496234Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Robert Liao <robliao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636662}

This CL moves web_feature.mojom in public/platform to
public/mojom/web_feature and updates paths for web_feature.mojom.

It also moves PRESUBMIT.py for web_feature.mojom to
public/mojom/web_feature.

Bug: 919393
Change-Id: I1d22645fdd081a81bf18a85511d208c2e7b0bf7a
Reviewed-on: https://chromium-review.googlesource.com/c/1488487
Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636661}

Remove FetchContext::DidLoadResource and move the operations in it
to DispatchDidFinishLoading and DispatchDidFail.

This leads to code duplication but I think this change is overall a
good change because:
 - DidLoadResource is called only twice. Repeating yourself twice is
   not that bad.
 - DidLoadResource is confusing; Whether that precedes or succeeds
   DispatchDidFinishLoading, whether it should be called in
   DispatchDidFail, etc.

Previously FetchContext::DispatchDidFinishLoading and DispatchDidFail
were called before Resource::Finish, but now they are called after
Resource::Finish. That affected a few devtools tests because some tests
finished as soon as resource loading finishes and some error messages
got missing with the new semantics. This CL delays test completion a
bit in order to keep the expectations.

Bug: 914739
Change-Id: I5245dca240ebc7496d1de51909d458a6ed5a61d2
Reviewed-on: https://chromium-review.googlesource.com/c/1474887
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636660}

According to the plan to replace |const LocalFrame*| arguement with |const Document&|,
|ExecutionContext*|, or |DocumentLoader*| in UseCounter::Count(), this CL
replaces all uses of UseCounter::Count(const LocalFrame*) with
UseCounter::Count(const Document&) in frame directory.

Bug: 936187
Change-Id: I41f03b9908258881d4a1d932defb7e42e2deae9a
Reviewed-on: https://chromium-review.googlesource.com/c/1491060Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com>
Cr-Commit-Position: refs/heads/master@{#636659}

Adds and removes strikes under the following conditions:
User closes LocalCardMigrationDialog: Add 3 strikes
User closes LocalCardMigrationBubble: Add 2 strikes
User adds new local card: Remove up to 2 strikes (keep strike count > 0)
User de-selected some local cards during migration: Add 3 strikes

And don't offer local card migration if strike limit has been reached.


Bug: 884817
Change-Id: I9853e2b5d1a1692fd47650aa6598f4d6df27a27c
Reviewed-on: https://chromium-review.googlesource.com/c/1474689Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Jared Saul <jsaul@google.com>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Anne Lim <annelim@google.com>
Cr-Commit-Position: refs/heads/master@{#636658}

The Finch configs for the Quota Expansion experiment were updated to
use groups that better suited our experiment.  This change rectifies the
disconnect between the field trial testing config which is a prequisite
to moving an experiment into the beta channel.

Change-Id: I469ff653bac1a6634c762a7c49ac6e75bb7ce7ea
Reviewed-on: https://chromium-review.googlesource.com/c/1492672
Commit-Queue: Jarryd Goodman <jarrydg@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636657}

Bug: 	930708
Change-Id: I0848524c1d637bd83cf40ddf2ac0402033ee1d5c
Reviewed-on: https://chromium-review.googlesource.com/c/1484496
Commit-Queue: Yann Dago <ydago@chromium.org>
Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636656}

This CL updates DialogPresenterDelegate's
dialogPresenterDelegateIsPresenting property to follow delegate naming
guidelines.  Additionally, rather than storing this value as a property,
this CL updates the implementation to calculate on the fly whether BVC
or any of its children are modally presenting a UIViewController.

Bug: 893660
TEST: Follow repro steps in c#0 on crbug.com/893660
Change-Id: Id75e4e72bac6365d47caed11317c8a6a11a08dbc
Reviewed-on: https://chromium-review.googlesource.com/c/1492364
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Auto-Submit: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636655}

Reland #2 Note:
{
  Previously (re-)landed:
  https://crrev.com/b37e23cbd52ba447ab89c6d18f4ea0ecdc3fa5c0
  Reverted:
  https://crrev.com/a9aa35a2fb904074cddb4e2d889217a899177721

  More leaky tests. Though we now resolve gesture callbacks at the end
  of the test, the callback may come in after the test.

  This re-land fixes the leaky tests by finishing the
  test only once the gesture is completed. Previously landed CL uploaded
  as PS1, fixed in PS2.

  NOTE TO SHERIFF:
  If this is causing issues on the leak bots, please ping me or disble
  the test rather than reverting, if possible.
}

This CL makes synthetic input - the kind used in web tests and
telemetry (e.g. gpuBenchmarking.scrollBy) - wait until a CompositorFrame
has been submitted by the renderer and displayed by the display
compositor before resolving the completion callback. This means client
code that wants to wait until any observable side-effects of this input
is visible to further input need only wait on the gesture's completion
callback.

To give a motivating example: suppose we wish to write a test that
scrolls an out-of-process iframe into view and clicks on a button in the
frame. The code might look something like this:

gpuBenchmarking.smoothScroll(1000, () => {
  gpuBenchmarking.tap(0, 0);
});

This code contains a race today. The callback for smoothScroll is
invoked as soon as the ScrollEnd is received in the renderer. However,
until a new compositor frame is submitted from the renderer, the tap
may occur against stale hit testing geometry. This is a major source of
flakiness in our tests.

This CL fixes the problem by forcing the renderer to perform a full
redraw at the end of each gesture. The redraw produces a compositor
frame and we invoke the callback once the compositor frame is displayed.
We do this by reusing the RequestPresentation mechanism in RenderWidget.
RequestPresentation required two modifications to work in web tests
which use a single thread proxy with no scheduler:

 - LayerTreeHost::Composite needs to check the forced redraw flag to
   determine whether we need to raster, otherwise it won't produce a
   frame
 - RequestPresentation must request a main frame since there's no
   scheduler to perform the commit, which is what SetNeedsForcedRedraw
   requests.

The timing change exposed an issue in the
overlay-play-button-tap-to-hide.html test so this CL also cleans that
test up to listen to the animation changes in media controls properly.

Finally, it's possible we may get input in a RenderWidget that's not
currently displayed. e.g. A click event sent via ChromeDriver causes a
TouchStart followed by a TouchEnd. The TouchStart causes a window.open
which opens and focuses a new tab. The TouchEnd then happens on the
background tab. In this case, we should resolve the callback rather than
waiting on a CompositorFrame that'll never come. See ChromeDriver test
testNetworkConnectionTypeIsAppliedToAllTabs for an example of this.


TBR=dtapuska@chromium.org,nzolghadr@chromium.org,samans@chromium.org,peter@chromium.org

Bug: 902446
Change-Id: I0f818168b3f36070f72369a072e7f5eac64eb77c
Reviewed-on: https://chromium-review.googlesource.com/c/1496244
Commit-Queue: David Bokan <bokan@chromium.org>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636654}

Logic for max size is now consistent with animation/dragging.
Animation correctly plays if drag does not end at exact size.

This is not a complete fix for extensions dragging.

Bug: 916209
Change-Id: Iccb0d11bbe540a16edec0720105d272e091cf43d
Reviewed-on: https://chromium-review.googlesource.com/c/1492049
Commit-Queue: Dana Fried <dfried@chromium.org>
Reviewed-by: Dana Fried <dfried@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636653}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/a365aff3ed0b..fe5a160336aa


Created with:
  gclient setdep -r src-internal@fe5a160336aa

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

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

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


TBR=mmoss@chromium.org

Change-Id: I2487c44442221abbe2e4e57a4aeade55f71e8bf8
Reviewed-on: https://chromium-review.googlesource.com/c/1495960Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#636652}

Prior to macOS 10.13.2, the only way to check for TouchID was to check
for the ability to evaluate the LAPolicyDeviceOwnerAuthenticationWithBiometrics
policy. 10.13.2 introduced a more granular check for LABiometryTypeTouchID.

This PR ensures on systems after macOS 10.13.2 that we check for this specific
biometry, and fall back to to the old check if we do not have access to this new enum.

R=martinkr@google.com

Bug: N/A
Change-Id: I3a3d2d332547b09cf3c6e062709ed06b6e1a052d
Reviewed-on: https://chromium-review.googlesource.com/c/1487811Reviewed-by: Robert Sesek <rsesek@chromium.org>
Reviewed-by: Martin Kreichgauer <martinkr@google.com>
Commit-Queue: Martin Kreichgauer <martinkr@google.com>
Cr-Commit-Position: refs/heads/master@{#636651}

Per UX request, increasing the dropdown min-width.

Bug: 936914
Change-Id: I7a8f12ad9489337d65d369502ce0bb0bdedfcf7c
Reviewed-on: https://chromium-review.googlesource.com/c/1495040
Commit-Queue: Evan Stade <estade@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636650}

Screenshots: https://imgur.com/a/juHC3Fz

R=dpapad@chromium.org
BUG=936977

Change-Id: Ibef2af96df3599b79d3c94e979f8039a2cea42f0
Reviewed-on: https://chromium-review.googlesource.com/c/1496295
Commit-Queue: Dan Beam <dbeam@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Auto-Submit: Dan Beam <dbeam@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636649}

Adds a crash key so we can identify oom intervention related crashes.

Bug: None
Change-Id: I96722ec0af4362aa42536d119f92c06084cd9012
Reviewed-on: https://chromium-review.googlesource.com/c/1484396
Commit-Queue: Keishi Hattori <keishi@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Yuzu Saijo <yuzus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636648}

Using wpt-import in Chromium 932fdb31.
With Chromium commits locally applied on WPT:
68d5240e "[ChromeDriver] Set key modifiers with mouse actions"
a6d9cfda "[WPT] Changes to generated files by CL 1476144"
4665f96b "RestrictDeviceSensorEventsToSecureContexts by default."
03505df6 "[Origin Policy] Implement error reporting via Reporting API"
2c455f7b "Move outputContext from XRSession to XRRenderState"


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

Directory owners for changes in this CL:
rego@igalia.com:
  external/wpt/css/vendor-imports/mozilla/mozilla-central-reftests

NOAUTOREVERT=true
TBR=lukebjerring

No-Export: true
Change-Id: I4657b21944de6dfc1a0f35c8aa408e0dc4684260
Reviewed-on: https://chromium-review.googlesource.com/c/1495746Reviewed-by: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#636647}

BUG=930203
TEST=passes cq
TBR=tguilbert

Change-Id: I05eae1e344cbeabc48b12aca6cac64b2ea87c9e5
Reviewed-on: https://chromium-review.googlesource.com/c/1493161Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636646}

NetworkPortalDetectorImpl uses a raw pointer to the
SystemNetworkContextManager's URLLoaderFactory, which will be invalid
if mojo pipe runs into any issues. This CL makes it use a
SahredURLLoaderFactory instead, which will handle failures
automatically.

Bug: 936625
Change-Id: I2ea71ef6cc46440f66349729e16cc2558ef145b1
Reviewed-on: https://chromium-review.googlesource.com/c/1495748Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Commit-Queue: Robbie McElrath <rmcelrath@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636645}

This is a temporary solution until we implement a Fuchsia version of
CastSysInfo. The only downside to using the dummy is that the strings,
such as serial number, model, build, etc are all faked out.

Bug: internal b/126768956
Test: CQ
Change-Id: I3364d87c57d7f40dbe2d1e80b19bb2f02532512b
Reviewed-on: https://chromium-review.googlesource.com/c/1496419Reviewed-by: Yuchen Liu <yucliu@chromium.org>
Auto-Submit: Aidan Wolter <awolter@chromium.org>
Commit-Queue: Aidan Wolter <awolter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636644}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/00b1f1e2a50e..14a41b316425


git log 00b1f1e2a50e..14a41b316425 --date=short --no-merges --format='%ad %ae %s'
2019-02-28 dgarrett@google.com cbuildbot_launch: Wipe out the paygen subdir every build.


Created with:
  gclient setdep -r src/third_party/chromite@14a41b316425

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

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

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

CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel
TBR=chrome-os-gardeners@chromium.org

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

For pre-CompositeAfterPaint, we let subpixel accumulation propagate
through isolation to keep consistent with legacy compositing code.

For ComposteAfterPaint, we discard subpixel accumulation at
isolation.

Change-Id: Id9460dbb3c7c89931c38119e75f044b9079f3de8
Reviewed-on: https://chromium-review.googlesource.com/c/1488020Reviewed-by: vmpstr <vmpstr@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636642}

- unsigned short / unsigned -> uint16_t.
- USHRT_MAX -> std::numeric_limits<uint16_t>::max()
- Reference: https://google.github.io/styleguide/cppguide.html#Integer_Types

Bug: 929986
Change-Id: If80935d74042f4f8e50275d0ec17832e19ae7046
Reviewed-on: https://chromium-review.googlesource.com/c/1490367
Commit-Queue: Miyoung Shin <myid.shin@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636641}

The dimming layer is a extra fullscreen quad that we do not need to
draw. Remove it and dim the wallpaper instead.

Does a minor refactoring and moves some code from wallpaper view to
wallpaper controller to watch overview/tablet changes. Exposes wallpaper
view so we can schedule paints outside. Also fixes a bug with the
wallpaper preview being dimmed in tablet mode.

Next we should be removing the whole shield widget.

Test: manual
Bug: 774690, 917902
Change-Id: I303762e3180de9f82f1d0b1398305f7c7f9e0837
Reviewed-on: https://chromium-review.googlesource.com/c/1481899Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636640}

https://chromium.googlesource.com/angle/angle.git/+log/5de69e91bd93..5546fb4fdd85


git log 5de69e91bd93..5546fb4fdd85 --date=short --no-merges --format='%ad %ae %s'
2019-02-28 tobine@google.com Vulkan:Adding custom pool allocator
2019-02-28 shrekshao@google.com Use string literal style for shader string in WebGLCompatibilityTest
2019-02-28 shrekshao@google.com add ext_float_blend feature and test
2019-02-28 ynovikov@chromium.org Require VK_EXT_queue_family_foreign for EGL_ANDROID_image_native_buffer support


Created with:
  gclient setdep -r src/third_party/angle@5546fb4fdd85

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

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

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

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel

BUG=chromium:None,chromium:930993,chromium:857138
TBR=ynovikov@chromium.org

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

Add setTabletModeEnabled() private Autotest JS API.

It is needed for certain ChromeOS tests that need to switch/to
TabletMode without restarting Chrome. And using dbus PowerManager signal
is not reliable.

misc: added missing #include<> in tablet_mode_controller.cc reported by
      git cl lint

TEST=Tested with browser_test'test that is included in this CL.
     Manually tested on DUT by calling JS API manually.
BUG=chromium:931767

Change-Id: I25e888cba38c08188ec21475d3530ad3090bbcd3
Reviewed-on: https://chromium-review.googlesource.com/c/1470937
Commit-Queue: Ricardo Quesada <ricardoq@chromium.org>
Auto-Submit: Ricardo Quesada <ricardoq@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Mitsuru Oshima (OOO til 3/4) <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636638}

R=drubery

Bug: 933065
Change-Id: If5b7c41875f36f53c217607c2270688c4af3cad9
Reviewed-on: https://chromium-review.googlesource.com/c/1496298
Commit-Queue: Varun Khaneja <vakh@chromium.org>
Commit-Queue: Daniel Rubery <drubery@chromium.org>
Commit-Queue: Nathan Parker <nparker@chromium.org>
Reviewed-by: Daniel Rubery <drubery@chromium.org>
Reviewed-by: Nathan Parker <nparker@chromium.org>
Auto-Submit: Varun Khaneja <vakh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636637}