| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
on 'using namespace icu'.
This is Chrome's counterpart to the ICU header change that disables
'using namespace icu' (http://codereview.chromium.org/171010/show), which is required to avoid the name colission between Chrome's StringPiece (in base) and ICU's StringPiece.
The webkit change (which is minor) will be dealt with in the webkit bugzilla.
This can go in before the ICU change/upgrade without affecting anything.
BUG=8198
TEST=All the targets are built without an error on all platforms.
Review URL: http://codereview.chromium.org/171012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23613 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is necessary since ProxyService is getting shared between chrome's url request contexts (off the record, media), and the current way it is being shared could result in free memory read/writes during shutdown.
This is a step towards fixing http://crbug.com/15289.
BUG=http://crbug.com/15289
TEST=The existing tests should continue to pass following this refactor.
Review URL: http://codereview.chromium.org/165430
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23612 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
also change the interface for SimulateOSKeyPress()/SendKeyPress() to take a VKEY_ value (defined in base/keyboard_codes.h) rather than a VK_ value.
BUG=19076
Review URL: http://codereview.chromium.org/171079
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23611 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=19433
TEST=obvious
Review URL: http://codereview.chromium.org/171077
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23608 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/171088
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23605 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This removes the Combobox* source arg from the Model methods, which wasn't really used by anything.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/165514
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23603 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
MemoryDetails::StartFetch() isn't implemented, thus it was never signalling that it was done fetching memory details. Now it does.
BUG=15418
Review URL: http://codereview.chromium.org/172052
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23602 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
sending us paths with .. in them.
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/171025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23597 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This brings us in line with Firefox's behavior. Ctrl-L highlights
the URL but doesn't update the PRIMARY selection, so Ctrl-C is a
common* way to get the current location into PRIMARY so you can
middle-click somewhere else to paste it.
* I use it and a user mentioned that they do too. :-P
Tested as follows:
- highlight some text on the page to make it PRIMARY
- hit Ctrl-L and confirm that text on page is still PRIMARY
- hit Ctrl-C and confirm that location is now PRIMARY
- repeat, but this time hit the right arrow key between Ctrl-L
and Ctrl-C to unhighlight the text first. the text on the
page should remain the PRIMARY selection
BUG=18403
TESTED=see above
Review URL: http://codereview.chromium.org/164539
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23596 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Also makes the label look native. I can't get the label inside the entry,
though.
http://crbug.com/17962
Review URL: http://codereview.chromium.org/172069
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23595 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
There are no significant logic changes except the addition of a link which points to the web ui.
BUG=none
Review URL: http://codereview.chromium.org/172007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23591 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/165513
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23589 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TBR=eroman
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23588 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=eroman
Review URL: http://codereview.chromium.org/171083
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23587 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=interactive ui tests builds and runs and BrowserFocusTest.TabInitialFocus passes
Review URL: http://codereview.chromium.org/172059
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23585 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=19353
TEST=None
Review URL: http://codereview.chromium.org/165538
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23584 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Command:
awk '{if ( length($0) > 80 ) { print FILENAME, FNR, $0 } }' *.cc *.h
# recursive
find . -type f \( -name '*.cc' -o -name '*.h' \) -exec awk '{if ( length($0) >
80 ) { print FILENAME, FNR, $0 } }' {} \;
Patch by Pierre-Antoine LaFayette <pierre.lafayette@gmail.com>
BUG=None
TEST=None
Code review URL: http://codereview.chromium.org/164566
Review URL: http://codereview.chromium.org/171066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23583 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=19440
Review URL: http://codereview.chromium.org/171069
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23578 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
work on actually fleshing out some tests in the next CL.
Review URL: http://codereview.chromium.org/171032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23577 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=Covered by unit_tests and other automated tests.
http://crbug.com/19202
Review URL: http://codereview.chromium.org/164518
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23572 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
progress and we'll pick up the other countries eventually.
Added some temporary comments on the Bing entry since the way it's coexisting with the Live entries right now makes for a few slightly unusual bits.
BUG=13065
TEST=Search providers should show Bing and not Live Search, setting Bing to default should result in working Suggest and search functionality.
Review URL: http://codereview.chromium.org/171065
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23571 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
can be called when we don't have a URLContext from which to return cookies.
Previously we crashed, this patch just logs an error and returns an empty string instead.
BUG=http://crbug.com/10706
Review URL: http://codereview.chromium.org/171050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23568 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
to get the IO thread's message loop when it was going away. Do the same acrobatics as in BufferedResourceHandler by going to the UI thread first, and doing refcounting manually.
Also fix an issue I saw by inspection in BufferedResourceHandler, where Release should be called at the end of the function.
BUG=19415
Review URL: http://codereview.chromium.org/171055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23563 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
These targets include from third_party/bzip2 but list it in
dependencies. This means that they don't get the
direct_dependents_settings and this is breaking the Ubuntu package
building with use_system_bzip2.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23556 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Per discussion:
http://groups.google.com/group/chromium-dev/browse_thread/thread/b4c671403265
Patch from rsesek@chromium.org
BUG=none
TEST=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23552 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=18902
TEST=items look like buttons all the time, not only on hover. icon is 16x16 :-/. spacing slightly modified (not final yet).
Review URL: http://codereview.chromium.org/172030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23550 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=15776 (http://crbug.com/15776)
TEST=Download an item, open the download page, drag the icon of
the download to the desktop.
Review URL: http://codereview.chromium.org/164459
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23548 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Add a script to make a link for "en.lproj" so we always have an english in the fall back list.
Run said script as a postbuild step.
TEST=Chrome should now show all 40 language when you do a get info in the finder.
BUG=19165
Review URL: http://codereview.chromium.org/171043
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23547 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=19335
TBR=aa
Review URL: http://codereview.chromium.org/171036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23543 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=19204
TEST=When reporting is enabled, Breakpad should pick up browser and renderer
process crashes;
When reporting is enabled, renderer should not log messages like
[mmdd/hhmmss:WARNING:/path/to/breakpad_mac.mm(57)] Breakpad disabled;
When reporting is disabled, browser and renderer should both log these
messages.
Review URL: http://codereview.chromium.org/172037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23538 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=19467
TEST=none
TBR=maruel
Review URL: http://codereview.chromium.org/172036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23536 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a bit of a Coverity nit, but if disposition
SUPPRESS_OPEN is ever passed in to this function,
it will crash because we try to call Focus on a
new_contents pointer that is null. I verified this
by hard coding the disposition as we open URLs.
BUG=17101
TEST=No significant change.
Review URL: http://codereview.chromium.org/155773
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23532 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
consistently passing tests from test_expectations. Add some svn:ignore's for .scons files.
BUG=18896,15217,18978,19428,19414
TEST=Green is good.
TBR=pkasting
Review URL: http://codereview.chromium.org/172022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23526 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=rafaelw
Review URL: http://codereview.chromium.org/171024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23525 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG= http://crbug.com/18767
TEST= none
Review URL: http://codereview.chromium.org/165478
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23524 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
variation that is showing up.
BUG=http://crbug.com/19393
TBR=ananta
Review URL: http://codereview.chromium.org/172021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23522 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
TBR=rafaelw@chromium.org,kathyw@google.com
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23521 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
footer.
TBR=rafaelw@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23520 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=http://crbug.com/19415
TBR=jam
Review URL: http://codereview.chromium.org/171022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23519 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=http://crbug.com/19414
TBR=jam
Review URL: http://codereview.chromium.org/171021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23518 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Sync gyp stuff moved from common.gypi to chrome.gyp.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/169018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23517 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TBR=mpcomplete@chromium.org
BUG=19335
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23515 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
The problem was that the BufferedResourceHandler could now be destructed on the file thread, if the task that it posted on the IO thread completed before the code on the file thread returned. Solved this by doing manual refcounting.
Review URL: http://codereview.chromium.org/171015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23512 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=http://crbug.com/19393
Review URL: http://codereview.chromium.org/171011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23510 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Initialize crash reporting in helper processes such as the renderer process.
Renderer crash reporting stopped working in r23006 when multiple .app bundles
were introduced, because the stats collection and crash reporting preference
is presently accessed via NSUserDefaults, keyed on the bundle ID. The main
browser process and helper processes have distinct bundle IDs. In the new
scheme, only the main browser process consults this preference, and passes it
to helper processes in their command lines.
BUG=19204
TEST=When reporting is enabled, Breakpad should pick up browser and renderer
process crashes;
When reporting is enabled, renderer should not log messages like
[mmdd/hhmmss:WARNING:/path/to/breakpad_mac.mm(47)] Breakpad disabled;
When reporting is disabled, browser and renderer should both log these
messages.
Review URL: http://codereview.chromium.org/165546
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23509 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This simplifies the logic and fixes a couple of bugs (and probably more):
- the first time the tooltip is shown after a navigation, it was stale (showed "stop loading the page")
- the first time the tooltip is shown after the text of the omnibox is changed, it was stale
Also, don't hardcode "Google" as the name of the search provider.
BUG=12836
Review URL: http://codereview.chromium.org/171014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23508 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Removes ProfilePersonalization (which is why the CL appears huge)
- Makes Profile do the work ProfilePersonalization did for creating PSS
- cloudy:stats > about:sync
- Removes the cloudy:// scheme and uses chrome:// for resources.
(Note SyncResourcesSource is a straight copy of CloudyResourceSource).
- Moves prefs and switches to pref_names and chrome_switches.
BUG=none
TEST=ProfileSyncServiceTest, LiveBookmarkSyncTests, SyncSetupWizardTest
Review URL: http://codereview.chromium.org/164544
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23505 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
No-op code change.
Review URL: http://codereview.chromium.org/172016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23504 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=9103
Review URL: http://codereview.chromium.org/165352
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23503 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
I had to move the locks from PluginService to PluginList, so that a lock (which can block other threads) isn't held while loading the plugins.
BUG=17938
TEST=added asserts which crash if plugins loaded on IO thread, current UI tests exercise them
Review URL: http://codereview.chromium.org/164305
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23501 0039d316-1c4b-4281-b951-d872f2087c98
|