GitLab

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266539 0039d316-1c4b-4281-b951-d872f2087c98

If a buffer is missing duration and is being held back, then the cluster
parser now holds back all other tracks' buffers that have same or higher
(decode) timestamp. This keeps the timestamps emitted for a cluster
monotonically non-decreasing and in same order as parsed, even across
tracks within the cluster.

Adds a related new WebMClusterParserTest, updates related
ChunkDemuxerTests and removes an obsolete TODO now that bug 361786 is
fixed by r265340.

R=acolwell@chromium.org
BUG=363421
TEST=All media_unittests and MSE http layout tests pass locally on Linux

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266538 0039d316-1c4b-4281-b951-d872f2087c98

BUG=367283

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266537 0039d316-1c4b-4281-b951-d872f2087c98

Adding the arm64 target to gyp_webview.

BUG=354405
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266536 0039d316-1c4b-4281-b951-d872f2087c98

Revert of Move  service_discovery_client_impl.* into chrome/common. (https://codereview.chromium.org/256923003/)

Reason for revert:
[Sheriff] Speculating that this lead to a sizes regression:
https://chromeperf.appspot.com/report?masters=Chromium&bots=chromium-rel-linux-64&tests=sizes&rev=266525&checked=core

Will reland otherwise.

Original issue's description:
> Move  service_discovery_client_impl.* into chrome/common.
> It's need to be used by utility and browser processes.
> 
> BUG=349645
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266487

TBR=thestig@chromium.org,vitalybuka@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=349645

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266535 0039d316-1c4b-4281-b951-d872f2087c98

Revert of Add nacl_loader_unittests to the chromium_builder_tests (https://codereview.chromium.org/260003005/)

Reason for revert:
Unconditionally adds a nacl dependency, breaks disable_nacl=1 builds

Original issue's description:
> Add nacl_loader_unittests to the chromium_builder_tests
> 
> TEST=ninja -C out/Release chromium_builder_tests
> BUG=364751
> R=hidehiko@chromium.org
> TBR=mseaborn@chromium.org
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266521

TBR=hidehiko@chromium.org,mseaborn@chromium.org,hamaji@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=364751

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266534 0039d316-1c4b-4281-b951-d872f2087c98

We'll have to figure out a way to maintain this file without blocking v8
rolls. While the gyp files in v8 don't change very often, they sometimes
do, and since the CQ checks the gn build, we have to find a way to
update gn in lock step

BUG=none
TBR=brettw@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266533 0039d316-1c4b-4281-b951-d872f2087c98

Gallery.app refers a few Files.app script files, so the CL also changes so that the files can be imported from the Gallery.app

BUG=358698
TEST=manually

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266531 0039d316-1c4b-4281-b951-d872f2087c98

BUG=367701
TBR=finnur@chromium.org
NOTREECHECKS=true
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266530 0039d316-1c4b-4281-b951-d872f2087c98

The kGoogleServicesUsername pref indicates that Chrome is signed in with a
GAIA account. Since device-local accounts do not have GAIA credentials,
this pref should not be set for them.

BUG=327849
TEST=Extended browser tests

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266529 0039d316-1c4b-4281-b951-d872f2087c98

BUG=367190

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266528 0039d316-1c4b-4281-b951-d872f2087c98

Gin Java Bridge passes values between Java and JS using base::Value.
Since there are some kinds of data that can't be represented in JSON,
we use base::BinaryValue that holds GinJavaBridgeValue. And we don't
use base::BinaryValue for anything else (this is enforced by our
Strategy for V8ValueConverter).

BUG=355644
R=bulach@chromium.org, jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266527 0039d316-1c4b-4281-b951-d872f2087c98

Verify how the existing implementation deals with Date, Function and RegExp
JS objects. This is to make sure that we configure V8ValueConverter appropriately
in the Gin-based implementation.

BUG=355644
R=benm@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266526 0039d316-1c4b-4281-b951-d872f2087c98

NOTRY=true
BUG=347927

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266525 0039d316-1c4b-4281-b951-d872f2087c98

CreateSnapshotFile and GetFileStreamReader used to sniff the content
of the files and returned error if it does not look like media data.
This CL adds a mode to disable the check, and use the disabled mode
in MTP mounting in Chrome OS file manager.

BUG=367004

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266524 0039d316-1c4b-4281-b951-d872f2087c98

Retry r266297 with static initializer fix.

This reverts commit 9ca50c9f.

BUG=351824
TBR=rmsousa@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266523 0039d316-1c4b-4281-b951-d872f2087c98

It seems that the per-file /components/password_manager* section should match the content of /components/password_manager/OWNERS. The diff is exactly dubroy and vabr (present in the latter, missing in the former). This CL adds them to the former.

NOTRY=true

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266522 0039d316-1c4b-4281-b951-d872f2087c98

TEST=ninja -C out/Release chromium_builder_tests
BUG=364751
R=hidehiko@chromium.org
TBR=mseaborn@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266521 0039d316-1c4b-4281-b951-d872f2087c98

As another step towards a platform-agnostic test suite for the password
management UI, this CL refactors ManagePasswordsIconView in a few ways:

1. It introduces a new 'ManagePasswordsIcon' abstract base class
   to provide a platform-agnostic interface to the Omnibox icon.

2. The ManagePasswordsBubbleUIController is now responsible for
   determining the Omnibox's icon's state, and also responsible
   for triggering automatic display of the bubble UI. The actual
   functionality is still sitting on the icon for the moment, but
   will be moved out to a platform-agnostic layer in a future CL.

3. Most importantly, some basic unit tests verify the functionality
   added above.

BUG=356678

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266520 0039d316-1c4b-4281-b951-d872f2087c98

As an optimization, we used to clear render surfaces when they were
elided from the RenderSurfaceLayerList. It turns out that these surfaces
are still important from time to time; a scroll child will need to
access the transforms of these surfaces in order to put its clip in the
right space.

This CL retains the layer lists and it also updates RenderSurface to
own a LayerList rather than a RenderSurfaceLayerList (which was always
a bit of a kludge).

R=danakj@chromium.org
BUG=366386

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266519 0039d316-1c4b-4281-b951-d872f2087c98

Separated the optional and required permissions in the dialog, and
added retained files as well. Also made the various types of
permissions collapsible, and added a message when the app has no
permissions whatsoever.

BUG=356985,350746

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266518 0039d316-1c4b-4281-b951-d872f2087c98

Turns out the "ordinal" values in GestureEventDetails are never used.
This patch replaces https://codereview.chromium.org/256593003/.

BUG=332418

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266517 0039d316-1c4b-4281-b951-d872f2087c98

This CL reorganizes code before adding the read directory implementation.

First of all, operations are now extracted to separate classes, to avoid huge
file_system_provider.cc. Such operation classes are now arguments for the
RequestManager which replaces the former pair of [success, error] callbacks.

Such operation classes are now easy to test, because of ability to cut out
the EventRouter by using Operation::SetDispatchEventImplForTests.

Finally, a RequestValue class has been introduced, which replaces former
base::DictionaryValue. The former solution was not good, since we already
parse the base::Value in api function implementations (using generated from
IDL parsers). So, passing a raw base::Value and later parsing is again is an
overkill.

The RequestValue class is a trivial wrapper for all kinds of IDL values, which
can be passed through the RequestManager to Operation classes.

TEST=unit_tests, browser_tests: *FileSystemProvider*
BUG=248427

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266516 0039d316-1c4b-4281-b951-d872f2087c98

BUG=363540
TEST=manually tested.

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266515 0039d316-1c4b-4281-b951-d872f2087c98

In https://codereview.chromium.org/238123003 an import
of base.gyp is introduced in build/common.gypi. This
makes GYP fail for some client projects using the Chromium
build toolchain. By adding a condition variable for this
part, it is possible to avoid this include by setting
use_sanitizer_options=0.

NOTRY=True
BUG=302040
TEST=passing runhooks on a variety of trybots.

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266514 0039d316-1c4b-4281-b951-d872f2087c98

This CL fixes the process that sends pending frames when a WebSocketChannel
receives quota from the renderer process. The IOBuffer data of a frame can
be null, but the implementation didn't take account of that.

BUG=364788
R=ricea@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266513 0039d316-1c4b-4281-b951-d872f2087c98

Revert of [Telemetry] Enable the measurement smoke test on android. (https://codereview.chromium.org/250593003/)

Reason for revert:
[Sheriff] This is failing on android now (slightly after its activation, but I can't figure out the actual culprit):
http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/20079

Original issue's description:
> [Telemetry] Enable the measurement smoke test on android.
> 
> This required disabling the benchmarks that were causing it to fail. It looks
> like those failures were due to lack of tab support in ChromeShell.
> 
> BUG=337829
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266457

TBR=bulach@chromium.org,tonyg@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=337829

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266511 0039d316-1c4b-4281-b951-d872f2087c98

BUG=367695
TBR=piman@chromium.org, mihaip@chromium.org
NOTREECHECKS=true
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266510 0039d316-1c4b-4281-b951-d872f2087c98

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266509 0039d316-1c4b-4281-b951-d872f2087c98

Some systems might like to use Mojo IPC to talk to services that are
running outside the auspices of the Mojo shell. DBus is one potential
channel to use for bootstrapping such a connection.

In order to allow the externally-running service to accept connections
from a Mojo shell, we need to get it a ShellHandle. This change
defines a DBusServiceLoader that implements the ServiceLoader
interface.  DBusServiceLoader creates a dedicated MessagePipe, passes
a handle to one end to the desired service over DBus, and then passes
the ShellHandle over that pipe.

This class assumes the following:
1) Your service is already running,
2) Your service implements the Mojo ExternalService API
   (from external_service.mojom).
3) Your service exports an object that implements the org.chromium.Mojo
   DBus interface:
   <interface name="org.chromium.Mojo">
     <method name="ConnectChannel">
       <arg type="h" name="file_descriptor" direction="in" />
     </method>
   </interface>

This change also includes a trivial DBusEchoService that can be
"loaded" using the DBusServiceLoader, as well as a mojo example app
that connects to this service.

BUG=364903
TEST=run mojo_dbus_echo_service in one shell, and then load mojo_dbus_echo in the mojo shell. "who" should successfully echo between the two.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=265927

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266508 0039d316-1c4b-4281-b951-d872f2087c98

NavigationList's activeModelItem_() is called even when the actual active model is not changed.
It occurs when the list items on navigation list are permutated by mounting/unmounting devices.
This patch checks if the actual active model is changed.
Code for clearing selection on clicking root items is moved to click event handler. 

BUG=361047
TEST=manually tested
ran browser_tests gtest_filter=*FileManager*

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266507 0039d316-1c4b-4281-b951-d872f2087c98

If the ancestorEntry is root directory, the URL for it ends with slash.
Appending one more slash cause failure to comparison.
I think appending slash is not necessary because we have already checked
that the ascestorEntry is directory.

BUG=361047
TEST=ran browser_tests gtest_filter=*FileManager*

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266506 0039d316-1c4b-4281-b951-d872f2087c98

BUG=331299
TEST=None, no functional changes. Just build content_shell,
content_unittests, content_browsertests to make it still link.
TBR=jochen@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266505 0039d316-1c4b-4281-b951-d872f2087c98

Missed a case when enabling the flag on CrOS in r266119.

BUG=367099

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266504 0039d316-1c4b-4281-b951-d872f2087c98

This makes the "Rasterization" field work properly again and adds
some constants to prevent this from breaking again.

BUG=367124

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266503 0039d316-1c4b-4281-b951-d872f2087c98

BUG=326577
TEST=Each detail how has both the time and status on one
row rather than split over two rows.

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266502 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266501 0039d316-1c4b-4281-b951-d872f2087c98

This is a speculative revert of a libvpx roll to unblock dev release.

BUG=366025

> Roll libvpx 263116:264320
> 
> Enable AVX2 code:
> https://codereview.chromium.org/236183003
> 
> Fix Win build issues:
> https://codereview.chromium.org/239003004
> 
> Leave AVX2 ready but off:
> https://codereview.chromium.org/240153006
> 
> Remove all the AVX2 references:
> https://codereview.chromium.org/240113008
> 
> Review URL: https://codereview.chromium.org/236263002

TBR=johannkoenig@google.com

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266500 0039d316-1c4b-4281-b951-d872f2087c98

Revert of cc: Enable impl-side painting by default on all platforms where threaded compositing is used. (https://codereview.chromium.org/242803007/)

Reason for revert:
This makes DevTools unusable on Linux Aura, see crbug.com/367673.

Original issue's description:
> cc: Enable impl-side painting by default on all platforms where threaded compositing is used.
> 
> BUG=169282
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266471

TBR=piman@chromium.org,vangelis@chromium.org,enne@chromium.org,reveman@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=169282

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266499 0039d316-1c4b-4281-b951-d872f2087c98

> Reland of 260688: [Files.app] Use getDriveEntryProperties() to retrieve metadata
> 
> This is a reland of 260688, which is reverted due to false charge of build failure.
> 
> Previously, we used FileEntry.getMetadata() to retrieve filesystem metadata and getDriveEntryProperties() to get Drive metadata, so we need 2 calls for 1 file on Drive.
> 
> With this patch, getDriveEntryProperties() returns not only Drive metadata but also filesystem metadata. It's enough to call only getDriveEntryProperties() and we can reduce a number of calls by half.
> 
> BUG=345196
> TEST=browser_test passes.
> R=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org
> TBR=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org
> 
> Review URL: https://codereview.chromium.org/195763004

TBR=yoshiki@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266498 0039d316-1c4b-4281-b951-d872f2087c98