GitLab

Change HistoryAndBookmarkRow favicon_ field to be scoped_refptr<base::RefCountedMemory> instead of std::vector<unsigned char>

Bug=138553
Test=Compiles

R=michaelbai
TBR=sky

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

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

to tell when the windows platform is 64-bit.

PNaCl needs to use 64-bit nexes in that case, even if
chrome is 32-bit.

Retry of: http://codereview.chromium.org/10834348/
but this time with baseurl...

BUG= http://code.google.com/p/nativeclient/issues/detail?id=2365
TBR=cpu@chromium.org


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

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

Previously "continue" statement left renderer_type undefined.

BUG= 140305
TEST= open chrome://memory page if NaCl extension is opened.

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

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

Since these classes are small, they can be all be moved in one cl without it becoming overly large.

This one moves:
- AppShortcutManager
- AutoUpdateInterceptor
- ConvertWebApp
- ExtensionActivityLog (now extensions::ActivityLog)

BUG=117261
TBR=jam@chromium.org, willchan@chromium.org

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

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

Also add test.

BUG=142925

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

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

CredentialCacheService listens for initial sync configuration during
sign in by directly observing sync preferences in the pref store. This
is not good because sync preferences are not always re-written
during sign in, and we cannot rely on NOTIFICATION_PREF_CHANGED.

In addition, if during startup, we find an older version of a credential
cache with some fields missing (like the last_updated_time), we could
run into bugs due to the missing fields.

This patch does the following:

1) Makes CredentialCacheService listen to the notification
   NOTIFICATION_SYNC_CONFIGURE_START for sign-in, restart and
   reconfigure scenarios.
2) No longer relies on NOTIFICATION_PREF_CHANGED for detecting
   changes to sync datatype preferences.
2) Freshly writes any missing sync credentials during startup to
   auto-heal the cache in case chrome crashed during a previous run,
   or if a signed-in user upgraded chrome from an older version that
   didn't support credential caching or was missing some newer fields.
3) Adds caching support for both sync encryption tokens:
   kSyncEncryptionBootstrapToken and kSyncKeystoreEncryptionBootstrapToken,
   and makes the required logic changes to sign in / reconfigure with
   one or the other.
4) Modifies PackAndUpdateStringPref and UpdateBooleanPref to only
   update the cache if the new value being written is different from the
   existing value.
5) Other misc. changes to support the design described above.

BUG=142550, 143214
TEST=Sign in to sync, sign out, exit chrome, delete the credential cache from the profile directory or merely delete some fields, and sign in again. All sync preferences must get written to the credential cache.

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

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

Add a regression test to verify this.

BUG=142872
TEST=Ensure that bug 142872 is no longer reproducible.

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

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

This does not match Safari 6, Spotlight, or Chrome omnibox behavior on other
platforms.

BUG=142887


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

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

The number of callers of FindEntryByPathAndRunSync() in GDataFileSystem
is reduced from 2 to 0, hence the function is finally removed from
GDataDirectoryService.

Along the way, GDataFileSystem::FindEntryByPathAsyncOnUIThread() is
simplified and renamed to LoadIfNeeded().

Remove GDataFileSystem::FindEntryByPathSyncOnUIThread() as well.

BUG=141196, 142420
TEST=file manager works as before

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

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

- Removed unnecessary List.redraw calls from
      -- network_list.js
      -- autofill_edit_address_overlay.js
      -- content_settings_exceptions_area.js
- Added an early return to List.redraw() for the case clientHeight == 0 (eg. if the list is not visible).
- Some small cleanup.

Overall there were about 68 expensive (ie. no early return) calls to redraw() which I could reduce to 26. The speedup is about 7% / 28ms for the "onload" event on my desktop machine.

BUG=chrome:143116


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

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

BUG=141826
TEST=Manual test.


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

TBR=serya@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10837318

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

* GDataEntry must always be constructed with a valid directory_service ptr. Add a DCHECK for this, and remove runtime tests for null directory_service_ ptr in all methods.
* RemoveEntryFromResourceMap takes resource_id instead of GDataEntry*.
* RemoveEntryFromResourceMap ensures resource_id is non-empty and that an actual deletion occurs every time.
* AddEntryToResourceMap has a DCHECK to guard against multiple additions, or addition of empty resource_id.
* Remove accessors child_files() and child_directories() and make typedefs for GDataFileCollection and GDataDirectoryCollection private.
* Move GetChildDirectoryPaths from gdata_wapi_feed_processor.cc anon namespace to GDataDirectory.
* Fixes for unit tests that added entries with null resource ids, and duplicate resource ids.

