| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/307007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29548 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
We haven't yet resolved how to integrate with OS-native password stores,
so printing this NOTIMPLEMENTED isn't helpful.
Review URL: http://codereview.chromium.org/308003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29545 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
click to close or right click for the context menu.
BUG=21627
TEST=On a tab, left mouse down, right mouse click, then drag. Left mouse
up should release the tab.
Review URL: http://codereview.chromium.org/305004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29542 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In unit tests, we were not removing the registration because we never get the BROWSER_WINDOW_READY notification. When a browser later gets allocated the same memory address, we hit a DCHECK in registrar_.Add().
This change should be safe because BROWSER_WINDOW_READY only gets called at most once per browser instance.
BUG=none
TEST=linux build bots stop flaking on this test
Review URL: http://codereview.chromium.org/306009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29538 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29537 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Refactored BrowserExtender and added NormalExtender and PopopExtender.
BrowserExtender::Create factory method returns appropriate extender for given BrowserView.
* Added #if/#else to PanelController to accept BrowserView in Constructor.
* added guard code in Close so that it will ignore 2nd invocation of Close. (views seems
to call Close() (when it requests, and when it's really closed)
Added missing code in TabOverviewMessageListener
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/276075
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29533 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This notimpl shows up every time a renderer crashes. People
frequently post it in bug reports as if it's meaningful. Get
rid of it.
BUG=TEST=none
Review URL: http://codereview.chromium.org/294028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29532 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
full width of the tab placeholder. Make tab dragging as smooth as glass.
BUG=24982, 24983
TEST=tab dragging between windows, tab dragging w/in a window.
Review URL: http://codereview.chromium.org/306006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29531 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
It works.
BUG=25078
Review URL: http://codereview.chromium.org/297007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29530 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=chrome doesn't dcheck if you launch with a command line such as "chrome %U"
Review URL: http://codereview.chromium.org/293010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29529 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=25257
TEST=none
Review URL: http://codereview.chromium.org/303009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29528 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
blocking. OS-Mac only.
BUG= http://crbug.com/20899
TEST= Open Firefox; start up Chrome and try to import settings from Firefox. Dialog box warns you to close Firefox.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=29481
Review URL: http://codereview.chromium.org/283023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29527 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
e.g. 'data:', by 'Alt + Click'.
Note: This change only fixes crash. Supporting a download for URL
with 'data:' scheme is not implemented in the change.
BUG=24846
TEST=See the first entry of bug 24846 and confirm that the browser won't crash.
Original patch by hayato@google.com at
http://codereview.chromium.org/287002/show
Review URL: http://codereview.chromium.org/306007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29524 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/308002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29519 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I separated this from the other changelist so that the
try servers wouldn't fail because a binary file had
been added.
TBR=rafaelw
BUG=12119
TEST=None, no code change.
Review URL: http://codereview.chromium.org/294024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29517 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
problems with some tests as the nested message loop run by session
restore gets prematurely exited and Chrome is left in a weird state.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/294009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29515 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
And made some minor lint fixes and code refactoring on the way, based on CR comments of previous attempt.
BUG=none
TEST=Make sure that the extension resources can still be properly localized and that they also load correctly when they are not localized.
Review URL: http://codereview.chromium.org/267051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29512 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
We probably want to discourage the use of ints for PIDs. This is a start; there are many other places where we should fix this.
BUG=25272
TEST=none
Review URL: http://codereview.chromium.org/300010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29511 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
on Vista
Original description:
Implement badges for page actions. Also add badge text color API.
Also change color APIs from wanting ARGB to RGBA.
BUG=24635
BUG=24644
BUG=25215
Review URL: http://codereview.chromium.org/291003
TBR=mpcomplete@chromium.org
Review URL: http://codereview.chromium.org/306004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29509 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29503 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
on new API. Deleted old stuff.
TBR=kathyw@google.com,rafaelw@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29500 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=18693
TEST=Added new tests to browser_tests.
Review URL: http://codereview.chromium.org/300005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29498 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
and remove experimentalWebSocketsEnabled from WebPreferences/Settings.
Patch for http://bugs.webkit.org/show_bug.cgi?id=29896 will remove Settings::experimentalWebSocketsEnabled(), so we need to remove it from WebPreferences too. This is because such configuration in Settings confuses other WebKit developers and such control is only required for chromium.
Instead add WebKit::enableWebSockets() in this change to
control WebSockets feature.
Review URL: http://codereview.chromium.org/292012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29497 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=22575
Review URL: http://codereview.chromium.org/269103
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29495 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/271020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29491 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=yutak
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/294017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29489 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=Focus background of google.com, hit "a". Shouldn't DCHECK.
Review URL: http://codereview.chromium.org/297017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29488 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Otherwise the whole string (with spaces) is passed to Chrome, and it doesn't work.
BUG=none
TEST=ui tests should work
Review URL: http://codereview.chromium.org/276073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29487 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
A step towards chromebot for mac. Sets up Breakpad to build and run with
Chromium build if mac_chromium_breakpad==1. Do things similar to linux so
far. Not enabled on any bot yet.
Review URL: http://codereview.chromium.org/278014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29484 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
it blocks the main message loop.
BUG=19441
Review URL: http://codereview.chromium.org/294014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29482 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
blocking. OS-Mac only.
BUG= http://crbug.com/20899
TEST= Open Firefox; start up Chrome and try to import settings from Firefox. Dialog box warns you to close Firefox.
Review URL: http://codereview.chromium.org/283023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29481 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
but that are used to produce images for the docs.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/293015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29480 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/285012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29478 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
I tried fixing all the Windows code but there's a *ton* of it.
This change will at least prevent people from adding new code that
uses the deprecated version (as that won't compile on Lin/Mac).
BUG=24672
Review URL: http://codereview.chromium.org/293013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29472 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Don't treat the body of an "authentication required" response as a valid dictionary!
- In the event of a 200 status code, check the response actually looks like a dictionary.
BUG=24486
TEST=Verified the appearance of new dictionaries after changing spell-check language; verified spell-checking on new dictionaries.
TBR=sidchat
Review URL: http://codereview.chromium.org/295014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29471 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reduces the size of the Preferences file and frees up some disk space.
The smaller Preferences file should help improve startup time for users
who have installed lots of themes.
BUG=24377
Review URL: http://codereview.chromium.org/303006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29468 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
With the recent fix for nested message loops on the Mac, this works correctly.
BUG=24808
TEST=Printing on the Mac should continue to work.
Review URL: http://codereview.chromium.org/295012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29466 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Add missing file (bookmark_model_observer.h) to chrome.gyp.
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/300009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29463 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
loads subresource with bad cert.
The idea is to make UI reflect loading of resources with bad certificates that were previously approved by the user.
bug=20176
TEST=SSLUITest.TestUnsafeContentsInWorker unit_test.
Review URL: http://codereview.chromium.org/268006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29461 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
in sysprefs.
Instead of just dispatching to the menu after a key comes back from the renderer, do a complete re-dispatch to NSApp (so that the event gets to the menu and cmd-` handlers) but then ignore it when it comes back to the web (because we already sent this event to the renderer once).
BUG=24817
TEST=Open sysprefs, change keyboard shortcut for "Move focus to next window in active application" to e.g. cmd-\. Open two chrome windows, focus the web, hit cmd-\. It should switch windows. All other keyboard shortcuts should still work (test that ctrl-tab works when web has focus, test backspace when text field is focussed in web, when background is focussed in web, when IME is active, test hitting cmd-1/2 when omnibox or web have focus, hit cmd-left when omnibox, textbox in web, background in web has focus, test that cmd-f in docs still opens doc's find interface)
Review URL: http://codereview.chromium.org/303002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29458 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also change color APIs from wanting ARGB to RGBA.
BUG=24635
BUG=24644
BUG=25215
Review URL: http://codereview.chromium.org/291003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29457 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
"http://..." we always parsed as a URL. Now we parse more like we would without the scheme, so that we can reject various kinds of invalid inputs.
BUG=none
TEST=Input "http://foo:bar" (without quotes) in the address bar, and the default action should be search, not navigate.
Review URL: http://codereview.chromium.org/292003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29450 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename GetType and SetType using the Unix style to match with the other usages in BookmarkNode.
GetType -> type
SetType -> set_type
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/297005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29443 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=24842
TEST=none
Review URL: http://codereview.chromium.org/304004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29442 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
No live code path uses it.
Review URL: http://codereview.chromium.org/273068
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29438 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
one of the permanent nodes are changed by a bookmark model event. I think we may hit a roadblock a bit deeper down right now and not actually poison the user's cloud state, but it seems best to
catch this at first-chance.
BUG=25067
Review URL: http://codereview.chromium.org/295007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29437 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
for words instead of wstring. I also changed some places where it converted the
word to a string to do that conversion at the last possible second before
giving it to Hunspell (since this conversion isn't needed for Mac).
TEST=Covered by unit tests
BUG=none
Review URL: http://codereview.chromium.org/274077
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29435 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=the bookmark bubble menu should have title case items for the two.
BUG=24912
Review URL: http://codereview.chromium.org/297004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29431 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/280002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29430 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the case where the user types something that on its own isn't navigable, but might be the prefix of something else navigable.
While there are no tests for this directly, another change of mine to treat more inputs with explicit schemes as queries (e.g. "http:/") relies on this, and does unittest for it.
In order to fit the new relevance scores into the table, I went through and simplified the relevance scoring so that generally providers used the same scores for more input types. The effects of this should be barely noticeable (it affects the ranking of past search queries that are old against results from the secondary search provider), and it simplifies the code noticeably.
This also fixes a "bug" that the NavSuggest results were incremented backwards, but since we only score one of these right now there's no visible effect.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/291005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29428 0039d316-1c4b-4281-b951-d872f2087c98
|