GitLab

Found with:
$ git grep --files-without-match --name-only "namespace views" -- ui/views/{*.cc,*.h}

TBR=ben@chromium.org

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

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

This was caused by a refactor to use the JSC (crrev.com/148166). I apologize for being so careless
and thank you mihaip for catching the problem so quickly.

BUG=140792

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

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

This flag is not feature complete on mac and linux, it's premature to enable field trials.

BUG=133602,140866


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

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

The original patch introduced a crash (GetCompositor() can return NULL), but
reverting the whole thing is not possible without a lot of other reverts, and
the proper fix is fairly involve (likely not reasonable to patch to the branch).

This only reverts the problematic part, and a follow up will do a proper fix.

BUG=141756


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=151153

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

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

Also added comment about how to use TokenService from tests.

BUG=141123
TEST=none


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

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

BUG=125846
TBR=ben@chromium.org

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

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

We were failing to associate the mobile bookmarks folder at sync startup time.
This could result in various seemingly-random bookmark unrecoverable errors.
We now associate the bookmark folder if it's there, and condition triggering
an association error on whether we expected it to be there (which we only do
on ANDROID platforms).

BUG=121587
TEST=Sign in to sync with a clean profile, but synced mobile bookmarks existing
in the mobile bookmarks folder on the server. Bookmarks should appear on the
desktop client.


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

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

BUG=141389
TEST=Run with --force-device-scale-factor=2, go to a page with standard scroll bars and observe that they are crisp.

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

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

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

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

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

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

- Reduce col padding for invalid transition to 15px and use a 50px for first
  column;
- Fix a problem that invalid transition gets reset when touch scroll is more
  than 100%;

BUG=none.
TEST=Verify icon moves less when attempting touch scroll to an invalid page.  Also invalid move is not abruptly reset in the middle while dragging the touch.

R=sky@chromium.org


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

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

I believe the crash (see bug link) is caused because we remove the delegate
connection, but we never stop observing the WebContents. I haven't reproduced
the crash, but I think this should fix the root cause. I've put in a bandaid as
well, which might be redundant, but good for safety. Will remove it once I can
confirm the fix works.

BUG=141875
R=sky@chromium.org
TEST=This type of crash (see bug) should go away.

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

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

Connects a command-line switch to the exclusions WebRuntimeFeature methods added for https://bugs.webkit.org/show_bug.cgi?id=91420
Contributed by betravis@adobe.com


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

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

This is in preparation for a NaCl-side change to make the debug stub
use the Windows debug exception handler process instead of Vectored
Exception Handling.

BUG=http://code.google.com/p/nativeclient/issues/detail?id=2948
TEST=none

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

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

Sync credential caching is a Win 8 feature that allows Metro and Desktop
to share sync credentials under some circumstances. The feature is
slated to ship with M23, and was disabled for the M22 branch point.
Now that M22 has been branched off, it is time to re-enable the feature.

This patch turns on the feature by default, and provides users with the
"--disable-sync-credential-caching" flag if the feature must be disabled.

BUG=139712
TEST=Sign in on Win 8 and make sure that "Sync Credentials" is written to the default profile directory

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

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

This takes over for part of http://codereview.chromium.org/10455004/.


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

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

TBR=ivankr@chromium.org
BUG=141559

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

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

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

TBR=yutak@chromium.org

BUG=


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

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

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

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

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

1. Reset the background of the tooltip frame view or it paints behind the
BubbleBorder2 and makes it look blurry.
2. LauncherTooltip bubbles should not consume mouse hovers.
3. BubbleBorder2::GetBorderThickness should return the actual thickness.

BUG=none
TEST=none


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

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

Bug=None
Test=Compile ChromeOS without enable_hidpi GYP_DEFINE.

R=flackr
TBR=sky

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

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

The original patch introduced a crash (GetCompositor() can return NULL), but
reverting the whole thing is not possible without a lot of other reverts, and
the proper fix is fairly involve (likely not reasonable to patch to the branch).

This only reverts the problematic part, and a follow up will do a proper fix.

BUG=141756


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

TBR=piman@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10824270

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

BUG=136394
TEST=manual

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

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

BUG=141559

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

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

Add the following UMA stats:
- Launcher_ClickOnAppListButton
  Count clicks on launcher's app list button.
- Launcher_ClickOnApp
  Count clicks on apps in launcher bar (including Chrome).
- AppList_ClickOnApp
  Count clicks on apps in app list bubble.
- AppList_Search
  Count search + open result in app list.
- AppList_ClickOnAppFromSearch
  Count clicks on apps in app list search results.

BUG=137358
TEST=Verify the added UMA stats work as expected.

R=sky@chromium.org

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

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

This function was added to support WebView.copyBackForwardList(). It
basically copies much of the content of the view's NavigationController
into the appropriate Java data structures.

BUG=138478


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

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

Fixed Issue 141873. Crash when BrowserActionButton get disabled or hidden.

BUG=141873
TEST=Try to disable or hide extension buttons, should not crash.

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

TBR=yefim@chromium.org

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

The original patch introduced a crash (GetCompositor() can return NULL), but
reverting the whole thing is not possible without a lot of other reverts, and
the proper fix is fairly involve (likely not reasonable to patch to the branch).

This only reverts the problematic part, and a follow up will do a proper fix.

BUG=141756


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

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

TBR=sergeyu@chromium.org

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

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

BUG=141906
TEST=Manual tested w/ normal & Media Source videos.


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

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

Add a constructor that takes only the "interesting" args (basically the args that aren't predictably defaulted). This will allow the cleanup a lot of tests and probably some rel code as well.


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

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

BUG=141873
TEST=Try to disable or hide extension buttons, should not crash.

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

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

I'm currently looking at this.
Review URL: https://chromiumcodereview.appspot.com/10825309

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

Consider the following scenario:
1. Type "g" in the omnibox.
--> The omnibox inline autocompletes "oogle.com".
2. Hit Tab.
--> You're now in keyword mode.

Here, inline_autocomplete_text_ is still "oogle.com", though the display text
is a blank string. The previous code caused a crash. Now, we check properly
that the inline text is a suffix of the display text.

Arguably, the inline text should always be a suffix of the display text, say by
clearing it in OmniboxEditModel::AcceptKeyword(): http://crbug.com/141890

R=pkasting@google.com
BUG=141878
TEST=Scenario in CL description above shouldn't crash.

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

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

BUG=137676
TBR=jamiewalch,tony

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

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

Need check size of both arrays before calling std::equal.

BUG=141904
Review URL: https://chromiumcodereview.appspot.com/10854099

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

TBR=ivankr@chromium.org
BUG=141986

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

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

This CL:
- eliminates unnecessary dispatches
- correctly handles devices that are discovered before interest is
registered

TEST=ran api test
BUG=133179

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150898

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

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