| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
TBR=ben@chromium.org
Review URL: http://codereview.chromium.org/173508
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24509 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
We can't delay destroying the download item view because it has to be destoyed before the download model is destroyed. Insted we delay the execution of the command.
Review URL: http://codereview.chromium.org/173458
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24507 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
trying to use one.
BUG=19456
Review URL: http://codereview.chromium.org/173238
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24506 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/174494
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24505 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=run app_unittests.exe, browser_tests.exe and selenium_tests.exe
Review URL: http://codereview.chromium.org/173418
Patch from Thiago Farina <thiago.farina@gmail.com>.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24504 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
NOTIMPLEMENTEDs to resolve (initiating drags is the biggest remaining
issue). I'll tackle the remaining issues separately.
BUG=none
TEST=make sure bookmark menus work well on windows still.
Review URL: http://codereview.chromium.org/173431
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24501 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Wires message ports to extensions through the devtools remote socket.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/174226
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24499 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=19750
TEST=none
Review URL: http://codereview.chromium.org/174463
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24497 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
When you close a tab using the mouse, we synthesize a fake WM_MOUSEMOVE message so that the close button that slides under the mouse will highlight. We now use GetMessagePos to read the position of the cursor and SendMessage to dispatch it, otherwise the receiving GetMessagePos call will not calculate the coordinates correctly (coordinates will be relative to the Chrome main window top left point, not the monitor's top left point, causing the tabstrip to resize).
BUG=18832
TEST=On Vista, open a bunch of tabs, then start closing the tab in the middle using the mouse. Don't move the mouse at all while doing this and observe that the tab strip doesn't resize. Also, the close button on the tab that slides under the mouse should be highlighted.
Review URL: http://codereview.chromium.org/173433
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24493 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
HTTP/HTTPs.
It seems that Firefox is doing the same thing.
BUG=16534 (http://crbug.com/16534)
TEST=Visit any website and append double-quote (") for the URL and invoke
"Select All" by Control-A, then copy the URL by Control-C. Paste the result into
notepad and check if the double-quote is NOT encoded.
TEST=Access file:///" and select whole Omnibox and copy the URL. Paste the text
into notepad and check if the double-quote is NOT URL encoded.
TEST=Put " in Omnibox and type enter key so that you will be navigated to
http://www.google.co.jp/search?sourceid=chrome&ie=UTF-8&q=%22. Copy whole
content in Omnibox and paste it into notepad. Then, check if the double-quote is
URL-encoded (%22).
Original Review: http://codereview.chromium.org/155495/show
Patch by hamaji@chromium.org (he's on vacation and asked me to land)
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24472 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If 'profile' is NULL, we should return
ResultCodes::NORMAL_EXIT so that we won't go on to
dereference 'profile'.
R=brettw,estade
BUG=http://crbug.com/17101
TEST=none
Review URL: http://codereview.chromium.org/171007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24469 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
depends on are ready.
UMR should be fixed after setting new appcache variable.
BUG=no
TEST=no
Review URL: http://codereview.chromium.org/173482
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24459 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=http://crbug.com/20131
TEST=see bug; was very specific.
Review URL: http://codereview.chromium.org/173428
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24455 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We need to wait for correct number of navigations when going back or forward
to the LinkDoctor page.
Re-enable previously disabled tests and remove the now-duplicate browser test.
The test is still not very solid, because the LinkDoctor should be mocked.
TEST=Covered by ui_tests.
http://crbug.com/19361, http://crbug.com/19395
Review URL: http://codereview.chromium.org/174396
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24446 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
page using SCIM
This CL fixes issue 19081 by setting im context's client window when host view's native widget is realized rather than focused in.
When setting im context's client window, GtkIMMulticontext will destroy and recreate its slave im context, which will cause this issue.
BUG=19081: Linux: Not able to switch input method correctly in web page using SCIM
TEST=Export GTK_IM_MODULE=scim then open chrome, open www.google.com and place cursor in search box, hit ctrl-space to open SCIM then click scim bar or tray icon and select another input method, then check if input method was switched correct.
Review URL: http://codereview.chromium.org/174217
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24442 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
new window. Cache gradient to avoid re-creating it every time we draw a button cell.
BUG=none
TEST=switch themes, ensure it works. Make sure buttons draw correctly.
Review URL: http://codereview.chromium.org/173445
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24441 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts 24426
TBR=yutak
Review URL: http://codereview.chromium.org/173460
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24429 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=19653
Review URL: http://codereview.chromium.org/174462
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24426 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This CL improves the key event handing code of BrowserWindowGtk. Issue 20118 is also fixed.
BUG=20118: Typing a colon (:) into the address bar causes the browser to freeze
TEST=Run chrome with a XIM input method, chrome and x server's cpu usage shall be very low after pressing shift key (or any other modifier key) in omnibox.
Review URL: http://codereview.chromium.org/173344
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24422 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Turns out download item views need to be destroyed before the associated download item is destroyed.
Review URL: http://codereview.chromium.org/173456
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24421 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
. Moves listening from DidProcess to WillProcess. This is necessitated
by the bookmark bar (chrome menu) tests that spawn a nested run
loop, which results in DidProcess not getting sent and the test
wedging. This brings the code closer in line with Windows.
. If there is a mouse/keyboard grab, mouse/keyboard events are sent to
the grabbed widget.
. Deals with a NULL widget for sending keyboard events (tries to find
focused widget).
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/173392
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24420 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
depends on are ready.
BUG=11547
TEST=See bug.
Review URL: http://codereview.chromium.org/173239
TBR=mpcomplete@chromium.org
Review URL: http://codereview.chromium.org/174501
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24419 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG= none
TEST= none
Review URL: http://codereview.chromium.org/173444
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24418 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Although we represent time internally starting from 1601, there are still
things like time explosion that will not work before the year 1900. This
limitation is the same as it was previously.
BUG=14734
Review URL: http://codereview.chromium.org/173296
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24417 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
Generalize the chromeos rules so we don't have to list every file in the exclusions.
Review URL: http://codereview.chromium.org/174446
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24416 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=13320
Review URL: http://codereview.chromium.org/173432
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24414 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=20186
Review URL: http://codereview.chromium.org/173407
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24413 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Added ALT+E for Chrome Page Menu
Added ALT+F for Chrome App Menu
BUG=906 (http://crbug.com/906)
TEST=Triggered ALT+A and ALT+P from the keyboard which successfully opened the corresponding menu.
Review URL: http://codereview.chromium.org/174044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24410 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/173443
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24399 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Fix examples in StringTokenizer header that recommended doing that.
BUG=none
TEST=on linux, open options, click proxy configuration button a bunch. It should not fail intermittently.
Review URL: http://codereview.chromium.org/174490
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24398 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This CL fixes issue 20066, a regression issue caused by a change in CL 172041.
BUG=20066: Regression: search keywords search with default search engine only.
TEST=Assuming 'y' is associated to a search engine, type "y abc" in omnibox and press enter shall search abc with the search engine associated to 'y'.
Review URL: http://codereview.chromium.org/173332
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24388 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG= none
TEST= none
Review URL: http://codereview.chromium.org/173371
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24385 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG= http://crbug.com/19162
TEST= Start browser with NNTP, note line of tips at bottom.
Review URL: http://codereview.chromium.org/173087
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24384 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
support. (Try 2)
TEST=none
BUG=19663
Original Review URL: http://codereview.chromium.org/173095
Review URL: http://codereview.chromium.org/173397
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24378 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=11547
TEST=See bug.
Review URL: http://codereview.chromium.org/173239
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24372 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=20262
TEST=Verify that the attribution 'theme created by' text matches the title color under the thumbnails
Review URL: http://codereview.chromium.org/173409
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24370 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
It carries state in singletons, which is evil. browser_tests launcher
deals with that properly.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/174470
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24360 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/174414
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24356 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
window matches the desktop size on startup, the WM puts chrome
in full screen mode that you can't get out of.
Our workaround is to never restore to a size that matches the
screen. This is better than telling people to edit their
Local State file.
Review URL: http://codereview.chromium.org/174457
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24344 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/174449
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24336 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=13142
Review URL: http://codereview.chromium.org/173334
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24327 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Getting clicks in the decoration areas right caused a change to how
clicks on the unfocused field worked. NSTextField also forwards the
event to get click-drag selection right, this does the same thing for
clicks in the decorations (other clicks are passed to super).
http://crbug.com/20142
TEST=Browse to www.google.com. Click in page. Click in omnibox
without releasing. You should get a caret insertion point.
Review URL: http://codereview.chromium.org/173362
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24325 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
- Add attribution support back to the NTP
BUG=20019
TEST=Install a theme with a white logo and attribution, and verify that both display correctly.
Review URL: http://codereview.chromium.org/173372
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24323 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
(We were previously just grabbing a raw GTK color, which didn't work out very
well in themes like High Contrast Inverse...)
Review URL: http://codereview.chromium.org/174407
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24316 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/173389
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24310 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL reworks the GtkSocket/GtkPlug code for windowed plugins on linux. Instead of having the plugin ask the browser to create a socket to plug into, it simply creates a plug and sends it to the browser. The browser creates a socket and attaches the plug when the socket becomes realized
This fixes 2 main issues:
- we can create windowed plugins in background tabs (Issue 16125)
- we can detach tabs with windowed plugins and reattach them (Issue 17110)
I reworked the IPCs, so it removes some amount of linux-specific things. We also need less synchronous IPCs to create/destroy plugins, so that should be a bit faster. In particular, I removed the plugin pid map, and instead made sure the renderer always destroys the plugin containers if the plugin process crashes - they will be destroyed if the renderer process crashes. Let me know if you have an issue with that.
Also, the intermediate plug/socket creation now happens in webplugin_delegate_impl_gtk. That means test_shell uses it as well. It made the code a lot simpler, and means we're testing it as well, albeit with a bit of extra overhead.
Bonus: I found a big bad bug in the GtkPluginContainer that made its width/height alias with some internal gtk structures. That was certainly causing some amounts of bugs.
Bonus 2: scrolling now looks more in sync with the rest of the page, though I'm not exactly sure which part caused that.
BUG=16125,17110
TEST=a lot of manual testing involving YouTube videos
Review URL: http://codereview.chromium.org/174295
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24309 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
issue.
BUG=19823
TEST=see bug
Review URL: http://codereview.chromium.org/173333
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24287 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
errors locally.
Review URL: http://codereview.chromium.org/174414
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24269 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
separator, status and tooltip.
BUG=19982
TEST=Assign @role application, document, radiogroup, region, separator, status or tooltip to any dom element, and use Inspect32 (or similar tool) to see it exposed correctly.
Review URL: http://codereview.chromium.org/174382
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24262 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=20158
Review URL: http://codereview.chromium.org/174365
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24255 0039d316-1c4b-4281-b951-d872f2087c98
|