GitLab

Submitting http://codereview.chromium.org/10823397 for chrisphan.

Also removed DOS line endings that seem to have been unintentionally
introduced in a previous CL.

BUG=143010

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

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

See bug link. The test flakiness is because of being randomly selected into a
suggest field trial that causes the Google-returned suggestion to be ranked
above the what-you-typed entry. This CL attempts to disable the suggest field
trial by forcing a specific group (which at the moment doesn't have any
experimental behaviour).

BUG=143308
R=mpearson@chromium.org,nirnimesh@chromium.org
TEST=This test should no longer flake on the bots. If it does, then it's likely
that group #10 has been selected into a field trial. The fix would be to
disable the suggest field trials properly (as mentioned in the TODO).

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

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

Also remote tabs that crept into the remoting.gyp file.

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

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

It just makes debugging difficult

BUG=111316
TEST=none


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

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

You can now drag tabs in and out of windows in linux_aura builds. This patch has a lot of moving parts and caveats though.

Moving parts in the patch:
- Added system location to aura::Events.
- Forwarding move events from RootWindowHostLinux to the TabDragController through an observer.
- MessagePumpAuraX11 now can block waiting for a window to be mapped. If we don't do this, we can't perform a grab on the new window.
- The drag offset is threaded from TabDragController through the WindowMoveController interface. (There's no other non-racey way of getting this data locally from X.)
- RootWindowHostLinux now has working Show()/Hide().

TODOs:
- After releasing a new window, the window looks focused, but isn't. You have to click on it for it to receive input focus.

BUG=133059
TEST=none

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

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

Adds missing parameter to FactoryGetFieldTrial() and
changes group id values to not use kConstant naming
convention since their values are dynamic.

BUG=none
TEST=none

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

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

BUG=None
Test=None

TBR=sadrul

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

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

The logic is no longer needed as we've been relying on the version number
since crrev.com/147782. Along the way, remove tests for the logic, and
make GDataEntry/GDataFile/GDataDirectory::FromProto() return void.

It's a bit sad to see gdata_files_unittest.cc gone, but gdata_files.h/cc
will also be removed in the near future anyway.

BUG=141135
TEST=file manager works as before
TBR=thakis@chromium.org
for chrome_tests.gypi

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

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

BUG=90448
Review URL: https://chromiumcodereview.appspot.com/10855231

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

BUG=


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

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

BUG=None
TBR=dale@chromium.org


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

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

TBR=jianli

BUG=None
TEST=Verify links still work in Panels.


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

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

[sync] 	Divorce DataTypeManager from NotificationService notifications by creating a new DataTypeManagerOserver interface

As of today, DataTypeManager sends out notifications for various
configure events via NotificationService notifications. This is bad,
because other services that depend on these events need to have
knowledge of DataTypeManager, which is internal to the sync
implementation.

This patch does the following:

1) Creates a new DatTypeManagerObserver interface through which
   observers are made aware of configure events.
2) Has ProfileSyncService implement the interface.
3) Gets rid of unnecessary sync configure * notification types.
4) Has PSS send out notifications for SYNC_CONFIGURE_START and
   SYNC_CONFIGURE_DONE.
5) Allows ForeignSessionHandler to listen to SYNC_CONFIGURE_DONE
   notifications from the PSS, instead of from all sources, while
   waiting to populate the "other devices" section of the NTP.
6) Updates various mock and test classes and fixes unit tests.

TBR=estade@chromium.org, thakis@chromium.org
BUG=142984,124717,111676,142550

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

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

Revert 152162 - 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

TBR=pkotwicz@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10857065

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

BUG=139442
TEST=CQ

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

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

BUG=137247,142402,141500,139683
TEST=about:gpu page
Review URL: https://chromiumcodereview.appspot.com/10832356

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

Failing on all platforms.

TBR=krenaud@google.com
BUG=143506

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

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

Tulip was incorrectly marked flaky on all platforms
due to an issue where #define and TEST_P don't interact
well (see issue 143151).

Also cleans up the disabled Flac test to use proper
DISABLED_ syntax instead of being commented out.

BUG=141020, 142896
TEST=media_browsertests.

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

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

BUG=141157
TEST=Build and boot on daisy.


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

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

These tests fail on Linux redux, that uses OpenSSL, due to
ec_private_key_openssl.cc which is not implemented yet.

They were already disabled on Android through an entry in
build/android/gtest_filter/net_unittests_disabled.

TEST=net_unittests
BUG=142833,140991

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

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

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