| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=38521
TEST=None, Chrome should work as before.
Review URL: http://codereview.chromium.org/1525009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43751 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
(also fixes a lint error)
BUG=40577
TEST=none
Review URL: http://codereview.chromium.org/1600011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43750 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Contributed by slamm@google.com
BUG=40079
TEST=DiskCacheTest.AutomaticMaxSize
http://codereview.chromium.org/1564007
Review URL: http://codereview.chromium.org/1566021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43749 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/1568015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43748 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/1629002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43747 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the 10.6 SDK, -[NSWindow delegate] returns id<NSWindowDelegate>, not just
id, and the three selectors used in r43708 are not declared in
NSWindowDelegate. This results in errors such as "warning: '-themeProvider'
not found in protocol(s)".
Testing the selectors before using them is safe and not incorrect, but you
guys might actually want to make more of an assertion about what it means to
be a ChromeBrowserWindow's or FullscreenWindow's delegate, or perhaps even
a ChromeEventProcessingWindow's delegate. Alternatively, it may be
appropriate to add a ChromeThemedWindow layer as a subclass of CEPW and
superclass of CBW and FW. (CEPW's other subclass is InfoBubbleWindow.)
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/1630001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43746 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=ericbidelman
BUG=40063
TEST=none
Review URL: http://codereview.chromium.org/1539025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43745 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
platforms.
BUG=none
TEST=Open a new tab, navigate, and pin it. Open a new tab. Close the pinned tab. No tab background should be drawn, just the favicon.
Review URL: http://codereview.chromium.org/1562014
TBR=rsesek@chromium.org
Review URL: http://codereview.chromium.org/1627004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43744 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/1290001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43742 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=Open a new tab, navigate, and pin it. Open a new tab. Close the pinned tab. No tab background should be drawn, just the favicon.
Review URL: http://codereview.chromium.org/1562014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43741 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=40565
TEST=ctrl-b does nothing; ctrl-shift-b toggles bookmark bar.
Review URL: http://codereview.chromium.org/1567025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43740 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
ChromeLoggingTest along the way.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/1559016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43738 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=37072
TEST=Install the VLC plugin, and Play embedded VLC content; it should work correctly.
Review URL: http://codereview.chromium.org/1528020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43737 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This fixes checkPermission for chrome-extension: urls that are granted the permission via requestPermission (which is stored in the prefs).
BUG=40495
Review URL: http://codereview.chromium.org/1517018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43736 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
webkitNotifications.checkPermission() & requestPermission() went awry and notification_provider.cc was no longer implementing a called method. As a result requestPermission() became non-functional.
Review URL: http://codereview.chromium.org/1542010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43735 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
not be freeing the relative pidl. It is
a pointer into the absolute pidl. Thanks to Tommi for pointing this out.
Bug=none
Review URL: http://codereview.chromium.org/1568017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43734 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
checkboxes, also bumps the fudge factor to try and better handle kerning of some Europeans accents characters.
TEST=see bug
BUG=39383
Review URL: http://codereview.chromium.org/1622007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43733 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
notifications.
BUG=32847
TEST=create notification with a very long word, should be no horizontal scroll
Review URL: http://codereview.chromium.org/1613005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43732 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
if it was within the bounds of the plugin. This was because of the
SetCursorPatch for the flash plugin in Windows. The plugin periodically calls SetCursor and we only allow this call in the context of a HandleEvent
for a mouse event. This results in the SetCursor attempt by the plugin to fail. The reason we do this is to prevent annoying flicker if the plugin
in the background tab keeps changing the cursor.
Fix is to set the cursor only if has changed.
Fixes bug http://code.google.com/p/chromium/issues/detail?id=33549
Bug=33549
Review URL: http://codereview.chromium.org/1576020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43731 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=36171
TEST=none
Review URL: http://codereview.chromium.org/1048002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43730 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=40481
TEST=The leaking code is only called in a GoogleAuthenticator unit test, so I ran `sh tools/valgrind/chrome_tests.sh -t unit "--gtest_filter=GoogleAuthenticatorTest.*"` and it passed
Review URL: http://codereview.chromium.org/1606014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43729 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
gfx::GdkPixbufFromSkBitmap() returns an image with a refcount of 1.
Fixes heapcheck error seen on buildbots.
Review URL: http://codereview.chromium.org/1578012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43728 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=32362
TEST=Manual.
Review URL: http://codereview.chromium.org/1568012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43727 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This plumbs the actual dropEffect that occurred back into WebKit so that dragend is dispatched with the correct dropEffect. Note that this only helps with drags that start in Chrome and end outside of Chrome. Since WebDropTarget always lies and claims that no drop occurred, drags that start in Chrome and end in Chrome will still be incorrect.
BUG=39399
TEST=Manual testing using the attached test case on the bug.
Review URL: http://codereview.chromium.org/1136007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43726 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
binary anymore after I added some shutdown tests.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/1569022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43725 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
generates different signatures for stripped/unstripped binaries.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/1627001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43724 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=40229
TEST=Visit a page. Click down, then up on the location bar icon. The
dialog should only appear on mouse up.
Review URL: http://codereview.chromium.org/1621003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43723 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=38367
TEST=In the bug.
Review URL: http://codereview.chromium.org/1625002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43722 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(Retry with less build breakage this time.)
BUG=37375
TEST=out/Debug/unit_tests --gtest_filter=StatusTrayGtkTest.*
Patch from Thiago Farina <thiago.farina@gmail.com>
Review URL: http://codereview.chromium.org/1421001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43721 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
should.
BUG=39900
TEST=Open a new tab, navigate, and pin it. Close the window. Restore the window. Tab should still appear pinned.
Review URL: http://codereview.chromium.org/1628004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43720 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
invisible instead of disabled
Review URL: http://codereview.chromium.org/1591017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43719 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Refuse to load plugins that violate the specs.
BUG=38341
TEST=none
Review URL: http://codereview.chromium.org/1558020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43718 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
We shouldn't call memset nor socketpair once we have crashed.
BUG=39292
TEST=none
http://codereview.chromium.org/1210006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43717 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Don't switch to the plugin process for offscreen windows (like the one opened by Unity).
BUG=38996
TEST=Open a Unity sample. It shouldn't lose focus on every mouse event (although it will still lose focus once).
Review URL: http://codereview.chromium.org/1626002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43716 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=38352
TEST=none
Review URL: http://codereview.chromium.org/1120006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43715 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=40475
TEST=none
TBR=oshima@chromium.org
Review URL: http://codereview.chromium.org/1566018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43714 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
linux and mac after http://trac.webkit.org/changeset/57147 is rolled.
BUG=38918
Review URL: http://codereview.chromium.org/1596011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43712 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=37070
TEST=none
Review URL: http://codereview.chromium.org/1596007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43711 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43710 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
browser window
BUG=40090
TEST=none
Review URL: http://codereview.chromium.org/1567014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43709 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
controller. These are actually the same object, but they are set at different times during the switch to fullscreen.
BUG=39323
TEST=Go fullscreen with the Android Robot theme installed. Tab text for unselected tabs should be grey, not black.
Review URL: http://codereview.chromium.org/1576019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43708 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
generate menu item notifications for gtk menus.
The code to generate menu open and close events is a little more complicated
and will come in a future patch.
BUG=none
TEST=navigated menus with keyboard, watched notifications fire
Review URL: http://codereview.chromium.org/1585011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43707 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I hit the NOTREACHED() inside it while debugging an unrelated problem.
Also, now there is one less special case in this function's contract.
TEST=none
BUG=38048
Review URL: http://codereview.chromium.org/1315009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43706 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The normalization is needed to avoid duplicates in the language table in
the language config diloag, as well as the combobox in the "add language"
dialog, when engines return "jpn" and "ja" as language codes.
Along the way, fixed typos in function names.
TEST=added unit tests, and manually
Review URL: http://codereview.chromium.org/1512022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43705 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
to be necessary anymore.
BUG=36616
TEST=see the linux valgrind bot keeps green
TBR=dpranke
Review URL: http://codereview.chromium.org/1559017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43704 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43703 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This change just relaxes the suppression rule for Issue 40505 to cover Linux bots.
TBR=georgey
BUG=40505
TEST=make valgrind bots green.
Review URL: http://codereview.chromium.org/1561015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43702 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* changed DCHECK in tree_node_model to make it easy to find how it failed.
* Fixed compilation error in find_bar_host_interactive_uitest.cc on linux/views.
* Disabled BrowserKeyEventsTest.AccessKeys for Chromeos
Added/Changed ui_test_utils::RunAllPendingInMessageLoop where necessary.
* Disabled tab_dragging_tests. I included the source so that we can catch
compilation error even if tests do not run. Changed the comment from // -> /*
as GCC complains the use of "\" (and potentially dangerous)
* Added SleepInMessageLoop to allow event loop to execute the tasks/events
while waiting.
* For RunAllPendingInMessageLoop change, please see
http://codereview.chromium.org/1591004
BUG=39736
TEST=interactive_ui_tests should pass with following cls
http://codereview.chromium.org/1594003
http://codereview.chromium.org/1545011.
Review URL: http://codereview.chromium.org/1576008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43701 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BookmarkBarView tests fixes
* Add BOOKMARK_CONTEXT_MENU_SHOWN notification. On Linux, event handling is asynchronous, but the showing context menu is blocking,
so we can't use RunAllPending().
* Convert gtk keycode to Win when checking accelerator.
BUG=39736
TEST=none (interactive_ui_tests should pass once enabled
it'll be enabled by http://codereview.chromium.org/1576008
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=43699
Review URL: http://codereview.chromium.org/1545011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43700 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=40505
TEST= chromium os valgrind (4) should cycle green
TBR=mrossetti@chromium.org
Review URL: http://codereview.chromium.org/1562013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43699 0039d316-1c4b-4281-b951-d872f2087c98
|