GitLab

Bug: 954088
Change-Id: Ia234842a363473a8feadc0b1091c462d91def7e5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574359
Auto-Submit: Łukasz Anforowicz <lukasza@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652330}

Bug: 952741
Change-Id: Ib4d37a308bf1b3c32304edd8bcdc29306b085df6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572708
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Dana Fried <dfried@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652329}

The transport roles in signaling playground have been flipped, such
that playgrounds can connect to each other but playground->host will
fail because both the client and the host send each other an offer.

This CL fixes this issue by flipping the roles.

Bug: 947337
Change-Id: If4f5413ca86dfed0d5a1b2c6e949679f44e021a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574825Reviewed-by: Joe Downing <joedow@chromium.org>
Commit-Queue: Yuwei Huang <yuweih@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652328}

This reverts commit 1b6db403.

Reason for revert: This change caused a regression,
it removed the link to the management page from the settings page.

Original change's description:
> Management WebUI: Remove duplicate link to management page from settings
>
> Bug: 879146
> Change-Id: Id461109a2fff5b3d077158abefb638dd5527c7f4
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1569511
> Commit-Queue: Yann Dago <ydago@chromium.org>
> Reviewed-by: Hector Carmona <hcarmona@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#651698}

TBR=hcarmona@chromium.org,ydago@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 879146
Change-Id: If57862619c86a8ff4c9174d1bbfabfea26faaba4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574621Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Yann Dago <ydago@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652327}

Bug: 951655
Change-Id: I74b1932cb717dc1d306e2594e33fe9591a50e953
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572358
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652326}

This patch:
* Reorders check conditions when reusing lineboxes so that
  it can fail earlier in common cases.
* Moved most of conditions to NGInlineNode to allow reordering
  and to avoid virtual calls.

No behavior changes.

Bug: 636993
Change-Id: I044871e5ab51724e5b256043e0471329c344d3fb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1568374Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652325}

Bug: 945272
Change-Id: I39f3613ec5310ae33e97c717150d6ceb8f94ac87
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1569105
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652324}

This reverts commit 3a554bbc.

Reason for revert: Causing crbug.com/954356

Original change's description:
> [ios] Turn on UMA feature flag by default
> 
> Bug: 931289
> Change-Id: Ibd211f4a28d394793b986ad835015ac9c4948768
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1569748
> Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
> Reviewed-by: Sergio Collazos <sczs@chromium.org>
> Reviewed-by: Steven Holte <holte@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#651915}

TBR=holte@chromium.org,sczs@chromium.org,thegreenfrog@chromium.org

Change-Id: I4bc5a56ae19fddd87adef1e948265abbdf31205f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 931289
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574428Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652323}

Keep Drive.PushNotification* for a few more milestones since they'll be
useful for monitoring the migration to FCM for Drive push notifications.
The remaining histograms are only recorded by the legacy Drive client,
which should be removed in 76, so mark them as obsolete.

Bug: 945540
Change-Id: I58914b29519bc16d30135ee0fd064686d926e017
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1568375
Commit-Queue: Jesse Doherty <jwd@chromium.org>
Auto-Submit: Sam McNally <sammc@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652322}

Vector icons allow us to use only one representation for all device
scale factors and take less bits.

This patch should save about 229kb on an official build:

Before patch
chrome_100_percent.pak    1,061,874 bytes
chrome_200_percent.pak    1,462,605 bytes
chrome                  236,097,832 bytes
total                   238,622,311 bytes

After patch
chrome_100_percent.pak      977,496 bytes
chrome_200_percent.pak    1,259,524 bytes
chrome                  236,155,296 bytes
total                   238,392,316 bytes

difference                  229,995 bytes

Change-Id: Ib03fad381d1b2bc4bc7b9fa7a9660264ab751d49
Bug: 950110
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1573665
Commit-Queue: Nina Satragno <nsatragno@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Reviewed-by: Kim Paulhamus <kpaulhamus@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Dan Beam <dbeam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652321}

This is a precursor CL for https://crrev.com/c/1532361.

In this CL, we remove dependencies of g_browser_process in
prefs_tab_helper.

Bug: 937469
Change-Id: I4724ccaa3769a721743979f4300ab4a45d1315ed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1567898
Commit-Queue: Xi Han <hanxi@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652320}

This is a precursor CL for https://crrev.com/c/1532361.
We introduce StartupData which owns all of the pre-create objects,
including the ChromeFeatureListCreator.

