GitLab

A BrowserPluginPropertyBinding is bound to a particular property name, so there
is no point to passing in the property name on GetProperty and SetProperty.

BUG=None


Review URL: https://chromiumcodereview.appspot.com/11299279

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

BUG=158821

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

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

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

This is needed before we can land:

https://codereview.chromium.org/11412233/

BUG=

Review URL: https://chromiumcodereview.appspot.com/11316232

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

Before focusing did not work, which was especially noticeable for users using ChromeVox or those using keyboard only. This patch resolves following issues:
* Automatically focus the default button (depending on the screen).
* Focus the cancel button by default in the reset screen.
Besides, now it is possible to focus any control by default, not only buttons.

TEST=Turn on ChromeVox and navigate through the oobe screens.
BUG=161896


Review URL: https://chromiumcodereview.appspot.com/11421111

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

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

This page set should be executed with a mobile user agent.

BUG=
TEST=./tools/perf/run_multipage_benchmarks --browser=system smoothness_benchmark tools/perf/page_sets/key_mobile_sites.json


Review URL: https://chromiumcodereview.appspot.com/11299273

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

TBR=ben@chromium.org
BUG=155525


Review URL: https://chromiumcodereview.appspot.com/11280115

Reason for revert: A review comment after commit showed that the
change is probably wrong (or at least changes behavior in a way that
was not intended). Reverting for now, until there is time to figure
out the correct change.

TBR=joi@chromium.org
Review URL: https://codereview.chromium.org/11308295

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

TBR=bartfab@chromium.org
BUG=none

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

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

Note that the feature is still disabled in WebKit and it will remain so until we finalize the UI.

TBR=thakis
BUG=135771


Review URL: https://chromiumcodereview.appspot.com/11316153

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

Just adding the API first to unblock other CLs and so
we can start to use the API. Async tasks are just
done synchronously, and the completion query is set
immediately when it executes in the GPU process.

BUG=161337


Review URL: https://chromiumcodereview.appspot.com/11412232

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

Right now, we use 0.025 * bounds as the pivot, which basically amounts to the
upper left corner of the window. I don't think that was intended. I think that
we mean to scale about the center of the window when hiding.

BUG=163385


Review URL: https://chromiumcodereview.appspot.com/11411283

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

This CL introduces two new concepts:
- API permissions that cannot be found in the manifest (and therefore
  need not be in the optional permissions section to be optionally
  requested)
- optional API permission requests that have associated data

BUG=119473
TBR=sky@chromium.org


Review URL: https://chromiumcodereview.appspot.com/11413099

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

base::lock provides a simple detector for single-thread deadlock in
debug mode.  Implement a similar approach for the proxy lock in debug
or release.

BUG=161286


Review URL: https://chromiumcodereview.appspot.com/11280041

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

This fixes the hudLayer on retina displays, where the FPS counter appears offscreen and the paint rects at wrong locations.

BUG=163014


Review URL: https://chromiumcodereview.appspot.com/11415208

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

BUG=163630
TBR=jam@chromium.org
NOTRY=True

Review URL: https://chromiumcodereview.appspot.com/11348324

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

This is probably overlooked when ShowRootWindow is added.

BUG=none
TEST=none


Review URL: https://chromiumcodereview.appspot.com/11412259

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

Local change slipped in to commit

BUG=163653
TBR=frankf@chromium.org

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

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

This is so it can be run optionally on the trybots.


Review URL: https://chromiumcodereview.appspot.com/11299263

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

This CL extends the UserManager to handle public accounts defined through
policy. User pods are dynamically added and removed when the list of
public accounts in policy changes. Any data belonging to obsolete accounts
is also removed, taking care not to remove it prematurely if a user is
currently logged into the account.

The CL also makes the user list handling more robust by checking for
duplicate entries in the user list prefs and logging these as errors.

The pods added for public accounts are not functional yet. The login flow
for public accounts will be the topic of another CL.

BUG=158509


Review URL: https://chromiumcodereview.appspot.com/11419184

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

BUG=161603
TEST=bubbles have shadows


Review URL: https://chromiumcodereview.appspot.com/11418244

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

This fixes the use after free problem that can occur if the Seek
or Read request on the worker thread runs after the job has been
deleted.

TEST=AndroidWebViewTests,android_webview_unittests
BUG=None
R=mnaganov@chromium.org


Review URL: https://chromiumcodereview.appspot.com/11428052

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

TBR=sky@chromium.org
BUG=145623
TEST=none

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

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

* New argument --bundle-version, can be passed multiple times. The updater will
use the version passed in.
* The updater won't upload a manifest if it hasn't changed.
* The updater won't rollback a bundle (i.e. if the automatically determined
  revision is older, it will keep the current bundle).
