GitLab

Call snd_ctl_close() to free all the resources allocated with
the control handle.

BUG=409016

Review URL: https://codereview.chromium.org/517363002

Cr-Commit-Position: refs/heads/master@{#292875}

The problem was introduced in https://codereview.chromium.org/110643005.
The URLBlacklistManager used to build the blacklist in the FILE thread
but it's now using the blocking pool; however, FlushBlacklistPolicy
wasn't updated to flush the blocking pool instead of the FILE thread.

This test (and the URL blacklist tests too) have been flaking now
and then on every bot but this race is extremely hard to trigger.
DrMemory on Windows was a consistent repro though.

BUG=339240

Review URL: https://codereview.chromium.org/520703002

Cr-Commit-Position: refs/heads/master@{#292874}

Analogically to CreateFile, we should just return an EXISTS error code if the
entry already exists. The EXISTS error code is later converted to OK if the
operation was requested to be not 'exclusive'.

TBR=benwells@chromium.org
TEST=browser_tests, unit_tests: *FileSystemProvider*CreateDirectory*
BUG=393141

Review URL: https://codereview.chromium.org/528683002

Cr-Commit-Position: refs/heads/master@{#292873}

Reland https://codereview.chromium.org/298333007/: Enable mac_strip_release under ASan on OSX. Remove the .saves files.

The reasons for reverting the CL hadn't been documented. I'm going to reland and update the bugs.

Unce upon a time the ASan runtime library used to be statically linked into
the executables on OSX.
Because that library provided a number of API functions needed by the shared
libraries, we had custom ASan-specific .saves files that told the `strip`
utility to preserve those functions when stripping the executables.

Then ASan switched to using dynamic runtime library, which instantly broke the
stripping step, because the executables weren't necessarily referencing all the
ASan API functions (issue 242503).
As a result, stripping has been disabled, and we haven't had .dSYM debug info
for ASan builds for more than a year now (issue 148383).

Because the ASan API functions are now undefined in the executables, it's
actually senseless to use the .saves files to preserve those functions.
This CL deletes the .saves files and turns mac_strip_release on for ASan
builds.

BUG=148383, 242503, 170739, 166857
TBR=mark@chromium.org

Review URL: https://codereview.chromium.org/517803004

Cr-Commit-Position: refs/heads/master@{#292872}

TBR=qinmin@chromium.org
BUG=398669

Review URL: https://codereview.chromium.org/527033003

Cr-Commit-Position: refs/heads/master@{#292871}

Cr-Commit-Position: refs/heads/master@{#292870}

https://chromium.googlesource.com/chromium/blink/+log/46b3b608360a078fb8ddae2c3c22042eacbc2e87..1cf1b6f4079b8fa8b7319d081de8f3031a0eee62
TBR=hclam@chromium.org,tkent@chromium.org

Review URL: https://codereview.chromium.org/530553007

Cr-Commit-Position: refs/heads/master@{#292869}

We check if we have camera or mic access when devices are enumerated to determine if labels should be visible or not. We must ensure all settings are checked when doing this.

* Add interface to ContentBrowserClient for checking media access permission. Add implementation in //chrome.
* Refactor permission checking in MediaStreamDevicesController to be able to use them from elsewhere.
* Add checking policy and app permission to the existing content settings check.
* Remove old interfaces on ResourceContext and its implementations.
* Move the permission check from MediaStreamDispatcherHost to MediaStreamManager.
* Refactor MediaStreamManager to be able to mock the check function and update unit test to mock it.

TBR=jochen@chromium.org (for a few trivial changes)

BUG=406094

Review URL: https://codereview.chromium.org/483523006

Cr-Commit-Position: refs/heads/master@{#292868}

See the issue (http://crbug.com/407392#5) for the image of restyled design.

BUG=407392
TEST=manually tested

Review URL: https://codereview.chromium.org/528733002

Cr-Commit-Position: refs/heads/master@{#292867}

BUG=407388
TEST=manual

Review URL: https://codereview.chromium.org/518843002

Cr-Commit-Position: refs/heads/master@{#292866}

This patch makes chromecast always bundle the frontend,
and exposes all content targets via remote debugging.

BUG=none

Review URL: https://codereview.chromium.org/517493003

Cr-Commit-Position: refs/heads/master@{#292865}

Revert of Update V8 to version 3.29.35 (based on bleeding_edge revision r23537). (patchset #1 id:1 of https://codereview.chromium.org/530683003/)

Reason for revert:
This makes DevTools in device mode crash silently (crbug.com/409604).

Original issue's description:
> Update V8 to version 3.29.35 (based on bleeding_edge revision r23537).
>
> TBR=jarin@chromium.org
>
> Committed: https://chromium.googlesource.com/chromium/src/+/5c3fee7d50d3f5984d4462fef9cfce694e1ae8a9

TBR=jarin@chromium.org,machenbach@chromium.org
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/530763002

Cr-Commit-Position: refs/heads/master@{#292864}

1) Use hash in inputview url to support fast inputview switching.
2) Add 'm17n:' to m17n keyboard layout id to avoid conflicts (e.g. 'si' in XKB vs. 'si' in M17n)
3) Change layout IDs for Chinese IMEs to support future compact layouts.

BUG=254398,406979
TEST=Verified on linux_chromeos.

Review URL: https://codereview.chromium.org/473603003

Cr-Commit-Position: refs/heads/master@{#292863}

R=ppi@chromium.org

Review URL: https://codereview.chromium.org/522653004

Cr-Commit-Position: refs/heads/master@{#292862}

Remove a bunch of code that is already longer used or
will become unused when https://review.webrtc.org/22149004/
and https://codereview.chromium.org/505153002/ are landed.

Remove hardcoded paths to the WebRTC isolate files to make
it easier for us to make changes to them, since we can pass
the path to the .isolate file on test execution instead
(keeping the configuration in the WebRTC buildbot recipe instead).
Previously we had to roll our Chromium revision in WebRTC DEPS
every time a change was made before it became used.

Add support for a CHECKOUT_SOURCE_ROOT environment variable
used to make it possible to override the hardcoded path traversal
that is done to find the src/ directory (since it's different for
WebRTC bots). I considered passing a flag into the test_runner.py
scripts instead of using an environment variable, but my opinion is
that it would pollute the code too much since
constants.DIR_SOURCE_ROOT is used in 70 different places.

BUG=webrtc:3741
TEST=local building and test execution using command lines like this:
cd /path/to/webrtc/src
export CHECKOUT_SOURCE_ROOT=`pwd`
build/android/test_runner.py gtest -s tools_unittests --isolate-file-path=webrtc/tools/tools_unittests.isolate

R=navabi@chromium.org

Review URL: https://codereview.chromium.org/500423004

Cr-Commit-Position: refs/heads/master@{#292861}

This CL removes:
- "Move window to..." menu item from the gear menu.
- Remove APIs fileBrowserPrivate.onProfileAdded and fileBrowserPrivate.onDesktopChanged as they are used only for updating gear menu.
- Remove corresponding tests.

BUG=408965
TEST=run browser_test, unit_tests
TBR=benwells@chromium.org
# TBRing for just removing unused events from file_browser_private.idl

Review URL: https://codereview.chromium.org/530553006

Cr-Commit-Position: refs/heads/master@{#292860}

BUG=408949
TEST=manually

Review URL: https://codereview.chromium.org/529693002

Cr-Commit-Position: refs/heads/master@{#292859}

https://chromium.googlesource.com/chromium/blink/+log/78807df2ce8a43f50941cee9ae05458df0cc05bf..46b3b608360a078fb8ddae2c3c22042eacbc2e87
TBR=

Review URL: https://codereview.chromium.org/526033002

Cr-Commit-Position: refs/heads/master@{#292858}

The hotwording checkbox was moved from the 'Privacy' section
of the settings page in https://codereview.chromium.org/382123004/

This cl moves it back to the 'Privacy' section and shows and hides
the 'Voice' section according to the enable-experimental-hotwording flag.

BUG=407893

Review URL: https://codereview.chromium.org/523573004

Cr-Commit-Position: refs/heads/master@{#292857}

Previously WebSocketFrameTestMaskBenchmark.* were included in
net_unittests (with a small number of iterations to avoid slowing down
testing). Move them to net_perftests.

Also remove the command-line option to control the number of iterations,
and use the standard base::PerfTimeLogger output instead of custom logging.

BUG=407445
TEST=net_unittests, net_perftests

Review URL: https://codereview.chromium.org/502353002

Cr-Commit-Position: refs/heads/master@{#292856}

TBR=jarin@chromium.org

Review URL: https://codereview.chromium.org/530683003

Cr-Commit-Position: refs/heads/master@{#292855}

R=navabi@chromium.org

Review URL: https://codereview.chromium.org/517983002

Cr-Commit-Position: refs/heads/master@{#292854}

This changes monitoring of video frames delivered to MediaStreamVideoSource to be started when the source is started and stopped when the source is stopped rather than depend on
that the source object is destroyed since destruction is dependent on when blink runs garbage collection.

It removes the unused MediaStreamVideoTrack::SetMuted and GetMuted.

It reduces the need for a thread hop between the io-thread and the render thread for each monitor intervall by only trigger the monitoring callback on if a change has occured.

It reduces the test time of media_stream_video_source_unittest.cc from more than 7s to around 200ms.

BUG= 404106

Review URL: https://codereview.chromium.org/509873002

Cr-Commit-Position: refs/heads/master@{#292853}

BUG=400674
TBR=palmer@chromium.org

Review URL: https://codereview.chromium.org/496703002

Cr-Commit-Position: refs/heads/master@{#292852}

This patch makes the method providing additional metadata for entries available
for provided file systems too.

NOTRY=True
TEST=Tested manually. Files app works as before.
BUG=408017

Review URL: https://codereview.chromium.org/515033002

Cr-Commit-Position: refs/heads/master@{#292851}

Fix mock expectation for the flaky test.

BUG=408443
TEST=unit_tests --gtest_filter=SyncFileSystemServiceTest.SimpleLocalSyncFlow

Review URL: https://codereview.chromium.org/524193004

Cr-Commit-Position: refs/heads/master@{#292850}

What’s New in OpenGL ES 3.0 among GL extentions
- Vertex array
- Element index uint
- npot texture
- float texture (1)(2)
- GLSL standard derivatives
- Blend min&max

(1) GLES3 supports float and half float texture but only half float texture is
filterable and doesn't allow them for color attachment.
(2) This CL doesn't enable half_float due to conflicting GL_OES_texture_float

BUG=405484

Review URL: https://codereview.chromium.org/489023002

Cr-Commit-Position: refs/heads/master@{#292849}

BUG=none

Review URL: https://codereview.chromium.org/517103003

Cr-Commit-Position: refs/heads/master@{#292848}

The underlying issue is that by the time we get around to processing a live
region node from a mutation observer, the node may no longer be part of
the DOM, in which case we should treat it as invisible.

To fix it, fix cvox.DomUtil.hasInvisibleAncestor_ so that it returns
true (invisible) if passed an element that's not actually part of the
document.

BUG=408821

Review URL: https://codereview.chromium.org/517853002

Cr-Commit-Position: refs/heads/master@{#292847}

The field is not used, so it should be removed.

NOTRY=True
TEST=Tested manually that Files app works correctly.
BUG=408017

Review URL: https://codereview.chromium.org/522533003

Cr-Commit-Position: refs/heads/master@{#292846}

When a user starts to drag an item before they haven't selected any item yet,
FileTable.lastSelection_ can be undefined and this ends up an incorrect drag image.
The existence of lastSelection_ should be checked.

BUG=409514
TEST=manually

Review URL: https://codereview.chromium.org/532453002

Cr-Commit-Position: refs/heads/master@{#292845}

This is causing problems when a function that returns an array of
nav descriptions returns more than one item, they're not all processed
correctly.

BUG=408821

Review URL: https://codereview.chromium.org/517843002

Cr-Commit-Position: refs/heads/master@{#292844}

PaginationController abstracts over mouse wheel, touch screen and
touchpad event handling for updating the PaginationModel. This allows
any client of PaginationModel (ie, ContentsView) to properly handle
scrolling events. It also removes the code duplication in OnMouseWheel
and OnScrollEvent.

BUG=406222

Review URL: https://codereview.chromium.org/524503003

Cr-Commit-Position: refs/heads/master@{#292843}

TBR=ygorshenin@chromium.org
BUG=381584
NOTRY=true

Review URL: https://codereview.chromium.org/526013002

Cr-Commit-Position: refs/heads/master@{#292842}

Cr-Commit-Position: refs/heads/master@{#292841}

Previously, thumbnails were generated from full size images, which in case of
cloud providers would invoke downloading the entire file.

This CL adds an ability to provide a thumbnail as a data URI, so:
(1) We can have thumbnails for other file types than images.
(2) Thumbnails are shown fast.
(3) Resource consumption is significantly reduced.

Note, that the thumbnails are not wired to Files app yet. That will be done in
a separate patch.

TEST=unit_tests: *FileSystemProvider*GetMetadata*
BUG=407954

Review URL: https://codereview.chromium.org/513683002

Cr-Commit-Position: refs/heads/master@{#292840}

This is a preparation to support AXObject in PagePopup.
PagePopup doesn't have WebLocalFrame, and it's difficult to provide it. We can
remove WebLocalFrame dependency of BlinkAXTreeSource by using
WebDocument::scrollOffset, minimumScrollOffset, and maximumScrollOffset.

This CL has no functionality changes.

BUG=123896

Review URL: https://codereview.chromium.org/510123003

Cr-Commit-Position: refs/heads/master@{#292839}

This change...

(a) updates the install sequence based on the latest spec. Before this
patch, the installing version is set after resolving the register promise.
After this patch, the installing version is set before resolving the promise.

(b) fills registration's version attributes before resolving the register
promise. This should fix the problem mentioned c#5 in the issue and should
absorb the change of the timing to set the installing version due to (a).

(c) adds a separate IPC for updatefound event to adjust the timing to fire
the event. Before this patch, the event shares SetVersionAttributes message.

[1] Blink: https://codereview.chromium.org/524193003/
[2] Chromium: THIS PATCH
[3] Blink: https://codereview.chromium.org/517223002/

Spec: https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#installation-algorithm

BUG=406240
TEST=content_unittests --gtest_filter=ServiceWorker*
TEST=run_webkit_tests.py http/tests/serviceworker/

Review URL: https://codereview.chromium.org/517493002

Cr-Commit-Position: refs/heads/master@{#292838}

OnBufferingUpdate does not work for local resources. Because of this
the buffered range does not get updated. This patch updates the bufferd
range atleast untill the video has played so that media controls can be
painted properly.

TBR=abarth@chromium.org
BUG=405474

Review URL: https://codereview.chromium.org/496683002

Cr-Commit-Position: refs/heads/master@{#292837}

https://skia.googlesource.com/skia/+log/700a78d72fdf4f09b28446818028b39f31e26b38..eb2662c08f07764491a17e0a397cdbeb0e2c05c1

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=jvanverth@google.com

Review URL: https://codereview.chromium.org/529613002

Cr-Commit-Position: refs/heads/master@{#292836}