GitLab

Apparently is_asan = true implies win_console_app = true.
Fix for https://crrev.com/689827

TBR=jam@chromium.org

Change-Id: I422b59c25d4c1f99d5328ab1809c0f1733e96c89
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769223Reviewed-by: Reid Kleckner <rnk@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Reid Kleckner <rnk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690063}

This will enable starting the implementation of the Storage Access API behind these flags.

Bug: 989663
Change-Id: I790b107ea52a28796c69729dc7cd417fd9d08d94
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763266
Commit-Queue: Brandon Maslen <brandm@microsoft.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690062}

Remove negative filter for tests that have been fixed or disabled
by Selenium, and move one test to ready-to-run filter. See
https://crbug.com/chromedriver/2858#c3 for more details.

NOTRY=true

Bug: chromedriver:2858
Change-Id: If75d34c83e2b5bea70b27f7c8ef780fcbe3786fa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769528
Auto-Submit: John Chen <johnchen@chromium.org>
Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690061}

Whether to enable/disable is checked on the launcher thread in
content. But the actual flag is only accessible on the main thread
from chrome, so the plumbing in render_process_host and
content_browser_client exists to wire these two together.

BUG=750886
TEST=set policy, check renderers start without CIG.

Change-Id: I2ea2286d9e244734abe15b9f8beb62b3f97ffd71
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758589
Commit-Queue: Will Harris <wfh@chromium.org>
Reviewed-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690060}

This CL just addresses some minor leftover cleanups from this CL:
https://chromium-review.googlesource.com/c/chromium/src/+/1761063

Bug: 995393
Change-Id: I099d20fdf5ff593e22c392f74f6f937dea4c97b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768667Reviewed-by: Mark Pearson <mpearson@chromium.org>
Commit-Queue: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690059}

The virtual test suite was removed in [1]. These expectations files are
no longer needed.

[1] https://chromium.googlesource.com/chromium/src/+/31676667da8670621530457a4b5ea1a10eafe585

R=pdr@chromium.org

Bug: 989251
Change-Id: I74cba4ce4f8ddf4d2e5d7c7ad11016292d9022dc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769520
Auto-Submit: Majid Valipour <majidvp@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690058}

Spec change https://github.com/whatwg/html/pull/4787

This change marks navigations from the renderer indicating if they are
originating from script. If they are from script (as opposed to the back
and forward button default event handlers) they will be subject to the
top level navigation sandbox flag.

Intent to ship: https://groups.google.com/a/chromium.org/d/msg/blink-dev/jOa27iZPJtg/2ArNlXIBBAAJ

BUG=705583

Change-Id: I5169b54df63a30e252566398b139a7db187eaa42
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1729530Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690057}

... since both callee and caller sites run on the same
thread (assured by THREAD_CHECKER).

This was spotted by jbroman@ in [1]

[1] https://crrev.com/c/1768778/5/third_party/blink/renderer/modules/mediastream/apply_constraints_processor.cc#b23

BUG=704136
R=guidou@chromium.org, jbroman@chromium.org

Change-Id: Ifd9f06a490a469926acb778917ba23aaf11f9927
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768787
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Auto-Submit: Antonio Gomes <tonikitoo@igalia.com>
Cr-Commit-Position: refs/heads/master@{#690056}

When opening the color picker second time, it is init by
|customize.customColorPicked| and therefore selecting the same color
doesn't trigger color change event.

Reset |customize.customColorPicked| the same time when rest of the
color picker related values are reset.

Bug: 997120
Change-Id: I494c0a718c0389986de9014603b41754d0299ca5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769297Reviewed-by: Kristi Park <kristipark@chromium.org>
Commit-Queue: Gayane Petrosyan <gayane@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690055}

https://skia.googlesource.com/skia.git/+log/a0ed070aa0c7..9313874349dd

git log a0ed070aa0c7..9313874349dd --date=short --no-merges --format='%ad %ae %s'
2019-08-23 egdaniel@google.com Rename GrContextOption fReduceOpListSplitting to fReduceOpsTaskSplitting.
2019-08-23 mtklein@google.com track SkNoPixelsDevice's origin
2019-08-23 reed@google.com remove redundant rect methods
2019-08-23 senorblanco@chromium.org Dawn: implement a ring buffer for uniform uploads.
2019-08-23 csmartdalton@google.com Don't pass around renderTargetContexts from onFlush callbacks

