| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
We previously disabled it it to see if it was involved with the double
free... but the cause has been shown to be a thread-unsafe reference
count (not a party to this change).
r=brettw
Review URL: http://codereview.chromium.org/246036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27612 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Belongs to http://codereview.chromium.org/254002
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/244044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27609 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BrowsingDataRemover deletes itself (see browsing_data_remover.cc:
MessageLoop::current()->DeleteSoon(FROM_HERE, this)).
BUG=22068
Review URL: http://codereview.chromium.org/218001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27607 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
window would crash. Be more specific about when the window will be closed/deleted during dragging in hopes of making future crashes a little more clear.
BUG=none
TEST=dragging tabs between windows should work. tabs closed via window.close() on a timer should work.
Review URL: http://codereview.chromium.org/246035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27606 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TBR=pfeldman
Review URL: http://codereview.chromium.org/256009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27603 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27602 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/248040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27600 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
actions always show up in a submenu of the wrench menu, and
if they have an icon, they also show up in the toolbar area.
BUG=23380,22883
TEST=Added new automated tests for the command handling, but we need to test that the menu items show up manually. To do that, run with no extension installed, you should see "extensions" in the wrench menu. Add an extension that adds a browser action, you should now see an "extensions" submenu with "manage extensions" and the browser action(s) in the submenu.
Review URL: http://codereview.chromium.org/246037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27599 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The failures must be caused by timing issue. This CL tries to fix the issue by initializing all necessary components before running tests.
BUG=23213 AutocompleteEditViewTest tests are failing
TEST=none
Review URL: http://codereview.chromium.org/242047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27597 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Fiona Chong.
Review URL: http://codereview.chromium.org/243041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27596 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implement a readsize throttle within the TCP socket.
Add a fieldgroup trial for testing it.
BUG=none
TEST=none
Original Review URL: http://codereview.chromium.org/173259
Review URL: http://codereview.chromium.org/243038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27595 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/246027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27594 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=4839
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/242064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27593 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=23411
TEST=None
Review URL: http://codereview.chromium.org/246045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27592 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=scherkus
Review URL: http://codereview.chromium.org/246043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27590 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=16013
TEST=Playing a video should still work with the flag in incognito mode or not.
Allow <video> to use cache if running with experimental byte range support.
Review URL: http://codereview.chromium.org/253001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27589 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=23391
TEST=None
Review URL: http://codereview.chromium.org/255014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27579 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This makes it work correctly in ion3 and other window managers that don't expect clients to try to move top-level windows themselves.
This implementation grabs the pointer and keyboard. By doing this and using an override-redirect window, we should be able to avoid worrying about interactions with different window managers. The only downside (alluded to in the previous code) is that window manager keybindings don't make it through until the user dismisses the bubble by clicking outside of it or hitting Enter or Escape. I don't think this will be a problem; it's no different from what happens when you open a context menu in an app.
BUG=20523
TEST=tested first-run and bookmark bubbles in Metacity, ion3, Fluxbox, KDE4, and the WM that I'm working on
Review URL: http://codereview.chromium.org/198016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27578 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This was only needed so that we could compile the sync glue code
without access to a syncapi binary. Now that syncapi can be built from
source, it's obsolete.
Review URL: http://codereview.chromium.org/242060
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27577 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27573 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/255012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27565 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
The implementation was dependent on webkit_glue::image_resource_fetcher and had to be removed if we want to upstream WebView.
DownloadImage has been moved into RenderView::DownloadImage. WebViewImpl's set of ImageResourceFetchers has been ripped off and moved into RenderView.
Review URL: http://codereview.chromium.org/255006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27564 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Refactoring some of the NativeViewHost and NativeControl focus management so their consumers don't have to explicitly set the focused view.
See original review:
http://codereview.chromium.org/235011/show
BUG=None
TEST=Run all tests. Make sure focus is stored/restored properly in Chrome.
TBR=ben
Review URL: http://codereview.chromium.org/246032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27563 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=See browser tests.
Review URL: http://codereview.chromium.org/242055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27562 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=1648
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/251037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27561 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fixes a thread-safety bug in ExecuteCodeInTabFunction caused by using
RefCounted instead of RefCountedThreadSafe. Instead of changing
ExecuteCodeInTabFunction to be RefCountedThreadSafe, I thought it would be
better to fork off a helper class.
R=aa
BUG=none
TEST=file_reader_unittest.cc and existing browser_tests (once my other patch
lands which adds thread-safety assertions to NewRunnableMethod)
Review URL: http://codereview.chromium.org/243033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27559 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=1635
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/249036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27557 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
The initialization of the ProfileSyncService was being called twice in one test.
Test=RenderViewHostManagerTest in unittests
Bug=none
Review URL: http://codereview.chromium.org/248036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27556 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=1642
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/255013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27554 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Must check iterator's Done() before dereferencing (list may be empty).
Review URL: http://codereview.chromium.org/243040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27552 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27551 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Not sure when this regressed. Looking at the svn history I can't even find a revision where it would have worked. We probably made some other change that revealed this long-standing bug.
BUG=23233
TEST=drag a bookmark out of a bookmark bar folder
Review URL: http://codereview.chromium.org/255011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27550 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=1650
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/251036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27549 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=2287
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/242065
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27548 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
creating an instance of the enum.
CID=5718
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/249035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27547 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Detected using this CL:
http://codereview.chromium.org/251012
R=jcampan
BUG=none
TEST=covered by browser tests once my assertion to NewRunnableMethod lands.
Review URL: http://codereview.chromium.org/251033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27544 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
TBR=agl
Review URL: http://codereview.chromium.org/251035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27541 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/256003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27540 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
CID=5447
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/243037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27538 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/256002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27536 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/251032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27535 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use Helvetica, Times, Courier, Apple Chancery, and Papyrus for sans-serif,
serif, monospace, cursive, and fantasy, respectively. These are the fonts
used by Safari and Firefox on the Mac, and the fonts that we run layout tests
with on the Mac.
BUG=19135 21458
TEST=http://www.w3.org/Style/Examples/007/fonts
Check the fonts for sans-serif, serif, monospace, cursive, fantasy
Note that the font used for "Helvetica, sans-serif" is actually Verdana
Review URL: http://codereview.chromium.org/244039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27533 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=23318
TEST=checked that it matches native window behavior in metacity and kwin
Review URL: http://codereview.chromium.org/246029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27531 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
This reverts the previous commit (compile failure).
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27530 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Otherwise, the first time the singleton is touched we've already started
loading a page! We surely have a bunch of threads going at that point.
Review URL: http://codereview.chromium.org/245039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27528 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- use one call string convert
- load the nib from the main bundle
- make the cancel button autosize to it's string
- make the cancel button tweak it's position based on the string (not dialog position, but keep the right edge as is)
- fix up some field overlaps, and position, dialog still isn't pretty, but this is how Cole said to leave it rather then reopen the discussion now.
- use the WithFixup helpers to deal with ellipses
TEST=button fits text, fields don't overlap any strings.
BUG=20534
Review URL: http://codereview.chromium.org/235055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27526 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
redirect_query_success_ in the constructor.
CID=1653
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/251031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27525 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
focus manager when closing Chrome (linux toolkit views).
See original review at:
http://codereview.chromium.org/220036/show
BUG=None
TEST=Make sure find works as expected on Chrome Linux toolkit views.
TBR=ben
Review URL: http://codereview.chromium.org/243035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27524 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
CID=4804
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/246025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27523 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
system time. Should have done that first, but I had started writing the test before writing the code and learning how to use TimeTicks.
TEST=SyncerThreadWithSyncerTest.Polling
BUG=23336
Review URL: http://codereview.chromium.org/248034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27522 0039d316-1c4b-4281-b951-d872f2087c98
|