BUG=137725
TEST=unit tests.
Review URL: https://chromiumcodereview.appspot.com/10854199

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

This script automates Chrome Endure setup process and
starts a local Http server to serve the results.

BUG=None
TEST=Ran the script on two Linux_x64 machines.
NOTRY=TRUE


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

TBR=fdeng@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10860020

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

Also updated me2me_virtual_host.py to use that signal to reload config (instead of restarting the host), and when requesting the currently running instance of the script to reload the config (instead of SIGUSR1).

BUG=120950

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

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

BUG=142993


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

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

BUG=141779


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

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

BUG=122753

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

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

NOTRY=True
R=maruel@chromium.org
BUG=None
TEST=None


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

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

BUG=129471


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

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

BUG=111316
TEST=none


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

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

This should get rid of the spurious ERROR messages when closing tabs, and log backtraces when we need them.

BUG=141055


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

TBR=bauerb@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10854207

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

I'm hoping this will reduce CL review spam for me.

R=jam@chromium.org


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

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

BUG=116317
TEST=compiles on linux heap checker bots
Review URL: https://chromiumcodereview.appspot.com/10824356

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

This CL removes UserManager::Get()(for ephemeral user check) out from
WallpaperManager::SaveUserWallpaperProperties(called when initializing UserManager).
It was creating a deadlock and make chrome hang on startup screen.

The ephemeral user check in SaveUserWallpaperProperties is not correct either.
We should not check if current logged in user is ephemeral user while we try
to save wallpaper properties for some other user specified by email.

BUG=142440

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

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

This makes the system tray the first child so that it is the first tray focused when the status area widget is selected via accessibility.

This depends on http://codereview.chromium.org/10823350/

BUG=142506
For ash/ash_strings.grd:
TBR=sky@chromium.org

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

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

TBR=bauerb@chromium.org
BUG=143431
TEST=waterfall.sh

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

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

This script automates Chrome Endure setup process and
starts a local Http server to serve the results.

BUG=None
TEST=Ran the script on two Linux_x64 machines.
NOTRY=TRUE


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

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

- My earlier CL
  https://chromiumcodereview.appspot.com/10823382
  is problematic.  This mostly reverts it and fixes the
  issue in a better way.

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

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

This is the cocoa version of the media galleries permission dialog. The dialog is implemented as an NSAlert displayed as a constrained window. The gallery checkboxes are added inside a accessory view.

The views and GTK versions are here:
http://codereview.chromium.org/10828166/
https://chromiumcodereview.appspot.com/10826129

Screenshots: http://i.imgur.com/fedm0.png

BUG=134929
TBR=mihaip@chromium.org

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

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

I reused LazyBackgroundTaskQueue so that this works for regular background
pages, which may not be fully loaded by the time this event is dispatched.

BUG=132328

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


Reason for revert: introduced a frequent crash

TBR=mpcomplete@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10836317

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

BUG=143255


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

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

This prevents Gyp from performing unnecessary build actions due to Ant
not updating the modification time of the output when Gyp believes the
output is stale.

BUG=143229


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

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

Note that setup is already configured to use chrome.7z directly if chrome.packed.7z is absent.

Change setup to have it copy chrome.7z to the <version>\Installer folder (instead of moving it) for component builds (to allow developers to test installs more than once with the same archive).

BUG=None
TEST=Install/uninstall chrome after copying setup.exe to another machine using tools\win\copy-installer.bat
Test non-component build and make sure archive is still moved (i.e.not copied).

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

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

NOTRY=true

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

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

Re-land 151364 - [sync] Auto-create credential cache for users who are already signed in and go on to upgrade Chrome

Originally reverted due to crashes in GetLastUpdatedTime(). See crbug.com/143214. Turns out 151364 wasn't the root cause. Feature has been disabled via switch on canary. Root cause fix is under review in a separate patch. Re-landing 151364.

Original description:

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,143214
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.

Original review URL: https://chromiumcodereview.appspot.com/10830239
First revert review URL: https://chromiumcodereview.appspot.com/10830362
TBR=zea@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10834395

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

BUG=140185


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

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

- Install the host binaries to versioned folders.
- Drop references to the COM objects quickly.
- Auto-generate CLSID of the daemon controller at GYP time. This makes sure that a freshly installed version of the controller will be launched after upgrade even if there is already running instance cached by RPCSS.

This CL restores https://chromiumcodereview.appspot.com/10831037 (r148671) and https://chromiumcodereview.appspot.com/10837087 (r149866) previously reverted by r150494. Unlike the previous two CLs this one does not require GYP changes (that is changes to GYP itself, not .gyp files).

BUG=137755


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

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

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