Created with:
  gclient setdep -r src/third_party/skia@9313874349dd

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
TBR=egdaniel@google.com

Bug: None
Change-Id: I512d3e7f3ab3a49018ee7bca8ff015fb1b2eb75d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769476Reviewed-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@{#690054}

If a tab wants to move to model index to the left of the leftmost tab
in a group, where it should end up in TabStripLayoutHelper::slots_ is
ambiguous. If it is in the same group as the grouped tab, it should go
to the right of the header so the group remains contiguous. But if not,
it should go to the left. To disambiguate this case,
GetSlotIndexForTabModelIndex now takes the group of the tab at
|model_index| as a parameter. It cannot look up the Tab* in
TabStrip::tabs_ in the general case, since it might be called in the
middle of processing a MoveTab or RemoveTab, where the model indices
have already been updated in that data structure.

Bug: 989964
Change-Id: I2f35218d01c8398e6e024096bd35244ca67ca330
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764181
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Reviewed-by: Taylor Bergquist <tbergquist@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690053}

Using wpt-import in Chromium d3726d98.
With Chromium commits locally applied on WPT:
1194775a "Add ability to block same-origin access via document-access feature policy"
5546163a "Temporarily send `Sec-Fetch-*` headers via non-secure transport."
4afcd1a6 "Started implementing the STAPIT algorithm"


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:
foolip@chromium.org, lpz@chromium.org, robertma@chromium.org:
  external/wpt/tools

NOAUTOREVERT=true
TBR=raphael.kubo.da.costa

No-Export: true
Change-Id: Iedef9a3ff0deea645f9057457767b15988398eea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768786Reviewed-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@{#690052}

This adds a list of external storages (e.g. USB sticks) in
Settings > Device > Storage management > External storage preference.
Each entry for an external storage has an toggle switch. If a user
enables the switch, the corresponding storage becomes visible to Android
apps, which means that Android apps can read/write files on the storage.
The list of enabled storage's UUIDs will be kept in a preference.

design doc: go/arc-visible-external-volumes

Bug: 954228
Test: Manual test on with multiple USB devices attached to Kevin.
Change-Id: I75abc8b527c8fdf35d0b325338845603558f53d6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763553Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Commit-Queue: Naoki Fukino <fukino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690051}

Intent to Ship:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/JwSN1yEVvnk

Bug: 861704, 967867
Change-Id: I1b58befba0838a6a14762e8846c10c8a4a752ea9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1756764
Auto-Submit: Rouslan Solomakhin <rouslan@chromium.org>
Commit-Queue: Philip Jägenstedt <foolip@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690050}

Bug: 996949
Change-Id: Ifd918931d1c4a8ad976655230efb1f5e70f53956
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766601Reviewed-by: Alexander Cooper <alcooper@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Brandon Jones <bajones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690049}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/9499165e..be1d8500

Please follow these instructions for assigning/CC'ing issues:
https://v8.dev/docs/triage-issues

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
CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel
CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel
CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel
CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel

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

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

This reverts commit 9c5df46a.

Reason for revert: timing out tests

Original change's description:
> Enable a new SYSTEM background tracing mode.
> 
> This mode will allow us to set up a background scenario and deliver
> triggers to the system when something interesting is happening. Which
> will allow us to only upload traces that are of interest.
> 
> Bug: 979583
> Change-Id: Ic48ea08e3f9fafccaf045847cc0ddfa76d11611c
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760736
> Commit-Queue: Avi Drissman <avi@chromium.org>
> Reviewed-by: Avi Drissman <avi@chromium.org>
> Reviewed-by: oysteine <oysteine@chromium.org>
> Auto-Submit: Stephen Nusko <nuskos@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#689563}

TBR=avi@chromium.org,oysteine@chromium.org,eseckler@chromium.org,nuskos@chromium.org

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

Bug: 979583,997206
Change-Id: I51889cecfdc1ac392fb199b1a190b14880801e6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768981Reviewed-by: enne <enne@chromium.org>
Commit-Queue: enne <enne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690047}

- Support IconConverter to return a bundle of
success result and data.

Bug: 963290
Change-Id: I50d2908047aa1554d7d3c5a3ba2f1bf913c03d01
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762819
Commit-Queue: Hesen Zhang <hesen@chromium.org>
Reviewed-by: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690046}

