GitLab

Previously, WebContents, NavigationController, and RenderViewHost all exposed
APIs that allowed one to retrieve the SessionStorageNamespace associated with
them. This is confusing because there were too many ways to access the object.
After this change, the SessionStorageNamespace is only exposed by the
NavigationController.

Conceptually a SessionStorageNamespace belongs to a WebContents but we store
it in NavigationController because on tab duplication, the
NavigationController becomes the authoritative state of the tab.

Also, to support StoragePartitions, the NavigationController
now maintains a map of partition_id -> SessionStorageNamespace. Someone
requesting a SessionStorageNamespace must either know which StoragePartition
they are coming from, or which child process they are acting on behalf of.

This change also changes the way TabContents and WebContents are created.
1) We now have explicitly separate creation methods for prepopulating
   with SessionStorage and creating with an opener.
2) Some of the WebContentImpl construct has been moved into an Init()
   function to avoid accidental calls to virtual functions by subobjects.

TBR-ing all the directories where I just remove a NULL.

TBR=sky,stevenjb,dominich,brettw,satorux,kalman
BUG=85121

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

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

If it fails, we still want to free the OMX resources.

BUG=chrome-os-partner:12316
TEST=by hand: play a video, look for buffer leaks.

Change-Id: Ifcc4fe4248b72067e4592224b24fdb716ab4b230


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

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

Instead of creating a fake touch-cancelled event and sending that to the
gesture recognizer, send the raw touch-release event with the status of
the event-processing. Update the gesture recognizer (GestureSequence)
transitions to handle these events correctly instead.

BUG=none

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

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

This page pulls the apps samples from github and constructs a page similar to
the extensions samples page. The links take you to github instead of
downloading zip files.

BUG=140252

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

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

By the time CloseFileOnUIThread() is called, the feed is already fetched
so there is no need to use GetEntryInfoByPathAsyncOnUIThread() which
initiates the feed loading as needed.

BUG=none
TEST=none

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

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

Linux/GTK has been using canvas_skia.cc since M21 and
ChromeOS has been using canvas_skia.cc since M19.

BUG=105550
TEST=compiles

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

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

The number of callers of FindEntryByPathSync() is reduced from 9 to 7.

BUG=137694, 126634, 139446
TEST=Moving a file to another directory and renaming in the same directory work as before from file manager.

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

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

TEST=functional/navigation.py navigation.NavigationTest.testTabsCloseOpen
BUG=None


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

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

Make Notification::ToString() base64 the data member (since it can hold
binary data).

Convert VLOGs to DVLOGs for push_notifications_*.cc.

BUG=141692


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

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

There was a place where CloseFile() is called with a null callback, but
with the caller fixed, it's safe to make the callback parameter mandatory.

Along the way, removed unused |file_path| parameter from
DataFileSystem::OnGetEntryCompleteForCloseFile()

BUG=126634
TEST=out/Release/unit_tests --gtest_filter=GData*

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

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

BUG=151345

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

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

CredentialCacheService on Windows 8 writes to the local credential cache
when a user actively signs in / reconfigures sync. Existing cached
credentials are not updated when Chrome is merely restarted after the
user is signed in.

This causes a problem when a user is already signed in, and then
upgrades (and restarts) chrome from a version that didn't originally
support credential caching. In such cases, we never end up caching
credentials, and therefore, the user will have to sign in separately to
Metro and Desktop.

This patch contains the following changes:

1) Adds logic to auto-heal already-signed-in users who upgrade from
older versions, by writing existing credentials to the local cache if
during restart, we notice that there is no local cache file, and the
user is already signed in to sync.

2) Simplifies the logic around checking if an alternate credential cache
file exists, and only then initializing |alternate_store_|. It turns out
that JsonPrefStore returns a useful PrefReadError field, and there is no
need for CCS to do funky stuff on the FILE thread.

3) Simplifies OnInitialzationCompleted, which was being used to observe
two separate JsonPrefStores. Instead of having CCS be a
PrefStore::Observer, we now use two helper classes -- LocalStoreObserver
and AlternateStoreObserver to cleanly divide what is done when each pref
store is initialized.

4) Updates prefs::kGoogleServicesUsername by listening to the notifications
NOTIFICATION_GOOGLE_SIGNED_OUT and NOTIFICATION_GOOGLE_SIGNIN_SUCCESSFUL
instead of directly listening to the pref change.