Bug: 937469
Change-Id: I9ea3d13d32de2a91e6eb4a0aecc13d2419f7f881
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1562573
Commit-Queue: Xi Han <hanxi@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652319}

Bug: 951860
Change-Id: I452b27c0c778296933e50ab897ff4e8e8a2f3939
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574423Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Yaron Friedman <yfriedman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652318}

This provides a real MediaKeySystemAccess to the
MediaCapabilitiesDecodingInfo object when decodingInfo() is  queried
with a supported KeySystemConfiguration.

This doesn't yet query the stats DB for smoothness and power
efficiency - coming in a follow up.

Bug: 907909
Change-Id: I049196ee2a3244a3f3ad6231c3d3da080beb91d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1554420
Commit-Queue: Chrome Cunningham <chcunningham@chromium.org>
Reviewed-by: John Rummell <jrummell@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Auto-Submit: Chrome Cunningham <chcunningham@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652317}

* Shorten code
* Use ClampToRange()
* Facilitate later conversion to range-based for
* Avoid implicit size_t->int conversions
* DCHECK_LE
* constexpr
* Using non-const ref locals is less typical than pointers
* No real need for ButtonLayout() to constantly DCHECK that it has one or two
  buttons

Bug: none
Change-Id: I0b6454f2cffdc9854358f81edc31491494909bd8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574626
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Peter Boström <pbos@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652316}

This CL adds new layout treatment for dictionary definitions when they
are returned from the server:
1) Uses the existing "Book Round" vector icon when a dictionary is present.
2) Styles the Title to have two parts, with the first part being the word
the second part being the pronunciation shown gray.

The icon is controlled by a new ContextualSearchCardIconControl to allow
displaying an icon for cards from the server that are not quick actions.

Also update the ContextualSearchImageControl to use the term "Card Icon" in
place of "Quick Action Icon" since the icon can come from either a Quick
Action or some other kind of card.

BUG=936695, 954225

Change-Id: I26230ae060a00deda621ab776e673da9103e5730
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572244Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Donn Denman <donnd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652315}

This patch disables the following tests:
* http/tests/devtools/oopif/oopif-presentation-console-messages.js
* virtual/nobinary-for-devtools/http/tests/devtools/oopif/oopif-presentation-console-messages.js
* http/tests/security/svg-image-with-cached-remote-image.html
* virtual/outofblink-cors/http/tests/security/svg-image-with-cached-remote-image.html

TBR=lukasza@chromium.org,pdr@chromium.org

Bug: 953994, 954297
Change-Id: I14221edefd296b1a9b9d838e67ab4df16b2aec53
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574678Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652314}

Bug: 954166
Change-Id: I7a59e32c60982eb4761d57bf64875991894adc43
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572588Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Yann Dago <ydago@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652313}

The packet_count_for_client_min is 1 in this case.  The default is
3 (so far).  The default of 3 works fine, but 0 is below min.  The min
will be enforced soon so this CL prepares for that.

Change-Id: Ia1184156d0aa4945f39dab2b69bea0d64ebab854
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574680Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Dustin Green <dustingreen@google.com>
Cr-Commit-Position: refs/heads/master@{#652312}

I don't have a @chromium.org address, so the @google.com is necessary.

Change-Id: Iad68b94a35e67582191ba81343707f64ede5f4be
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574228
Commit-Queue: Jared Saul <jsaul@google.com>
Commit-Queue: Anne Lim <annelim@google.com>
Auto-Submit: Jared Saul <jsaul@google.com>
Reviewed-by: Anne Lim <annelim@google.com>
Cr-Commit-Position: refs/heads/master@{#652311}

Added ifdef for OS_FUCHSIA in GetPlatformSpecificTextureTarget().
Previously it was marked as NOTREACHED().

Bug: 852011
Change-Id: I5c8d05b2db95e559d11586a39fb75928b2916d28
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572536
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652310}

Prevents an "infinite" list of extensions to push beyond the screen. The
actual max height for this scroll view was picked kind of arbitrarily
and might be revisited.

Bug: chromium:943702
Change-Id: Iee47cdfb97e6214f6b80a2d362abbfbda4d9babd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572877
Commit-Queue: Peter Boström <pbos@chromium.org>
Reviewed-by: Collin Baker <collinbaker@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652309}

This change adds a connection error handler to InterfaceTarget
and exposes it in Foo and FooCallbackRouter generated classes,
as well as modifies some conditions for consistency with
the old bindings.

