| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
directionality in download shelf.
File names in download shelf are forced to be LTR in DownloadItemView and
through ElideFileName().
BUG=http://crbug.com/10860
TEST=1. Open chrome with Hebrew UI.
2. Right click a link and chose Save As... (4th item from the top for
non-Hebrew
speakers)
3. In the save as dialog name the file קובץ.html
4. In the download shelf the filename should display as קובץ.html (not
html.קובץ)
Review URL: http://codereview.chromium.org/131001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26359 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Remove old main(argc, argv) from syncable_unittest.
- Since the old notifier target had only one unittest,
merge it with the syncer tests.
- Remove listener_unittest.cc entirely.
- Don't compile natserver_main.cc, which has another main()
- Use a test main from the test_support_unit library. This
has the downside of pulling in a lot of deps we don't need,
but it gives us a run_all_unittests.cc.
- Fix a few compile errors in test code.
TEST=unit tests pass
BUG=none
Review URL: http://codereview.chromium.org/205021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26358 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
"Search downloads" pages.
BUG= http://crbug.com/17892
TEST= Open history / downloads page. Search for "<h1>HI</h1>". Note that the HTML is not interpreted, but displayed as text.
Review URL: http://codereview.chromium.org/201129
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26357 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change makes WebViewDelegate extend from WebViewClient as
a temporary means to start having consumers implement and use
WebViewClient.
R=dglazkov
BUG=10033
TEST=none
Review URL: http://codereview.chromium.org/196128
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26355 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=21903
TEST=Open 20 tabs. Put mouse over close button of some tab in the middle and keep clicking to close tabs. When the tab below the mouse is right-most, closing it should cause the other tabs to grow so that another close box ends up below the cursor (until the remaining tabs have reached the maximum tab width).
Review URL: http://codereview.chromium.org/196137
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26354 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=extension bar is no longer visible
Review URL: http://codereview.chromium.org/206030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26352 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
PathService method, also avoids the conflict with the the windows function.
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/196072
Patch from thiago.farina@gmail.com.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26351 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
and Linux.
TBR=agl
BUG=21958
TEST=none
Review URL: http://codereview.chromium.org/204040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26350 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
a TabWindowController.
Review URL: http://codereview.chromium.org/204030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26349 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
The SDK has gotten stupider in 10.6, and has conspired with the compiler,
which now thinks that when the receiver is held in a scoped_nsobject, NSColor
and NSSize arguments should be converted to their CG cousins.
Review URL: http://codereview.chromium.org/205016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26348 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/205015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26347 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Nib changes:
- Remove the "utility," "minimize," and "hide on deactivate" bits from the
window.
- Remove the "rich text" bit from the title text field.
- Remove the "selectable" bit from the version text field.
BUG=21918
TEST=About box should not float above other windows
Review URL: http://codereview.chromium.org/206019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26346 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This is #9 flaky test and rising (#5 in tests failing recently).
TBR=rvargas
Review URL: http://codereview.chromium.org/193127
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26345 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=16929
Review URL: http://codereview.chromium.org/203069
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26344 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/195103
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26343 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix the class name to match a rename that happened before commit.
Make the label size changes force the fields over, but not string the fields (window grows).
Make the url description autosize to the text it gets.
Make the window track the height needed for the url description field.
BUG=21893
TEST= The url description text should stay under the url box in all languages. The labels should never clip or overlap the text fields.
Review URL: http://codereview.chromium.org/195096
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26342 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=21919
TEST=covered by layout test
Review URL: http://codereview.chromium.org/201123
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26341 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We statically link this.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/193120
TBR=nick@chromium.org
Review URL: http://codereview.chromium.org/205020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26339 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TBR=tim
TEST=none
BUG=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26338 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Patching in a 64-bit crash fix.
TBR=whesse@chromium.org
Review URL: http://codereview.chromium.org/206028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26334 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
http://src.chromium.org/viewvc/chrome?view=rev&revision=26326
It came out that your change didn't cause purify failure. I'd like to reland reverted change. Sorry.
TBR=sgk
Review URL: http://codereview.chromium.org/204037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26331 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=21978
TBR=agl
TEST=none
Review URL: http://codereview.chromium.org/206027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26330 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
XP Unit (purify) failed on r26326.
r26325: fine
http://build.chromium.org/buildbot/waterfall/builders/XP%20Unit%20(purify)/builds/5564 (please cut and paste the address or %20 will be URLencoded...)
r26326: failure
http://build.chromium.org/buildbot/waterfall/builders/XP%20Unit%20(purify)/builds/5563
TBR=sgk
Review URL: http://codereview.chromium.org/196138
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26329 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/195100
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26328 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Looks like without this, pattern match doesn't work on reliability test.
Review URL: http://codereview.chromium.org/193124
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26327 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
* Fix options in variants like SYMBOLS=1.
* Generated Makefiles auto-regenerate when .gyp files change.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/196129
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26326 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
disable file.
Add top six tests causing the most recent crashes/hangs on the mac valgrind ui bots.
BUG=21949
Review URL: http://codereview.chromium.org/196134
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26325 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=21958
TBR=agl
Review URL: http://codereview.chromium.org/196136
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26324 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=21958
TBR=agl
Review URL: http://codereview.chromium.org/206023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26323 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This change just adds a namespace. I wish this fixes.
TBR=pkasting
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/201132
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26322 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
It seems this "static_cast<GtkWidget*>(window)" has been moved from a static member "InfoBubble::Show()" to a non-static member "InfoBubble::Init()". So, we don't need this cast.
TBR=pkasting
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/204036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26321 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26320 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=brg
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/194125
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26318 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26316 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL fixes issue 21587: REGRESSION: Omnibox does nothing when pressing enter after a single character of input.
It's a regression of my CL: http://codereview.chromium.org/196020, which reverts the text of omnibox in OnAfterPossibleChanges(), when enter key is pressed:
...
// If the change is caused by an Enter key press event, and the event was not
// handled by IME, then it's an unexpected change and shall be reverted here.
// {Start|Finish}UpdatingHighlightedText() are called here to prevent the
// PRIMARY selection from being changed.
if (enter_was_pressed_ &&
(char_inserted_ == '\n' || char_inserted_ == '\r')) {
StartUpdatingHighlightedText();
SetTextAndSelectedRange(text_before_change_, sel_before_change_);
FinishUpdatingHighlightedText();
return false;
}
...
Unfortunately, this piece of code causes the char_inserted_ to be set to wrong value in HandleInsertText(), if the text has only one character. See the source code of HandleInsertText():
...
// Filter out new line and tab characters.
// |text| is guaranteed to be a valid UTF-8 string, so it's safe here to
// filter byte by byte.
//
// If there was only a single character, then it might be generated by a key
// event. In this case, we save the single character to help our
// "key-press-event" signal handler distinguish if an Enter key event is
// handled by IME or not.
if (len == 1)
char_inserted_ = text[0];
...
This CL uses a boolean |enter_was_inserted_| instead of char |char_inserted_| to record the enter key status, to make sure it won't be reset unexpectly.
The test has been updated to cover this case. An unexpected dns lookup issue in the test which causes a failure on Windows was also fixed.
BUG=21587: REGRESSION: Omnibox does nothing when pressing enter after a single character of input.
TEST=Input one character in omnibox and press enter, the default link matched with the character should be opened.
Review URL: http://codereview.chromium.org/200131
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26315 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/196135
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26313 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
locally rewritten versions, so that the diff will be readable.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/196131
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26312 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This brings our extension support to about the level it has on linux.
One issue is that the toolstrips are webpages with a background image that just happens to look like the shelf they are on. But the background images are not updated on key->nonkey window changes, so the toolstrip backgrounds look slightly off in one of the two cases. If we decide to keep the shelf, we should fix this, but see the bug for erikkay's stance on this.
Also, the NTP is only loaded after all toolstrips have been loaded for some reason. That's what happens on the other platforms too, I believe.
The extension shelf uses the DownloadShelfView as background view for now.
Screenie: http://imgur.com/wSHgU.png
BUG=19073
TEST=Extensions that live in the shelf should show up. They should be clickable, resize correctly (e.g. the build status extension), and the shelf should interact in a sane way with the status bubble.
Review URL: http://codereview.chromium.org/175025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26311 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/193122
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26310 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
xvfb.
BUG=15445
Review URL: http://codereview.chromium.org/204032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26309 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
browser.
BUG=9847
adapted from patch by <minyu.huang [at] gmail>
Review URL: http://codereview.chromium.org/203062
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26308 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=14899
TEST=https sites have a lock icon in the address field, plain http sites do not
Review URL: http://codereview.chromium.org/199072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26307 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
(It was previously flakly, so this isn't a huge step backwards).
BUG=13907
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26306 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
- prevent newline injection attacks
- correctly unescape credentials provided in the URL
TEST=Covered by net_unittests.
http://crbug.com/20336
Review URL: http://codereview.chromium.org/183046
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26305 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
error: cannot convert 'scoped_nsobject<N>' to 'objc_object*' in argument passing
Review URL: http://codereview.chromium.org/204028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26304 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Why: Simpler build code. If everybody includes it, it should be included automatically.
Why now: The webkit chromium builds need it be specified, since can't default to build/common.gypi.
What was done:
1. build/common.gypi's contents were moved to a new file build/gyp_chromium.gypi
2. tools/gyp/gyp_chromium was moved to build/gyp_chromium and made to automatically include build/gyp_chromium.gypi.
3. lots of gyp files were fixed to not refer to build/common.gypi any more.
4. o3d which also builds independently of chrome, was fixed to have a gyp_o3d that includes gyp_chromium.gypi too.
5. build/common.gypi was left empty, because there are some external projects that still refer to it.
Things that are left to do after this patch is in:
1. The following external files (in other repositories) need to stop include common.gypi
./third_party/hunspell/hunspell.gyp
./third_party/icu/icu.gyp
./v8/tools/gyp/v8.gyp
2. Once nobody refers to common.gypi anymore, delete common.gypi
-or-
Delete gyp_chromium.gypi and move its content back to common.gypi
Tested on mac, win and linux. On win, got a few unit tests errors on chrome bookmarks, which should not be related. I'm running again with clobber to verify.
Review URL: http://codereview.chromium.org/206006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26302 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
We statically link this.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/193120
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26300 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/204031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26299 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Landing of Anton Muhin's patch (antonm@google.com)
(Reland. First landed in r26264, reverted in r26276. This reland hopefully
doesn't build tcmalloc on Linux.)
Review URL: http://codereview.chromium.org/206017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26298 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This should help debug the linked bug, or at least rule out some possibilities.
TEST=none
BUG=21925
Review URL: http://codereview.chromium.org/196132
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26297 0039d316-1c4b-4281-b951-d872f2087c98
|