5) Fixes a stray instance where we were accessing the gaia username
pref via SyncPrefs instead of via the SigninManager.

BUG=141555
TEST=Sign in to chrome, exit the browser, and delete "Sync Credentials" from the default profile directory. Restart Chrome and make sure that the credential cache file is freshly written using existing sync credentials.

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

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

Add JellyBeanAccessibilityInjector/JellyBeanContentView to support JellyBean gesture
accessibility features

BUG=http://crbug.com/138218


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

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

http://crbug.com/125937
TBR=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10828265

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

It is possible that a window starts doing synchronous touch-event handling
after doing asynchronous handling for some time. As a result, these sync
events may be processed before the acks for the queued asynchronous events
have been processed. This results in unexpected behaviour. So in such cases,
queue up the synchronous events so that they get processed only after the
asynchronous events are processed.

BUG=139339, 134486

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

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

If a touch-event happens outside of the root-window bounds, then dispatch
the touch-events to the root-window. This makes sure that the event-filters
(installed on the root-window or env) get these touch-events.

BUG=138846

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

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

By the time CreateFileOnUIThread() is called, the feed is already fetched
so there is no need to use FindEntryByPathAsyncOnUI() which initiates the
feed loading as needed.

Along the way, make callback parameter of CreateFile() mandatory.

BUG=142420,126634
TEST=editing a photo from Files.app works as before.

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

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

BUG=127323
TEST=Enabled tests for refactored panels on Linux


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

TBR=jennb@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10828289

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

Rename all of experiments_helper to variations_helper, including files, namespaces, and various function names.

TEST=none
BUG=none
Review URL: https://chromiumcodereview.appspot.com/10795060

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

We were uploading when failing to associate the synced bookmarks node,
regardless of whether we expected it or not. Now we do the upload
at a layer above, where we know whether we care or not.

BUG=142387
TEST=Signing into sync on an account with no mobile devices should not
trigger an unrecoverable error upload.


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

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

R=rsleevi@chromium.org
BUG=none
TEST=none

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

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

R=gab
BUG=138615
TEST=setup.exe --multi-install --chrome; setup.exe --multi-install --uninstall --chrome; (check 'Delete user profile' option); verify C:/Users/<you>/AppData/Local/Google/Chrome is gone.

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

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

BUG=127323
TEST=Enabled tests for refactored panels on Linux


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

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

This context3d is meant to be created on the render thread but can only be
dereferenced on the compositor thread.  Before this CL this was violated as
RenderThreadImpl::GetGpuVDAContext3D() checked for context loss to decide
whether to recreate the context.  Now context loss proactively notifies the
render thread, instead.

BUG=142047


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

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

of DownloadFileImpl by DownloadItemImpl.

BUG=123998
R=benjhayden@chromium.org


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

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

BUG=110111


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

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

BUG=140037


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

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

This reverts commit 145978.

BUG=136462,142388
TBR=rch@chromium.org

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

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

Pings weren't sent, so this information was only collected locally.
However, other parts of the RLZ code don't collect events for non-branded
builds at all, so don't do it here either.

BUG=141108,138158


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

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

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

It's yet unclear how the GR receives two press-events for the same
touch-point without a release/cancel-event before the second press,
but when that does happen in a release build, try to handle it
gracefully instead of crashing.

BUG=134486

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

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

chrome and skia.
Review URL: https://chromiumcodereview.appspot.com/10830286

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

R=gavinp@chromium.org,darin@chromium.org

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

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

E.g. this makes the video_decode ppapi example not error out at EOS.

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

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

This is a patch taken over from zel's original patch:
http://codereview.chromium.org/10832258/

BUG=131915
TEST=NetworkScreenTest.* and number of other tests should start passing in BVT CrOS tests
TBR=zelidrag@chromium.org

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

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

BUG=142380


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

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

The ant build files for each of the jars that we create consisted of
only fairly straightforward boilerplate. With the recent changes to
java.gypi and java_aidl.gypi we can now build these correctly from a
single .xml with GYP passing in defines for the minor differences.

BUG=136756


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

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

BUG=141570

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

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

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

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

BUG=141932

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

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