- 14 Aug, 2012 4 commits
-
-
tbarzic@chromium.org authored
This is needed to support hidpi for browser/page actions. BUG=138025,135271 TEST=manually checked that browser and page action buttons are displayed correctly. Review URL: https://chromiumcodereview.appspot.com/10827191 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151399 0039d316-1c4b-4281-b951-d872f2087c98
-
simonmorris@chromium.org authored
When user signs out and transit to the login screen, the wallpaper fade in from a white screen. Described in 141563, this Cl fixed it. Also, after user login, or switch users at login screen. when wallpaper change, the new wallpaper should fade in nicely. There is a regression in recent builds. This fix fixed the regression as well. BUG=141563, 141676, 142042 Review URL: https://chromiumcodereview.appspot.com/10827284 TBR=bshe@chromium.org Review URL: https://chromiumcodereview.appspot.com/10854136 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151398 0039d316-1c4b-4281-b951-d872f2087c98
-
marcheu@chromium.org authored
If we do this, the OMX_FillThisBuffer function will fail, and will put the OMX class in error mode. Then resource freeing will not happen. This is especially annoying since this can happen during Destroy and leads to resource leaks. BUG=chrome-os-partner:12316 TEST=by hand: play a video, look for buffer leaks. Change-Id: If14311d188f1a2048c11b5bb6186dbf5fc20c1f5 Review URL: https://chromiumcodereview.appspot.com/10837212 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151396 0039d316-1c4b-4281-b951-d872f2087c98
-
benjhayden@chromium.org authored
DownloadsDOMHandler::download_items_ does not always contain all of the items that it observes, so std::find may return end(). I have no idea what happens when you set *end()=NULL, but it could explain the crash in DownloadsDOMHandler::ClearDownloadItems() on windows, somewhere in "for (OrderedDownloads::iterator it = download_items_.begin(); it != download_items_.end(); ++it)". BUG=142389 Review URL: https://chromiumcodereview.appspot.com/10855135 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151395 0039d316-1c4b-4281-b951-d872f2087c98
-
- 13 Aug, 2012 36 commits
-
-
bshe@chromium.org authored
When user signs out and transit to the login screen, the wallpaper fade in from a white screen. Described in 141563, this Cl fixed it. Also, after user login, or switch users at login screen. when wallpaper change, the new wallpaper should fade in nicely. There is a regression in recent builds. This fix fixed the regression as well. BUG=141563, 141676, 142042 Review URL: https://chromiumcodereview.appspot.com/10827284 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151393 0039d316-1c4b-4281-b951-d872f2087c98
-
stevet@chromium.org authored
Update unit tests to reflect these changes. BUG=140732 TEST= Review URL: https://chromiumcodereview.appspot.com/10806065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151391 0039d316-1c4b-4281-b951-d872f2087c98
-
vandebo@chromium.org authored
Handle some todo's minor performance issues and some corner cases. BUG=NONE Review URL: https://chromiumcodereview.appspot.com/10836199 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151390 0039d316-1c4b-4281-b951-d872f2087c98
-
rlarocque@chromium.org authored
This is part of a series of patches intended to add more types which behave similarly to Nigori. These new types will be downloaded early, much like the Nigori node currently is. Removing these flags will make it easier to calculate the appropriate parameters for configure calls during early initialization. BUG=122825 Review URL: https://chromiumcodereview.appspot.com/10854050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151389 0039d316-1c4b-4281-b951-d872f2087c98
-
binji@chromium.org authored
This broke the mono waterfall BUG=none TBR=noelallen@chromium.org NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10824282 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151388 0039d316-1c4b-4281-b951-d872f2087c98
-
nduca@chromium.org authored
Creates the WebCompositorOutputSurface, which is the new mechanism for rendering interactions between the compositor and RenderWidgetHost. As part of this, we plumb vsync information to the compositor. The new IPC::ForwardingMessageFilter is modified to have route-specific handlers rather than a global handler. This simplifies message routing considerably. BUG=129674 Review URL: https://chromiumcodereview.appspot.com/10798006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151387 0039d316-1c4b-4281-b951-d872f2087c98
-
cduvall@chromium.org authored
BranchUtility was crashing when it tried to fetch the branch numbers for the dev, beta, and stable channels. There was also a problem using the branch number where the name should have been used. ServerInstances are no longer leaked. BUG=141909 Review URL: https://chromiumcodereview.appspot.com/10831269 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151386 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151385 0039d316-1c4b-4281-b951-d872f2087c98
-
viettrungluu@chromium.org authored
TBR=mstarzinger@chromium.org Review URL: https://chromiumcodereview.appspot.com/10829304 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151384 0039d316-1c4b-4281-b951-d872f2087c98
-
xiaomings@google.com authored
The original code is correct. The fix is a work around for mips gcc 4.3.2. The constructor of Callback is a template. In the context of the constructor, there are two type named RunType. One is its second template parameter, the other is a typedef inside Callback. The correct compiler will pick the "local" one, i.e. the template parameter. However, gcc 4.3.2 on mips incorrectly uses the typedef. The solution is to rename the template parameter. After discussing with ajwong, we decide to push the work around up stream as it is cleaner not to shadow names anyways. BUG= Review URL: https://chromiumcodereview.appspot.com/10836215 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151383 0039d316-1c4b-4281-b951-d872f2087c98
-
vandebo@chromium.org authored
BUG=141322 Review URL: https://chromiumcodereview.appspot.com/10855132 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151382 0039d316-1c4b-4281-b951-d872f2087c98
-
yusukes@chromium.org authored
Add a unit test which verifies if the resizer sets/resets DisplayController::dont_warp_mouse_ as expected. BUG=None TEST=try Review URL: https://chromiumcodereview.appspot.com/10826240 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151381 0039d316-1c4b-4281-b951-d872f2087c98
-
craigdh@chromium.org authored
Converts the old bookmark automation commands. TEST=PyAuto CONTINUOUS suite BUG=132794 Review URL: https://chromiumcodereview.appspot.com/10828245 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151380 0039d316-1c4b-4281-b951-d872f2087c98
-
ajwong@chromium.org authored
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
-
marcheu@chromium.org authored
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
-
sadrul@chromium.org authored
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
-
cduvall@chromium.org authored
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
-
satorux@chromium.org authored
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
-
asvitkine@chromium.org authored
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
-
satorux@chromium.org authored
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
-
craigdh@chromium.org authored
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
-
akalin@chromium.org authored
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
-
satorux@chromium.org authored
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
-
thakis@chromium.org authored
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
-
rsimha@chromium.org authored
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
-
dtrainor@chromium.org authored
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
-
ben@chromium.org authored
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
-
sadrul@chromium.org authored
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
-
sadrul@chromium.org authored
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
-
satorux@chromium.org authored
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
-
simonmorris@chromium.org authored
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
-
stevet@chromium.org authored
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
-
zea@chromium.org authored
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
-
wtc@chromium.org authored
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
-
erikwright@chromium.org authored
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
-
jennb@chromium.org authored
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
-