GitLab

When signin cookies are "cleared on exit" through the Chrome settings,
Dice would rebuild them on next startup, which defeats the point of
that cookie settings.
To fix this, tokens are not loaded when signin cookies are cleared on
exit. Instead, the tokens are revoked on the server, and deleted from
the database.
The secondary accounts are completely removed from Chrome, but the
primary account is kept in authentication error state, so that Sync
can display an error.

TBR=treib, eugenebut

Bug: 836862
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I5a72b3a173242494ad1c996765c838bb9a7f09f1
Reviewed-on: https://chromium-review.googlesource.com/1042392
Commit-Queue: David Roger <droger@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555771}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/e043a8b3..1f462965

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;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,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org

Change-Id: Ic0de3034053e0f7368ba535b770ae0bd2e92bfa8
Reviewed-on: https://chromium-review.googlesource.com/1042211Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555770}

Also adds an integration test for path stripping, but not for
quick check.

Bug: 715695
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I45d37089d305c5a6683afbac762ef767b910fb0f
Reviewed-on: https://chromium-review.googlesource.com/1028177
Commit-Queue: Matt Menke <mmenke@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555769}

Changes:
When empty results from Android, barrier closure immediately runs,
we should first initialize items_ to avoid nullptr crash.

Bug: 803291
Test: tested on device saw crash fixed
Change-Id: I4fd3a7dc29a056ac2e7e22a5c535d6a38a69e171
Reviewed-on: https://chromium-review.googlesource.com/1041576Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Qiang Xu <warx@google.com>
Cr-Commit-Position: refs/heads/master@{#555768}

CL https://chromium-review.googlesource.com/c/chromium/src/+/1019245 seems
to have introduced a jumbo build failure:

./../../content/browser/renderer_host/media/media_devices_manager.cc:704:37: error: reference to overloaded function could not be resolved; did you mean to call it?
                  ignore_group_id ? operator== : EqualDeviceAndGroupID)) {
                                    ^~~~~~~~~~
../../content/common/media/media_devices.h:52:6: note: possible target for call
bool operator==(const MediaDeviceInfo& first, const MediaDeviceInfo& second);
     ^
../../content/public/common/page_state.h:63:13: note: possible target for call
inline bool operator==(const PageState& a, const PageState& b) {
            ^

Let's move page_state.h's ==/!= operators out of the global scope, to avoid this.

TBR=avi@chromium.org

Bug: 834281
Change-Id: I27f0493ee2d2957b8318c969edd5615ec4dab687
Reviewed-on: https://chromium-review.googlesource.com/1042308
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555767}

The old predictor is automatically disabled when the new one is enabled.

Change-Id: Ia5c1a79dbeb0e096720fc3d50da3dc707b3761a6
Reviewed-on: https://chromium-review.googlesource.com/1042388Reviewed-by: Egor Pasko <pasko@chromium.org>
Commit-Queue: Alexandr Ilin <alexilin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555766}

Most changes are in preparation for supporting MojoDecryptorService
directly in MediaService.

Bug: 806018
Test: Tests updated.
Change-Id: I4ea5a6098ee62f1b1a2ecf4ada559e2825076768
Reviewed-on: https://chromium-review.googlesource.com/1041651
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555765}

TBR=smut

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ic5060153174b0258ee7505b8760a9d143afe489b
Reviewed-on: https://chromium-review.googlesource.com/1042449
Commit-Queue: iOS Autoroller <ios-autoroller@chromium.org>
Reviewed-by: iOS Autoroller <ios-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555764}

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

As Views based UI is undergoing new changes, that applies to MacViews
browser as well, but diverges from Cocoa based browser. This CL solves
that problem by creating a function for Cocoa browser only. This
function can get Cocoa specific constants which won't be affected by
Views side of changes.

BUG=838630,838856
TEST=Pls make sure bookmark bar, icons and menus look ok for both Cocoa
version and MacViews version on Mac. They should be same as before for
other platforms.

Change-Id: I73152c05cfac77eb6b24a4d6204acb78767c833d
Reviewed-on: https://chromium-review.googlesource.com/1041141Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555762}

There are no new files, this just splits sources into public and
sources. This is in preparation to adding some implementation specific
files.

BUG=680698
TEST=build file change only

Change-Id: I198f7961094909ea9015719efb4ec563b83abb7d
Reviewed-on: https://chromium-review.googlesource.com/1041526Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555761}