```---------------------------------------------------------------------
r1472 | sbc@chromium.org | 2012-08-16 15:17:56 -0700 (Thu, 16 Aug 2012) | 7 lines

fix link deadlock when setting LINK.host to $(LINK)

This resulted in double call to flock on linux/make.

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

Review URL: https://chromiumcodereview.appspot.com/10854195
```

---------------------------------------------------------------------
r1471 | iannucci@chromium.org | 2012-08-15 17:05:50 -0700 (Wed, 15 Aug 2012) | 6 lines

Fixes msvs generator for projects with non-default IntermediateDirectories.

Issue was introduced by r1466.

R=bradnelson,scottmg
Review URL: https://chromiumcodereview.appspot.com/10832322
------------------------------------------------------------------------
r1470 | sbc@chromium.org | 2012-08-15 10:11:26 -0700 (Wed, 15 Aug 2012) | 9 lines

Honor $CC_target and friends in make generator.

These get baked into the generated build file if they
are set at gyp time.

BUG=
TEST=compiler-override

Review URL: https://chromiumcodereview.appspot.com/10833021
------------------------------------------------------------------------
r1469 | iannucci@chromium.org | 2012-08-14 13:53:47 -0700 (Tue, 14 Aug 2012) | 17 lines

Prepend 'call' to all non-cygwin commands to prevent incredibuild from aborting
the scripts early.

There are three cases:
  * Command is a batch file
  * Command is an exe/com
  * Command is a builtin (to cmd.exe).

In the first case, call will prevent the overall command action from aborting
when multiple commands are attached to the same file (look for 'and also' in the
msvs generator).

In the second, call does not change the exe's behavior

In the third case, call still performs the requested action correctly. This even
applies to the 'call' builtin, so, 'call call echo foo' is still correct.
Review URL: https://chromiumcodereview.appspot.com/10824293
------------------------------------------------------------------------
r1468 | thakis@chromium.org | 2012-08-13 15:43:17 -0700 (Mon, 13 Aug 2012) | 10 lines

Fix the Android Eclipse CDT generator.

This fixes a couple of problems:
- CONFIGURATION_NAME is now getting set. This seems to be a required
variable now.
- Macro values are now escaped before getting written to the XML out
in order to handle macros with XML syntax.

Review URL: https://chromiumcodereview.appspot.com/10823266/
Patch from Jesse Greenwald <jgreenwald@chromium.org>!
------------------------------------------------------------------------
r1467 | iannucci@chromium.org | 2012-08-10 14:38:59 -0700 (Fri, 10 Aug 2012) | 3 lines

This makes it easier to test combinations of msvs version without having to
muck with environment variables
Review URL: https://chromiumcodereview.appspot.com/10830232
------------------------------------------------------------------------
r1466 | iannucci@chromium.org | 2012-08-09 14:33:12 -0700 (Thu, 09 Aug 2012) | 12 lines

Fixes an issue where OutputDirectory and IntermediateDirectory were being
incorrectly generated without a trailing slash on MSVS 2008.

This only manifested as an issue when running non-cygwin actions under MSVS2008,
where the symptoms would be that a tool using OutDir would write to directories
such as "Releaseobj\\some\\subdir" instead of "Release\\obj\\some\\subdir".

Unfortunately, the wrong path usually wouldn't cause the tool to _fail_, except
when a subsequent step tried to read the output of the previous tool, causing the
secondary tool to fail with confusing errors (reporting that the correct path
could not, in fact, be read).
Review URL: https://chromiumcodereview.appspot.com/10831183
------------------------------------------------------------------------
r1465 | iannucci@chromium.org | 2012-08-09 12:36:02 -0700 (Thu, 09 Aug 2012) | 16 lines

Fix usage of shlex in unit tests on windows.

When shlex was getting an arg like:
  r'"some\interesting\path.file" other_file'

shlex was turning it into:
  ['someinterestingpath.file', 'other_file']

Which _happened_ to work, putting 'someinterestingpath.file' in the working
directory.

This patch doubles up all backslashes prior to splitting, to make the incoming
string conform to shlex's expectations

R=scottmg
Review URL: https://chromiumcodereview.appspot.com/10829252
------------------------------------------------------------------------
r1464 | scottmg@chromium.org | 2012-08-09 11:20:22 -0700 (Thu, 09 Aug 2012) | 4 lines

Use flock-style lock on Windows to limit number of concurrent links

This is mostly for goma builds.
Review URL: https://chromiumcodereview.appspot.com/10837168
------------------------------------------------------------------------

R=cmp, scottmg, Sam Clegg, Nico
NOTRY=true

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

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

BUG=139444
TEST=manually open the page and check if it shows the amount of local free space.


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

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