Bug: 894376
Change-Id: I317d0ae5eb6cbf534d2411dd9c99c5770080b2df
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1555011
Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
Reviewed-by: Ken Rockot <rockot@google.com>
Cr-Commit-Position: refs/heads/master@{#652308}

user_cloud_policy_manager_builder.cc now contains the code for
creation. It is just a helper function to abstract details away from
ProfileImpl.

Bug: 940522
Change-Id: I2e37c34162afd366674b0b817c3c6760aabfa560
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1568621
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: Xi Han <hanxi@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652307}

When SSLInfo is empty, we need to trigger a regular net error instead
of an interstitial, otherwise a crash happens. A case where this was
happening was when loading a cert related error via
chrome://network errors. This CL fixes that.

Bug: 953812
Change-Id: Iee7a46e71586fde5f4bd51c40aae93bc796fbfa6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1573858
Commit-Queue: Carlos IL <carlosil@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652306}

I added some manual instrumentation, and planes are very rarely
R_8 and never RG_88. They are usually RGBA_8888 and occasionally
LUMINOSITY_F16.

This is a follow up to a Skia side change:
https://skia-review.googlesource.com/c/skia/+/206900

Change-Id: I87a9b7c1cf1069712cd132dca659377272c53c5b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1573663Reviewed-by: Peng Huang <penghuang@chromium.org>
Reviewed-by: Brian Salomon <bsalomon@google.com>
Cr-Commit-Position: refs/heads/master@{#652305}

Now uses a different target. Also explicitly call out the option to
copy the expectations from the trybot.

Change-Id: I2d2865d830c1186557b2075c6b14389117ae87c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574152Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Commit-Queue: Eric Stevenson <estevenson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652304}

ShelfView should check existence of AppListController before using it
to avoid a crash.

Bug: 954317
Change-Id: Id5d5797f1867c6f88f00d5a3e09e587f22e0fcdd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574624Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Michael Giuffrida <michaelpg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652303}

This reverts commit f45ff75d.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 652196 as the
culprit for flakes in the build cycles as shown on:
https://analysis.chromium.org/p/chromium/flake-portal/analysis/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vZjQ1ZmY3NWQxZWU5MzJlODY5NWEzNDI2OWMyNjRmOTNiNzc0MzllMgw

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-dbg/11937

Sample Failed Step: browser_tests

Sample Flaky Test: DownloadTest.SaveLargeImage

Original change's description:
> Move loading of data URLs out of the network process.
> 
> For subresources the renderer was already loading it in-process. Now for navigations and downloads, we keep them in the browser process. This avoids:
> 1) sending GURLs bigger than 2MB to a different process, which strips them as they're bigger than url::kMaxURLChars
> 2) having the network process deal with a scheme that's not going over the network, which we avoid
> 
> Bug: 937146, 939871
> Change-Id: I8ca8225308b2a5f1f90594ab06943f0641a5c551
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1570805
> Commit-Queue: John Abd-El-Malek <jam@chromium.org>
> Reviewed-by: Ken Rockot <rockot@google.com>
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Reviewed-by: David Trainor <dtrainor@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#652196}

Change-Id: I599629fd38c350366c44d4d5a88d86ab0b4bd6b5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 937146, 939871
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574685
Cr-Commit-Position: refs/heads/master@{#652302}

As crbug/948234 described, currently from within DFM it is not
possible to access resources from base module via
org.chromium.chrome.autofill_assistant.R. The error happens at
runtime only, so you can compile, but then running into crashes.

The solution is to use  org.chromium.chrome.R instead, which means
we need to use full path if autofill_assistant.R is already imported.

Also, this CL fixes a bug in AutofillAssistantFacade which skips
autobot start when it should not.

Change-Id: I52b58e03ec1cf0a80fa5669b6c7e39d3ac6593d9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574459
Commit-Queue: Hui Wu <wuandy@chromium.org>
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652301}

An earlier refactor (*) accidentally removed the clearing of
previous visual overflow before computing a new value.

This has the effect of the visual overflow being too large after
movement or animation, but also overflow of LayoutRect precision
for elements far off-screen.

(*) https://chromium-review.googlesource.com/c/chromium/src/+/1366767

Bug: 950963

Change-Id: Idb0609c814d0acf6058383202063ed34a8caf69e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572924
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Reviewed-by: Stephen Chenney <schenney@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652300}

> Created UpdateNotificaionController listening to the
UpdateStatusProvider, and will push notification when available.
> Implemented Play Store update flow and inline-update flow
after receiving the notification.