* Switch --dryrun to use difflib instead of system diff.
* Add equality testing for manifest_util.SDKManifest. It already existed for
  Bundles.

BUG=none
R=sbc@chromium.org
TBR=noelallen@chromium.org
NOTRY=true


Review URL: https://chromiumcodereview.appspot.com/11421138

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

This patch sets a missing accessible name for the buttons.

BUG=152089


Review URL: https://chromiumcodereview.appspot.com/11421167

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

BUG=


Review URL: https://chromiumcodereview.appspot.com/11413065

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

Remove GetExtensionService calls and replace with PKSF calls through the ExtensionSystem. Extension services were made into ProfileKeyedServices a while ago; this replaces the deprecated calls that ran directly through the Profile object.

This is about half of the total GetExtensionService calls in the codebase, but this CL is already ridiculously large.

BUG= http://code.google.com/p/chromium/issues/detail?id=104095


Review URL: https://chromiumcodereview.appspot.com/11365181

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

There is no usage of selection_item_id and radio grouping logic is in engine.
So it is safe to remove selection_item_id from Chrome.

I checked property behaviors and there is no regression on Japanese/Simplified Chinese/Traditional Chinese/Hangul.
(including radio handling, property order, activation handling etc...)

I also checked any properties in all m17n engines, but there is no property.

BUG=None
TEST=Manually done on lumpy


Review URL: https://chromiumcodereview.appspot.com/11416261

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

In Chrome ASH on Windows 8 the ASH process could be closed with open browser windows. Make sure that we close them before
uninitializing ASH as that causes DCHECKs/crashes.

Added a local helper function CloseOpenAshBrowsers which runs through the list of open ASH browsers and closes them.

BUG=151718
R=sky
Review URL: https://codereview.chromium.org/11434013

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

BUG=
Android only whitelist change. WebView license check step passes
NOTRY=true
TBR=torne@chromium.org

Review URL: https://chromiumcodereview.appspot.com/11414263

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

BUG=163111
TEST=ran chromeos_unittests

Review URL: https://chromiumcodereview.appspot.com/11416243

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

BUG=157918
TEST=Manually on device

Review URL: https://chromiumcodereview.appspot.com/11316214

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

That patch speeds up Google Maps use-case smoke test by a factor of 10x.
It also reduces memory pressure on the GPU. See the related bugs for more info.


BUG=161434,160871


Review URL: https://chromiumcodereview.appspot.com/11316171

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

This will hide the popup when the browser changes size or location. Also add some tests for the WidgetObserver for the function we are observing with.


BUG=


Review URL: https://chromiumcodereview.appspot.com/11308124

TBR=csharp@chromium.org
Review URL: https://codereview.chromium.org/11416286

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

This should be a good speedup once checking for if a matrix is identity is faster  (i.e. https://codereview.appspot.com/6854113/)- otherwise it's a bit of a wash.

BUG=

Review URL: https://chromiumcodereview.appspot.com/11434027

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

* View system calls onDetachedFromWindow after HTMLViewer already
calle destroy so need a check in onDetachedFromWindow.
* PageID in restored entries needs to be reset (0-indexed).
* Need to call NavigationController::LoadIfNecessary after calling
Restore to actually load the restored page.
* Need to call onTitleUpdated callback since title is restored and
the callback is optimized out inside chromium.

BUG=
TBR=joth@chromium.org
Android only change. Ran through android trybots.
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/11428070

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

TBR=ben@chromium.org
BUG=155525


Review URL: https://chromiumcodereview.appspot.com/11280115

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

Add focus rules for Ash. They are empty for now. I will begin migrating code over from ActivationController once I have a flag to turn this on.

http://crbug.com/162100
R=sky@chromium.org
Review URL: https://codereview.chromium.org/11414252

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

This uncovered an old bug in Window - if the removal results in a restack, the iterator is invalid. Fortunately we can just defer the iterator creation until after the notification.

With the restacking one activation test expecation also changes.

http://crbug.com/162100
R=sky@chromium.org
Review URL: https://codereview.chromium.org/11348314

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

It is mostly unused, except for one place in RenderWidgetHostViewAura, where depending on the event type etc it will have a consequence. I think it's bad for a method with a const-sounding name like "ShouldActivate" to have side-effects.

Instead, I made the RWHVA store the data that would indicate a pointer activate and check that state during OnActivated(). It'd be a little cleaner if OnActivated had some way of knowing what the currently processed input event was, hence the TODO.

http://crbug.com/162100
R=sadrul@chromium.org
Review URL: https://codereview.chromium.org/11419251

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