Refs (or hashes) are useful for navigating, but not useful for
comparing URLs. We compare URLs when deduping URLs in Omnibox
matches, and when testing eligibility for switching to already
open tabs. This CL changes GURLToStrippedGURL() to remove refs
from the stripped version of destination URLs (only used for
comparison) and when testing URLs for tab switching.

We only remove refs when the input doesn't specify a ref.
This is critical to proper deduping.

Also added unit test.

Bug: 780835
Change-Id: Ifef8d8b9d2efd4394b598342c13ef8aa439089ce
Reviewed-on: https://chromium-review.googlesource.com/1025953
Commit-Queue: Kevin Bailey <krb@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555760}

Bug: 834281
Change-Id: I5ef11a3343e5b534bee6e088a36fb3365bd218e8
Reviewed-on: https://chromium-review.googlesource.com/1019323
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Reviewed-by: Harald Alvestrand <hta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555759}

https://skia.googlesource.com/skia.git/+log/4722a4135911..2f62de01d3a7

$ git log 4722a4135..2f62de01d --date=short --no-merges --format='%ad %ae %s'
2018-05-03 borenet [recipes] Mostly remove bot_update patch_ref hack
2018-05-02 khushalsagar fonts: Cleanup header guard and stray printfs.

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;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
TBR=brianosman@chromium.org

Change-Id: Ib5863c6e9da4c25334bba511b39dae62a4c54c07
Reviewed-on: https://chromium-review.googlesource.com/1042490
Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#555758}

https://chromium.googlesource.com/catapult.git/+log/978be7439d1e..a87e87c2a7f4

$ git log 978be7439..a87e87c2a --date=short --no-merges --format='%ad %ae %s'
2018-05-01 perezju [soundwave] Replace bugs.DataFrameFromApi with DataFrameFromJson
2018-05-03 perezju [soundwave] Store timeseries data in database
2018-05-03 wangqing-hf [MIPS64] Add MIPS64 d8 binary (version 5.7.492)
2018-05-03 wangqing-hf [MIPS64] Update the update_v8 script to support MIPS64.

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.


TBR=sullivan@chromium.org

Change-Id: I60e30aad7a212a48f02d21645dd079a1b3fcb0a1
Reviewed-on: https://chromium-review.googlesource.com/1042526Reviewed-by: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#555757}

Allows user drop down menu to activatble so that it can be cycled
through.
Add a custom focus search so the drop down menu closes after all items
have been cycled through.
Migrate Closure to OnceClosure.
Make drop down menu a transient child of system bubble.

Test: manual
Bug: 799401
Change-Id: I67af2bb3e58c88f6926ab65ecf565d69325d00b8
Reviewed-on: https://chromium-review.googlesource.com/1038656Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555756}

We weren't including the `include-subdomains` field of each endpoint
group in the data we were handing over to the net-internals page.  (The
net-internals code that reads this field is already there; we just
weren't filling in a value.)

Bug: 829353
Change-Id: Ifdff1d6e0baf802eb7f094571060b93c761936af
Reviewed-on: https://chromium-review.googlesource.com/1019661Reviewed-by: Julia Tuttle <juliatuttle@chromium.org>
Commit-Queue: Douglas Creager <dcreager@google.com>
Cr-Commit-Position: refs/heads/master@{#555755}

Following bot results are included.
5144 5146 5148 5149 5150 5151 5152 5154 5155

1 lines were removed and 0 lines were deflaked by consecutive
results since 5100.

TBR=eae@chromium.org, mstensho@chromium.org
NOTRY=true

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ib0614dfcb6f39c7e101920875079278f1bb795c7
Reviewed-on: https://chromium-review.googlesource.com/1040225
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555754}

Don't DCHECK on password being plain text in existing user controller.
Make it if statement.

BUG=chromium:839314
TEST=manual

Change-Id: I7a649ddb922edd2105563cb2b4959b764d0383a4
Reviewed-on: https://chromium-review.googlesource.com/1041958
Commit-Queue: Roman Sorokin <rsorokin@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555753}

These virtual test suites no longer exist.

Bug: 823365
Change-Id: I62503d3b99f192f46e09acd6b8f38fad84e1ee9a
Reviewed-on: https://chromium-review.googlesource.com/1042369Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Steve Kobes <skobes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555752}

