GitLab

. Eliminate the implementation of LayerAnimationObserver. It seems no one expects WED to be a LAO anymore.
. Remove WindowTreeHostDelegate. This involves moving some of the functions to WTH, removing some completely, and moving the remainder to the WED public API. A little icky for now, but I plan to take a pass on cleaning up WED's public API once I get it pared down.
. window_tree_host_x11_unittest provided an implementation of WTHD that was not WED. This is not a valid situation now since various functions in WTH now call directly through to WED. Replaced this with a simple EventHandler to test that events are being dispatched.

R=sky@chromium.org
http://crbug.com/308843

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

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

BUG=341607
TEST=Make sure that you have the --new-profile-management flag turned on, at least two profiles, and at least one of them signed in. Go to chrome://settings. All the users should display an avatar image in the Users section, including the signed in users, which will have their Gaia photo.

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

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

BUG=348629
TBR=padolph@netflix.com

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

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

Reason for revert:
Experimentally reverting this CL because it may have broken AshTestHelperTest.AshTestHelper on Windows Vista . Will revert the revert if this CL is innocent

See http://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%282%29/builds/41640/steps/ash_unittests/logs/AshTestHelper

Original issue's description:
> Enable the volume slider in Ash for windows.
> 
> BUG=227247
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255215

TBR=henrika@chromium.org,tommi@chromium.org,jennyz@chromium.org,dewittj@chromium.org,zturner@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=227247

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

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

Move to focusedElement() as focusedNode is deprecated and will be removed soon..

BUG=258763
NOTRY=true

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

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

The 1x resources landed here: https://codereview.chromium.org/176903008/

BUG=324052
R=oshima@chromium.org

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

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

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

In this way, more information can be added (currently added 'description' and 'owner' for each action) 

A few functions are moved from tools/metrics/histograms to tools/metrics/common to be shared by tools/metrics and tools/histograms.

BUG=340735
NOTRY=true

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

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

