GitLab

Intended for the beta channel.

BUG=143505
TEST=none
TBR=willchan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10857064

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

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

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152195 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@152193 0039d316-1c4b-4281-b951-d872f2087c98

BUG=143440

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

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

In order to call SendSAS() successfully the calling process's binary has to:
 - be signed.
 - be installed under %ProgramFiles% or %SystemRoot%\system32.
 - specify uiAccess='true' in the manifest.
 - run under SYSTEM or the currently logged-on user.

All of those requirements are met now, so SendSAS() can be called directly.

BUG=135217

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

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

Need to get window.open() working for browserless panels before baking in canary/dev.

BUG=143506
TEST=None

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

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

BUG=142716

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

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

Also suppress two warnings in third_party targets.

gyp changes reviewed by willchan in
http://codereview.chromium.org/10832369/

BUG=141532,138571
TBR=hans,willchan

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

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

beginSmoothScroll tries to do a simple animation for a given length in seconds, specified as a double. To do a half-second animation, it did base::TimeDelta::FromSeconds(0.5). base::TimeDelta::FromSeconds takes an in64, so this silently converts to FromSeconds(0) causing the smooth scroll to do exactly one step and stop.

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

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

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