Using wpt-import in Chromium 4ff5b671.

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

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:
jsbell@chromium.org, mek@chromium.org:
  external/wpt/FileAPI

TBR=kereliuk

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

https://pdfium.googlesource.com/pdfium.git/+log/6312bf94bb44..ccd942158992

$ git log 6312bf94b..ccd942158 --date=short --no-merges --format='%ad %ae %s'
2018-05-03 tsepez Add CPDF_Page::Extension::GetDocExtension()
2018-05-03 tsepez Make GetPDFDocument return a CPDF_Docment*, not void*.

Created with:
  roll-dep src/third_party/pdfium


The AutoRoll server is located here: https://pdfium-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.


TBR=dsinclair@chromium.org

Change-Id: I57a6b3d5a99ada684414265fb3d323869a4d13f9
Reviewed-on: https://chromium-review.googlesource.com/1042371
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#555750}

This is a reland of 480d1516. Patch was not responsible for bot failures.

Original change's description:
> Scroll support for hosted dialogs in VR
>
> Now that VR supports html <select>, we should send scroll events
> to dialogs.
>
> Bug: 836292
> 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;master.tryserver.chromium.linux:linux_vr
> Change-Id: I45d44af2069e21bedd0662d3f26ab6c89ff82288
> Reviewed-on: https://chromium-review.googlesource.com/1028584
> Commit-Queue: Amirhossein Simjour <asimjour@chromium.org>
> Reviewed-by: Ian Vollick <vollick@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#555454}

TBR=vollick@chromium.org, asimjour@chromium.org

Bug: 836292
Change-Id: I8391daee7a1908273d5349106237e2177ad3c293
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;master.tryserver.chromium.linux:linux_vr
Reviewed-on: https://chromium-review.googlesource.com/1042546Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555749}

The new text makes it more clear how the setting impacts Chrome OS vs
Chrome browser.

BUG=839308
R=atwilson@chromium.org

Change-Id: I73de539820bd1fb7b822c88fad2eb6f2f2e8b3ab
Reviewed-on: https://chromium-review.googlesource.com/1042306Reviewed-by: Drew Wilson <atwilson@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555748}

R=alexclarke@chromium.org

Change-Id: I77cf2e26643a862217d009cc7539fd838d3c1583
Reviewed-on: https://chromium-review.googlesource.com/1042189
Commit-Queue: Alexander Timin <altimin@chromium.org>
Reviewed-by: Alex Clarke <alexclarke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555747}

The version without the "virtual/video-surface-layer/" prefix already had it.

Bug: 520736,838128
Change-Id: I815f420dd134143837d1602e795075176f4e4eb5
Reviewed-on: https://chromium-review.googlesource.com/1042565Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555746}

Now that AudioInputDevice doesn't automatically redirect calls to the
correct thread, ProcessedLocalAudioSource needs to do that itself.

Bug: 838929
Change-Id: I55f1cf9c6446b90b464b229e4c9a621613bf9dfb
Reviewed-on: https://chromium-review.googlesource.com/1041954
Commit-Queue: Oskar Sundbom <ossu@chromium.org>
Reviewed-by: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555745}

I think digit@chromium.org is more preferable these days, so change the
remaining cases of digit@google.com to the preferable way for extra
consistency.

TBR=estevenson@chromium.org
NOTRY=true

Bug: None
Change-Id: Ie545282b334f056c7e72836caabec8539441ec3b
Reviewed-on: https://chromium-review.googlesource.com/1030376
Commit-Queue: Egor Pasko <pasko@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: David Turner <digit@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555744}

If we pinch an element with a touch-action that prevents pinch in an
OOPIF, we route the pinch events to the child so that the child's
TouchActionFilter filters them. Due to crbug.com/771330, the
TouchActionFilter could let the pinch events through. We'll
filter such events in RenderWidgetHostViewChildFrame::FilterInputEvent
to prevent them from being sent to the child renderer.

Bug: 827182
Change-Id: I3782a2d135a008df271a73d79ecade25dcc9f1a5
Reviewed-on: https://chromium-review.googlesource.com/1040354Reviewed-by: James MacLean <wjmaclean@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555743}

TBR=pauljensen@chromium.org

Bug: 838524
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I1ae71d8c07f91827e164b046d4c379aeee4125ac
Reviewed-on: https://chromium-review.googlesource.com/1042397
Commit-Queue: Marc Treib <treib@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555742}