Revert of android: Make dex_action.gypi not call md5sum. (https://codereview.chromium.org/183883024/)

Reason for revert:
Experimentally reverting because it is possible that this CL broke the android build http://build.chromium.org/p/chromium/builders/Android/builds/21418/steps/compile/logs/stdio

Will undo the revert if this CL is innocent

Original issue's description:
> android: Make dex_action.gypi not call md5sum.
> 
> No intended behavior change.
> 
> dex_action.gypi didn't pass its input_paths variable to dex.py,
> but that was only set in a single place, and only to a single file there.
> It was only used in java_apk.gypi which does manual threading of
> stamp files to order actions (since it's a gypi, it can't easily use type
> none targets with dependencies). Since dex.py doesn't look at this
> stamp file at all, the it doesn't need to rerun when the stamp file disappears.
> 
> To make this a bit more obvious, remove dex_action.gypi's input_paths
> variable and set 'inputs' directly in the one place with the stamp file.
> (dex.py will still rerun if the name of the stamp file changes, due to
> regular timestamp handling.)
> 
> BUG=177552
> NOTRY=true
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255325

TBR=cjhopman@chromium.org,thakis@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=177552

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

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

The tap and click_element page actions have a very similar purpose but use a very
different implementation. This patch renames click_element to javascript_click
to indicate that it is not using synthetic gestures as tap does. It also replaces
click_element with tap inside smoothness measurements, so we exercise the entire
input stack. In other places we keep click_element (now javascript_click) because
tap's functionality isn't always sufficient (click elements outside of the view;
clicking on very small elements causes Chrome to magnify the element first).
Further, tap now supports all ways to select an element that click_element
supported (by text, xpath etc).

This CL also removes the tap_element page action, which was only used inside
smoothness for the pica page set. This case is currently broken and adamk@ said
he'd re-implement it using synthetic gestures if needed.

BUG=339233

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

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

Revert of Eliminate the dependence of tts extension API for tts_controller.cc (https://codereview.chromium.org/181123004/)

Reason for revert:
Breakes ChromeVox spoken feedback on OOBE.

Original issue's description:
> Eliminate the dependence of tts extension API for tts_controller.cc
> 
> This is the first patch to move TTS implementation from browser layer
> to content. To move TTS to conent, the direct dependence of browser
> compoent should be eliminated from TTS classes first.
> 
> R=tommi@chromium.org, dmazzoni@chromium.org, sky@chromium.org
> BUG=347045
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=254740

TBR=dmazzoni@chromium.org,tommi@chromium.org,sky@chromium.org,chun.gao@intel.com
NOTREECHECKS=true
NOTRY=true
BUG=347045

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

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

Allow wrapping a media::VideoFrame with a new view_rect as long as the new view_rect is within the original rect.

BUG= 349450
R=fischman@chromium.org, tommi@chromium.org

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

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

This CL introduce the core classes for handling classification of
data in a hierarchical fashion. It introduces two concepts:
- A rule tree: hierarchy of rules defined by the end-user.
- A result tree: the corresponding tree which aggregates the
memory counters.

BUG=340294
NOTRY=true

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

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

From this patch, libjingle is able to get the signal level and stats from the local audio track in chrome.

Note, that this patch works with and without kEnableAudioTrackProcessing. When kEnableAudioTrackProcessing is set, libjingle gets values from the chromium track, otherwise it will continue to use the values got from webrtc.

After https://codereview.chromium.org/178223013/ is landed, this CL will rebase and implement the GetSignalLevel() correctly.

BUG=334273
TEST=content_unittests, nothing breaks;

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

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

BUG=348850

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

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

This CL adds AnimationFrameCallback class to update the properties of progress
center items at the correct timing, and ensure it invokes the CSS animation.

Previously, we use setTimeout function to ensure changes of progress items
properties are applied to one by one, but if the number of milliseconds is less
than the frame inteval, it does not work.

BUG=348493
TEST=Make a repro function and call it multiple times.

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

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

This patch simplifies the code and removed the dependency on mount path, which was redundant after migrating to volume ids and file systems per volume.

TEST=Tested manually, mounted and unmounted a archive and a usb device.
BUG=341263

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

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

Without CoreAnimation, the bookmark bar buttons got redraw
messages whenever they were damaged (like, moved around,
made visible, etc). With CoreAnimation, the bookmark bar buttons
have their drawn contents cached and therefore need to be
explicitly told to re-draw.

The function BookmarkBarController::layoutSubviews should do
this, but it only iterates through the buttons in the buttons_ array,
which skips the "apps" and "other" bookmark buttons.

This patch adds those two back in. The general question of "where
else is the apps button assumed to be part of the buttons_ array" is
not answered.

BUG=340238
NOTRY=True

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

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

NOTRY=true

BUG=264611

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

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

BUG=314791

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

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

PageTestRunner.GetModernizedTestName was used to support old testnames to be specified, but it is not used anymore. The conversion map in tools/perf/run_measurement
 was already empty.

NOTRY=true
BUG=None

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

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

BUG=347424

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

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

BUG=297480, 331301

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

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

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

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

PNaCl is being upgraded to Clang 3.4, which now warns about unused
declarations inside anonymous namespaces.

BUG=https://code.google.com/p/nativeclient/issues/detail?id=3757
TEST=none
R=binji@chromium.org

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

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

Use MetadataDatabaseIndex as the separate index management class in MetadataDatabase.

BUG=344769
NOTRY=true
TEST=unit_tests --gtest_filter='DriveBackendSyncTest.*:MetadataDatabaseTest.*'

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

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

The black screen issue at starting frame was fixed a few months ago(https://codereview.chromium.org/92603002).
But it seems to work only for the initial Shell.
The last patch sets the background color to white, but this value is reset to 0 (black) by the VSyncAdapter after compositing the first frame.
So when a child Shell is newly opened, the black screen comes out again.
To fix this problem, setSurfaceViewBackgroundColor(Color.WHITE) needs to be moved to surfaceCreated() of SurfaceHolder.Callback.

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

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

Turns out the DACL won't apply in most cases, and may run some
risk of bad interactions with third-party software. It's also
mostly redundant given the existing job and desktop restrictions.

BUG=346586
R=cpu
TBR=cpu

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

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

Receiving messages from GCM needed separate object with lifetime tied to
TiclInvalidationService on UI thread. This change is to replace
GCMNetworkChannelDelegateImpl with pair of objects:
GCMInvalidationBridge/Core.
Here is object diagram for this refactoring: http://www.plantuml.com:80/plantuml/png/VPBRQiCm343VynKYZuNs0xgoXsLZ17PZxPwBwwYf5sC7nqiPs_vzbfsB4sWXuDYHZexar8nNh4H8FtBOdGoo8eMV9D67J-GMqftWCbSjaw9WLcYrHTCAZYcm1gpxvc816gvhBFQrDPix8ueIR-7WrvlahoSzPiyFBIhxhhyich60Eu_v8T_Z0uBuPsGLqO5hWmuP9xyPVHGIOw-oRhpMjEG2SDoG63rrttt-6dL_rMszpfo5BqGwC14CpZF55xHdRQhin9H2UOyIIsO3eUWsZd8j9rGBxxE93uW8iFrMF1P_4er5XMQUgaAHVKUceJY8lzR5Sjat4nRGgrMO_tIiiFaTG_r0AB3VTyf46po7dYdJ4-d2j0kP_ICNso1SXmwsLF9Iz3y0

BUG=325020
R=rlarocque@chromium.org

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

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

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

Chromium base revision: 255230 / bc47f7e0
Old Skia revision: 13662
New Skia revision: 13678
Control CL: https://codereview.chromium.org/188343002

This CL was created by Skia's roll_deps.py script.

Bypassing commit queue trybots:
NOTRY=true
R=rmistry@chromium.org

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

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

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

http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=168581:168617&mode=html

TBR=morrita1

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

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

It's ok to remove or add the observers after the onDestroyed is received.

NOTRY=True

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

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

Adjusts the call sequences in cc::ResourceProvider::MapImage to map image first and then call GetStride.

Current call sequences may get wrong stride for the on-going SurfaceTexture-based GpuMemoryBuffer for Android.

BUG=347890

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

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

R=nduca@chromium.org
BUG=348875,347802
NOTRY=True

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

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

This CL implemented part of chrome power profiler, including the service, data provider and observer base.

BUG=337138

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

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

Export the API CreateJavaBitmap to be utilized from other places instead of
using the BitmapHelper class.

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

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

Revert of HttpServerProperties - Implement MRU for AlternateProtocolMap. Persist (https://codereview.chromium.org/182383016/)

Reason for revert:
Broke compile on a bunch of bots:
Chromium Linux Builder (valgrind)
Chromium OS Builder (valgrind)
Chromium Windows Builder (DrMemory)
Linux Builder (TSan v2)

Log:
http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20Linux%20Builder%20%28valgrind%29/builds/73182/steps/compile/logs/stdio


Original issue's description:
> HttpServerProperties - Implement MRU for AlternateProtocolMap. Persist
> only the most recently used 200 entries. In memory keep all the entries.
> 
> BUG=258189
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255323

TBR=rch@chromium.org,rtenneti@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=258189

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

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

No intended behavior change.

dex_action.gypi didn't pass its input_paths variable to dex.py,
but that was only set in a single place, and only to a single file there.
It was only used in java_apk.gypi which does manual threading of
stamp files to order actions (since it's a gypi, it can't easily use type
none targets with dependencies). Since dex.py doesn't look at this
stamp file at all, the it doesn't need to rerun when the stamp file disappears.

To make this a bit more obvious, remove dex_action.gypi's input_paths
variable and set 'inputs' directly in the one place with the stamp file.
(dex.py will still rerun if the name of the stamp file changes, due to
regular timestamp handling.)

BUG=177552
NOTRY=true

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

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

BUG=285976

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

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