The service stores a Mojo interface pointer in a TLS slot for each
thread that want to use this service. This is invalid because the
InterfacePtr class has sequence affinity and is thread hostile.

Bug: 995459
Change-Id: Icd1912998a11aa198415ec2657771a1cb57fa54b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1762486Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Reviewed-by: Ken Rockot <rockot@google.com>
Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690045}

Uses the existing 9-patch image from the Bottom Sheet for the Preview Tab
Overlay Panel and Tap to Search as well.

Increases the Bar height slightly, and makes the height no longer
variable (was needed for the promo-in-the-bar).

BUG=961454

Change-Id: I9edfef49ce2c37c4ad8bfd8a5dc67f5128dae93c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1606582
Commit-Queue: Donn Denman <donnd@chromium.org>
Auto-Submit: Donn Denman <donnd@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690044}

I reached out to Dai and he asked me to remove him from these lists.

I also see that tools/deep_memory_profiler was removed in
https://crrev.com/348397 in 2015.

R=dmikurube@chromium.org

Change-Id: I2b2cd820874cdf4485089746a5c43afef815c7ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764882Reviewed-by: Dai Mikurube <dmikurube@chromium.org>
Commit-Queue: Reid Kleckner <rnk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690043}

Bug: 993912
Change-Id: I76590932b60aeda6faef7e0b16ae35b0f38c3a27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761520
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Reviewed-by: Kelvin Jiang <kelvinjiang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690042}

Check to see if the |char_index| is within bounds, and return early if
it is. Otherwise, the code internal to PDFiumPage, which only works with
valid indices, will behave badly.

Bug: 996629
Change-Id: I71c27b1039e6d997d3ce5613e8df7a30882e8347
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1767606
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Kevin Babbitt <kbabbitt@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#690041}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4c555ee9b300..9e5ca57ed417


Created with:
  gclient setdep -r src-internal@9e5ca57ed417

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


CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome
TBR=jbudorick@google.com

Bug: chromium:None
Change-Id: I672620dfb31f0ec31d6fe8d35368eb3dd8ef4d97
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769377Reviewed-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@{#690040}

Change-Id: I7599b8c08a602307da5a5dc7b181d3cec86daac8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769303Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690039}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/afad531cd256..b408a71b6999

git log afad531cd256..b408a71b6999 --date=short --no-merges --format='%ad %ae %s'
2019-08-23 chrome-bot@chromium.org Update config settings by config-updater.

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

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@google.com

Bug: None
Change-Id: I11e4ca4e34524faeb082d8a80655b74ee2caba18
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769376Reviewed-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@{#690038}

This CL adds another test path for safety tip testing, this time with a
publicly-accessible URL.

Change-Id: I3aabd87401bd8bc602188293dfe9bdbc13a766a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769305
Auto-Submit: Joe DeBlasio <jdeblasio@chromium.org>
Reviewed-by: Emily Stark <estark@chromium.org>
Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690037}

This is an internal helper that's not meant to be publicly exposed.

Change-Id: Id318136ff07477a1c37d56ee3400c72842efdff4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769165
Auto-Submit: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690036}

SVG images can use the feature extraction and
classification logic that is being used by bitmap
images once their bitmap is available.

GetImageBitmap() is renamed to a more appropriate GetBitmap().

GetBitmap() is moved to the Image base class as the
implementations for this function are essentially
the same for Bitmap and SVG images.

CheckTypeSpecificConditionsForDarkMode() for SVGImage now
returns kNotClassified by default so that SVG images can also
proceed with the classification.

Few test cases are added in svg_image_test.cc to check
the working of the existing feature extraction and
classification logic for SVG images.

Bug: 951290
Change-Id: I55a6422a65c7e02010ba2d6d40c6bfdd24960cee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1602452Reviewed-by: Stephen Chenney <schenney@chromium.org>
Reviewed-by: Aran Gilman <gilmanmh@google.com>
Reviewed-by: Prashant Nevase <prashant.n@samsung.com>
Commit-Queue: Varun Chowdhary Paturi <v.paturi@samsung.com>
Cr-Commit-Position: refs/heads/master@{#690035}

This change updates top level mojo README, as well as the ones for
C++ bindings and IDL reference with Remote/Receiver types.

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

This reverts commit ff6293de.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> [tools/perf] Fix run_benchmark list command
> 
> Bug: 996493
> Change-Id: I3ebbe717f4ac64fede6c64afcab142918a9edaf9
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1763762
> Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
> Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#689865}

TBR=perezju@chromium.org,crouleau@chromium.org

Change-Id: Idad617cd6181685a01d1871c6bdcff118a585974
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 996493
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769519Reviewed-by: Behdad Bakhshinategh <behdadb@chromium.org>
Commit-Queue: Behdad Bakhshinategh <behdadb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690033}