This method is creating a net::HttpRequestHeaders object, serializing
it to a string, and then parsing that string to create a new
net::HttpRequestHeaders object.

CL just returns the net::HttpRequestHeaders object directly.
No expected behavior change.

TESTS=Covered by existing tests.

Bug: None
Change-Id: I356c5987e0dcf60e454efc08f93d25ec75954c11
Reviewed-on: https://chromium-review.googlesource.com/1040705
Commit-Queue: Antonio Gomes <tonikitoo@igalia.com>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555741}

The horizontal padding around the browser actions
container was incorrectly reduced.

This causes issues where you can't resize the
omnibox (because the extensions now overlap the gripper)
or drag the extensions into the wrench menu

Bug: 825378, 825782
Change-Id: I66061a67546fab679c280a8d816b78dbe19b876e
Reviewed-on: https://chromium-review.googlesource.com/1040914Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Sarah Chan <spqchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555740}

The test WorkspaceLayoutManagerBackdropTest.OpenAppListInOverviewMode is flaking
so I'm disabling.

TBR= xiyuan@chromium.org
TEST=WorkspaceLayoutManagerBackdropTest.OpenAppListInOverviewMode

Bug: 838756
Change-Id: I68f90b668996a885ef37e047d453e6ecca12217c
Reviewed-on: https://chromium-review.googlesource.com/1042350Reviewed-by: Jonathan Ross <jonross@chromium.org>
Commit-Queue: Jonathan Ross <jonross@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555739}

Now that we can safely depend on C++11, add a few move
constructors to the String and Vector utility classes.

This will be used in future CLs used to fix various
issues with the crazy linker.

BUG=802068
R=pasko@chromium.org, lizeb@chromium.org, agrieve@chromium.org

Change-Id: I7fe420fe16e03e2eb1dbabafdcc0fb72c40ce3ba
Reviewed-on: https://chromium-review.googlesource.com/1041960
Commit-Queue: David Turner <digit@chromium.org>
Reviewed-by: Egor Pasko <pasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555738}

Bug: 706592
Change-Id: I5e399e76e3ca98ee80d76aa4fd92b2d3bba60d39
Reviewed-on: https://chromium-review.googlesource.com/976085Reviewed-by: Kevin Marshall <kmarshall@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555737}

This is done specifically to filter out the aggregate devices created
when using kAudioUnitSubType_VoiceProcessingIO. Those should not be
directly user-selectable.

This CL allows other aggregate devices, but ensures they do not claim
to support native echo cancellation, since they don't. Those devices
will have to use software AEC.

Bug: 837227
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
Change-Id: Iebb5286cafebe40204899ee2b7f1e0ff263d1fd6
Reviewed-on: https://chromium-review.googlesource.com/1030170
Commit-Queue: Oskar Sundbom <ossu@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555736}

NOTRY=true
TBR=nednguyen@google.com

Bug: 839411
Change-Id: I659bb4ca9a7cac243e6ccc294d383ce71776f84d
Reviewed-on: https://chromium-review.googlesource.com/1042527Reviewed-by: Annie Sullivan <sullivan@chromium.org>
Commit-Queue: Annie Sullivan <sullivan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555735}

With DCHECKs enabled, GetStorageKey() might potentially fail in some
codepaths that are updated in this patch.

In addition, we introduce a CHECK for a scenario that is subject to
investigation.

Bug: 837517
Change-Id: I903f83e8ed7ca96aa525c0640aa9ffc629fc5721
Reviewed-on: https://chromium-review.googlesource.com/1042393Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555734}

Bug: None
Change-Id: Ib80295f0aaa4bb2001c2ffd28d78924030b06e4f
Reviewed-on: https://chromium-review.googlesource.com/1042289Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Vadym Doroshenko <dvadym@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555733}

Currently MSE_ExternalClearKey/EncryptedMediaTest.Playback_Multiple_VideoAudio_WebM
is flaking on Mac. So I'm disabling it until that can be resolved.

TBR=xhwang@chromium.org
TEST=MSE_ExternalClearKey/EncryptedMediaTest.Playback_Multiple_VideoAudio_WebM

Bug: 835585
Change-Id: Ifb82978c391d99804667c87a3d43ea1539bf9110
Reviewed-on: https://chromium-review.googlesource.com/1042373Reviewed-by: Jonathan Ross <jonross@chromium.org>
Commit-Queue: Jonathan Ross <jonross@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555732}