Bug: 952354
Change-Id: I81fa80c7f1cb350fdfb0b33df86c3ef8fd078f93
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1564584Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Xing Liu <xingliu@chromium.org>
Commit-Queue: Hesen Zhang <hesen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652299}

Implement toggleMostVisitedOrCustomLinks for the EmbbededSearchApi.
This will allow users to toggle between Most Visited tiles and custom
links.

Bug: 953822
Change-Id: Ide2d1a62456649a5b6f9ff45d16548ac1212e0b5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572233Reviewed-by: Kyle Milka <kmilka@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Greg Kerr <kerrnel@chromium.org>
Commit-Queue: Kristi Park <kristipark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652298}

The preference minimum font size has a default value of 0. The preference is exposed
in the UI on the appearance fonts page in settings via a slider. The allowable values
for the preference excluded the default value of 0.

The settings slider code was changed to ensure the value of the preference was also
an allowable value. When the font settings page loaded, the preference was then
updated to the closest allowable value. If the preference is set to the default value,
the closest allowable value is 6px.

This change adds 0 as an allowable value which will allow a user to turn off the
effects of a minimum font size.

Bug: 944002
Change-Id: Ifc1c72e3673eef60e62f3edecd07ccba6b67928a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1573238
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652297}

This function is used for controlling audio experiments. The only experiment
available to control via this API was the AEC3 (new echo canceller). That has
now been fully rolled out, and there are no plans to add other experiments to
control this way. Thus removing the function completely.

Bug: 696930
Change-Id: I398e783747865b8774637d712f33c4c4c07260ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1570827Reviewed-by: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Commit-Queue: Henrik Grunell <grunell@chromium.org>
Auto-Submit: Henrik Grunell <grunell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652296}

Bug: 919685
Change-Id: I306637940553044ed6fd57b13f00c0d12b6cd0c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1566612Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: Yi Su <mrsuyi@chromium.org>
Commit-Queue: Chris Lu <thegreenfrog@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652295}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/060c5300eae0..86dcbc73338c


git log 060c5300eae0..86dcbc73338c --date=short --no-merges --format='%ad %ae %s'
2019-04-18 evanhernandez@chromium.org api: Nuke dead controller/protos.


Created with:
  gclient setdep -r src/third_party/chromite@86dcbc73338c

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: I2d1212844179c8c12ce057ad07096be515e8ca2d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574244Reviewed-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@{#652294}

https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/d754b70592ca..7aad9653f9a3


git log d754b70592ca..7aad9653f9a3 --date=short --no-merges --format='%ad %ae %s'
2019-04-18 zoddicus@users.noreply.github.com Remove legacy utility functions (#2530)


Created with:
  gclient setdep -r src/third_party/SPIRV-Tools/src@7aad9653f9a3

The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-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.


TBR=dsinclair@chromium.org

Change-Id: Icb91fe0d002e3e2644f6ef6cfb78fff13a611672
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574242Reviewed-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@{#652293}

Check in bits of code from the pending two-way sync CL. It allows me to monitor the flakiness of webdriver_test_suit on Linux Tests (dbg)(1) while the other CL is being reviewed. The changes are taken from https://chromium-review.googlesource.com/c/chromium/src/+/1484616/36

Sample JSON output:
https://pastebin.com/CfuVTuP8

The is_unexpected fields for all of the three tests are false because their expectations exist in the WebDriverExpectations.

Bug: 934919
Change-Id: If068f2318f2f2cbe6b866f96f14cb97f941a4df0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574360Reviewed-by: John Chen <johnchen@chromium.org>
Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: Kyle Ju <kyleju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652292}

In
https://chromium.googlesource.com/catapult/+/734e7ea0cbf9e0965596350ea51e5b4c8fc623f0,
I introduced a change to Web page Replay Go (WPR Go) to prefer previously
unmatched responses. This change fixed a number of issues with capturing
checkout workflows on captured sites. The change hides the fix behind a
feature flag for compatibility reasons.

This change enables the fix by turning on the feature flag in Captured
Sites tests.

Bug: 850756
Change-Id: I0331b3b5cfe8b0ddde84211bd0b320ef65faa8fb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574358
Auto-Submit: Yiming Zhou <uwyiming@google.com>
Reviewed-by: Fabio Tirelo <ftirelo@chromium.org>
Commit-Queue: Fabio Tirelo <ftirelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652291}