| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
Split so it's not a 2 sided patch.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/242056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27575 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=23396
TEST=None
Review URL: http://codereview.chromium.org/244043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27574 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27573 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
investigate.
BUG=23394
TBR=huanr
Review URL: http://codereview.chromium.org/251039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27570 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
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/235056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27560 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
|
|
|
|
|
|
|
|
| |
TBR=bradnelson
Review URL: http://codereview.chromium.org/242066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27558 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
|
|
|
|
|
|
|
|
|
|
| |
R=darin@chromium.org
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/217006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27555 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
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=pray for green
TBR=atwilson
Review URL: http://codereview.chromium.org/235057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27553 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
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/242063
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27546 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
newer revision from googlecode, which has screenshot name changes.
Review URL: http://codereview.chromium.org/235041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27545 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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
baselines instead of scraping them.
Greatly speeds up the tool and circumvents the need to scrape remote sites for the baselines.
BUG=none
R=ojan
TEST=run "test_output_formatter.bat -l" -- the script should not scrape anything
Review URL: http://codereview.chromium.org/211025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27543 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Step 1. It builds on OSX and I guessed on linux
Will try that next. Also will start adding place
holders for options next
Review URL: http://codereview.chromium.org/251034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27542 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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
output that matches jsc.
http://trac.webkit.org/changeset/48841
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/244040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27539 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
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/242045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27537 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
|
|
|
|
|
|
|
|
|
| |
BUG=15478
TEST=none
Review URL: http://codereview.chromium.org/242057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27534 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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
instead of string16.
BUG=none
TEST=base unit tests
Original patch by avd@google.com at:
http://codereview.chromium.org/245046
Review URL: http://codereview.chromium.org/242061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27532 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
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=green is good
TBR=atwilson
Review URL: http://codereview.chromium.org/251030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27529 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
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/242051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27527 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
|
|
|
|
|
|
|
|
|
|
|
| |
know the details of how to build and run tests on every
platform.
This is just the initial checkin with lots of TODOs
Review URL: http://codereview.chromium.org/248030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27521 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG= http://crbug.com/17696
TEST= none
Review URL: http://codereview.chromium.org/222025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@27520 0039d316-1c4b-4281-b951-d872f2087c98
|