Change-Id: I9aa4b202feecfebbc330e67883031bd281a16fc6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769179
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Commit-Queue: Eric Stevenson <estevenson@chromium.org>
Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690032}

We want to be able to experimentally control whether/if we preresolve or
preconnect in the background loop.

Technically, this API now supports doing both on a single trigger but we
don't intend on this being an active use case. Conversely, it also
allows both to be disabled which is a needed use case.

Bug: 971918
Change-Id: I41a814394f54c4fc3596834df7ef507435a8c450
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738644
Commit-Queue: Robert Ogden <robertogden@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690031}

Bug: 978682
Change-Id: I5e377df4121a1ef1992a30fbeff822712b52b8ca
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1766843Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690030}

The the function returns true only on the CSSNumericLiteralValue
subclass, this patch moves it from the base class to the subclass.

This might also help catching callers that are unaware of calculated
lengths and handle them incorrectly.

Bug: 979895
Change-Id: I114cc4ea58a652e02a5cb7d948010651584c6208
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1767096
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690029}

This reverts commit 4afcd1a6.

Reason for revert: repeatn-remove-add-animation.html is flaky (crbug.com/997279)

Original change's description:
> Started implementing the STAPIT algorithm
> 
> Changed SMILTimeContainer to step through all times where
> an animation stops or ends when just playing it normally. Currently
> it will only add slowdowns but also makes the animations more "correct".
> 
> This commit also fixes issue 688742
> 
> Bug: 688742, 986219
> Change-Id: I847abedc45aba110d09123793abfb3d9c018c94a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1730421
> Commit-Queue: Edvard Thörnros <edvardt@opera.com>
> Reviewed-by: Fredrik Söderquist <fs@opera.com>
> Cr-Commit-Position: refs/heads/master@{#689930}

TBR=pdr@chromium.org,fs@opera.com,edvardt@opera.com

Change-Id: If0fca62a4515a0771745f7b9d69f3f84b6665073
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 688742, 986219
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768829Reviewed-by: Nicolas Ouellet-Payeur <nicolaso@chromium.org>
Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690028}

Bug: 997258
Change-Id: I086e5378840d199a522b5e72fc4fc7b379a5ab50
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761444
Commit-Queue: Jarryd Goodman <jarrydg@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690027}

To create this change:
  * cd content
  * ../base/android/jni_generator/jni_refactorer.py -R --nonstatic
  * ../base/android/jni_generator/jni_refactorer.py -R
  * Manually fix up errors (primarily due to classes calling static
    native methods defined in other classes)
  * Refactor linker_test_apk to not use java_files since APK targets
    don't support the annotation_processor_deps arg yet
  * Remove JNI that is part of test targets (filed crbug.com/997199)

Bug: 929661
Change-Id: Ia4c81513696bf70266010830bed59dac0c3b163a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758655Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Eric Stevenson <estevenson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690026}

R=johnchen@chromium.org, kxloh@chromium.org

Change-Id: I9b50043385af5fb93f19a6d1c4c7244754486591
bug: 853749, 949246
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765443Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Auto-Submit: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690025}

Adds four new tests around tailored consent/level scenarios:
* TestConsentNotNeededForInline
* TestConsentPersistsLowerLevel
* TestConsentRepromptsHigherLevel
* TestConsentRepromptsAfterReload

Refactored WebXrVrConsentDialogBrowserTest methods into the base class
and re-works how base class methods work.  The mock (which always
accepts consent requests, but does not call out to display the dialog)
is now used unless the fake consent manager has previously been set up.

This allowed changing the consent tests to run on all runtimes rather
than just the OpenVr Runtime.

Renamed existing tests so that all test names fit a similar pattern.

Bug: 987029,996454
Change-Id: I9a4224fb169ee7df764a061fdb65038e42c05a8a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1767085
Commit-Queue: Alexander Cooper <alcooper@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690024}