summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix linux unit tests by checking for NULL that's impossible in real code, ↵erg@google.com2009-07-161-2/+2
| | | | | | | | | | but needed in unit tests. TBR=willchan Review URL: http://codereview.chromium.org/155663 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20917 0039d316-1c4b-4281-b951-d872f2087c98
* GTK Themes: Make the location bar area look native.erg@google.com2009-07-1611-38/+218
| | | | | | | | | | | | | | | | | Replace the star and go buttons with a native looking thing, where we manually draw the theme's button image for the full omnibox area and then position the star/go buttons on top of that. Also adds stand in versions of star_noborder and starred_noborder until Glen can make real icons. http://crbug.com/16227 http://crbug.com/13967 http://crbug.com/16915 Review URL: http://codereview.chromium.org/149681 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20914 0039d316-1c4b-4281-b951-d872f2087c98
* Do not set/unset magic string during setup.exe patching step.kuchhal@chromium.org2009-07-161-7/+0
| | | | | | | | | BUG=NONE TEST=Make 3 stage updater fail by deleting <chrome>\Application\Installer\chrome.7z file and make sure "ap" key still contains "-full". Review URL: http://codereview.chromium.org/149776 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20912 0039d316-1c4b-4281-b951-d872f2087c98
* Adds window position save/restore support for multiple monitors on Mac.rohitrao@chromium.org2009-07-162-18/+59
| | | | | | | | | | | | Fixes a bug where windows would creep up the screen every time you quit/restarted. BUG=http://crbug.com/16176 BUG=http://crbug.com/12854 TEST=Windows should save/restore in the correct location, with either a single monitor or multiple monitors. Review URL: http://codereview.chromium.org/155647 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20911 0039d316-1c4b-4281-b951-d872f2087c98
* First step in abstracting extension api to common/shared "IDL" files ↵rafaelw@chromium.org2009-07-1612-174/+158
| | | | | | | | | | | | | | (jsonschema). In this step, All "normal" api calls and events (those that arrive or originate from the browser process) are routed through a single v8::Extension API Call ("StartRequest"). Additionally, internal string names for methods and events now match the js namespace (i.e. "RemoveTab" -> "tabs.remove"), in anticipation of having names be implicit from their api module & name in the api "IDL". TBR=aa,mpcomplete Review URL: http://codereview.chromium.org/149730 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20910 0039d316-1c4b-4281-b951-d872f2087c98
* Fix for the Mac download shelf crash when clearing all downloads.paul@chromium.org2009-07-166-4/+60
| | | | | | | | | | | | | We also now automatically hide the shelf when the downloads are cleared. BUG=15855,15893 TEST=Download a few things, clear all downloads and the shelf will empty and hide. Review URL: http://codereview.chromium.org/149732 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20907 0039d316-1c4b-4281-b951-d872f2087c98
* WebKit merge 45975:45985.japhet@chromium.org2009-07-161-1/+1
| | | | | | | | BUG=none TEST=green build git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20905 0039d316-1c4b-4281-b951-d872f2087c98
* Don't run MediaLeakTest.VideoBear under valgrind for nowdkegel@google.com2009-07-161-0/+2
| | | | | | | | | BUG=16945 TEST=run test_shell_tests under valgrind; they shouldn't hang. Review URL: http://codereview.chromium.org/149769 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20901 0039d316-1c4b-4281-b951-d872f2087c98
* Implement batch deletion in PasswordStoreMac.stuartmorgan@google.com2009-07-164-29/+132
| | | | | | | | | | | Add a new keychain helper search and refactor a bit to support the deletion. BUG=15070 TEST=Deleting passwords in Clear Browsing Data... will now work per the design doc (which means we clear passwords we created, and our metadata, but leave passwords created by other browsers alone). Review URL: http://codereview.chromium.org/149759 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20900 0039d316-1c4b-4281-b951-d872f2087c98
* GTK: Fix bookmark manager crash on startup introduced by r20565estade@chromium.org2009-07-162-9/+11
| | | | | | | TBR=munjal git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20899 0039d316-1c4b-4281-b951-d872f2087c98
* Show progress Window when importing profile from First Run dialog.kuchhal@chromium.org2009-07-162-5/+23
| | | | | | Review URL: http://codereview.chromium.org/149766 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20898 0039d316-1c4b-4281-b951-d872f2087c98
* Select correct default folder in bookmark editor gtk.estade@chromium.org2009-07-161-1/+1
| | | | | | | | | BUG=none TEST=right click in a dropdown from a folder on the bookmark bar, select add, you should see that the correct folder is seleted in the bookmark editor tree view. Add a bookmark, it should show up in the right place. Review URL: http://codereview.chromium.org/149725 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20897 0039d316-1c4b-4281-b951-d872f2087c98
* r20591 broke themed buttons. Fixing.avi@chromium.org2009-07-161-0/+9
| | | | | | | | | BUG=none TEST=install theme, check to see that buttons are themed. Review URL: http://codereview.chromium.org/155646 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20896 0039d316-1c4b-4281-b951-d872f2087c98
* Change getLanguage to detectLanguage, and incorporate corresponding changes, ↵sidchat@google.com2009-07-1611-27/+28
| | | | | | | | | | in tab extensions. Also, use ISO language (e.g. "fr" instead of "French") as return language. BUG=none TEST=Enable extensions, add the given toolstrip as extension. For a page, the language of the page should be returned as ISO language (e.g., "french" as "fr") Review URL: http://codereview.chromium.org/149627 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20895 0039d316-1c4b-4281-b951-d872f2087c98
* Fix Valgrind Linux leakidanan@chromium.org2009-07-161-0/+5
| | | | | | Review URL: http://codereview.chromium.org/155645 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20894 0039d316-1c4b-4281-b951-d872f2087c98
* Modify ARAB to use simpler data types in FillBuffer() calls.kylep@chromium.org2009-07-166-33/+24
| | | | | | | | BUG=16011 TEST=none Review URL: http://codereview.chromium.org/155615 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20892 0039d316-1c4b-4281-b951-d872f2087c98
* Clear the window delegate in dealloc. Just being safe, I don't think this ↵pinkerton@chromium.org2009-07-161-0/+4
| | | | | | | | | | fixes any known issues or breaks anything. BUG=none TEST=none Review URL: http://codereview.chromium.org/149702 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20891 0039d316-1c4b-4281-b951-d872f2087c98
* gtk: Handle RTL layout when rendering the dragged tab.jhawkins@chromium.org2009-07-161-2/+5
| | | | | | | | BUG=none TEST=Drag a tab out of the browser in a RTL language. The tab should be rendered on the right side of the container. Review URL: http://codereview.chromium.org/155643 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20890 0039d316-1c4b-4281-b951-d872f2087c98
* Reverting 20497.thomasvl@chromium.org2009-07-161-5/+1
| | | | | | | BUG=16512,16871 Review URL: http://codereview.chromium.org/149765 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20889 0039d316-1c4b-4281-b951-d872f2087c98
* GTK: Move icon parsing to UI thread.estade@chromium.org2009-07-162-14/+47
| | | | | | | | | | | | | Loading the data from disk remains on the background thread. This fix is pseudo-speculative. BUG=14746,14745,15244,15416 TEST=UI tests stop flaking out. Review URL: http://codereview.chromium.org/149764 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20888 0039d316-1c4b-4281-b951-d872f2087c98
* Valgrind doesn't have an option (yet) to put suppressions into its xmldkegel@google.com2009-07-162-5/+24
| | | | | | | | | | | | | output file, so generate them by hand, and always print them. Lets you copy and paste from the bot into the suppressions file rather than having to reproduce yourself, which is handy for hard-to-reproduce warnings. BUG=none TEST=watch the bot. See the suppressions. Review URL: http://codereview.chromium.org/149762 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20887 0039d316-1c4b-4281-b951-d872f2087c98
* Blacklist API change for allowing multiple rule matchesidanan@chromium.org2009-07-166-49/+265
| | | | | | | | | | | | | | | The findMatch function now returns a Match object which aggregates multiple entries into one, in case (which is expected to be common) that multiple rules match a given URL. Since the set of matches is highly dependent on the URL, the Match class replaces the old RequestData object to be added to URLRequests. BUG=16932 TEST=Blacklist* Review URL: http://codereview.chromium.org/149737 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20886 0039d316-1c4b-4281-b951-d872f2087c98
* Remove DCHECK() in FindInPageNotificationObserver::Observe that gets ↵robertshield@google.com2009-07-161-7/+8
| | | | | | | | | | | triggered when FindInPage automation messages come in too quickly. BUG=Multiple AutomationMsg_Find messages sent rapidly to Chrome (without waiting for the synchronous responses) will trigger this DCHECK(). TEST=Send many AutomationMsg_Find messages very quickly without waiting for responses. Ensure no DCHECKing. Review URL: http://codereview.chromium.org/155644 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20885 0039d316-1c4b-4281-b951-d872f2087c98
* Add return-error-from-new-stream-callback-in-full-frame-plugin back in to ↵japhet@chromium.org2009-07-161-0/+1
| | | | | | list of expected mac failures. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20884 0039d316-1c4b-4281-b951-d872f2087c98
* Enable isolated worlds for content scripts.abarth@chromium.org2009-07-169-12/+33
| | | | | | | | | | R=aa TEST=LayoutTests/http/tests/security/isolatedWorld/ (will be landing soon after this patch) BUG=12218 Review URL: http://codereview.chromium.org/155626 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20883 0039d316-1c4b-4281-b951-d872f2087c98
* Add back in some expected failures that I shouldn't have removed.japhet@chromium.org2009-07-161-0/+9
| | | | | | | | | BUG=none TEST=green build TBR=ericroman git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20882 0039d316-1c4b-4281-b951-d872f2087c98
* Call the DragSource* API on the RenderViewHost during a drag.pinkerton@chromium.org2009-07-161-2/+36
| | | | | | | | BUG=16909 TEST=Dragging thumbs on the NTP Review URL: http://codereview.chromium.org/155640 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20881 0039d316-1c4b-4281-b951-d872f2087c98
* Fix for bug where we would not create ExtensionFunctionDispatcheraa@chromium.org2009-07-165-0/+30
| | | | | | | | | | instances for ExtensionDOMUI if RenderView was reused. BUG=16853 TEST=Added browser test Review URL: http://codereview.chromium.org/155592 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20880 0039d316-1c4b-4281-b951-d872f2087c98
* Roll Webkit DEPS 45958:45975 and remove some items from test expectations.japhet@chromium.org2009-07-162-48/+12
| | | | | | | | | | BUG=none TEST=green build TBR=ericroman Review URL: http://codereview.chromium.org/149756 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20879 0039d316-1c4b-4281-b951-d872f2087c98
* Implement bulk password lookup API in PasswordStoreMac.stuartmorgan@google.com2009-07-164-76/+241
| | | | | | | | | | | | Refactor password merge to support the new search. Rename the fill-targeted search on the adapter for better clarity about the distinction between it and the new search. BUG=16485 TEST=none yet; UI doesn't exist. Review URL: http://codereview.chromium.org/149708 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20877 0039d316-1c4b-4281-b951-d872f2087c98
* Print only the focused frame. This makes more sense than trying to print ↵sverrir@chromium.org2009-07-1610-274/+216
| | | | | | | | | | | | | | all frames since most of the time you end up with ugly clipping and scroll bars on the printed page. This also fixes printing issue with print selection since we don't pick up the currently selected text if's not in the main frame. Also did some refactoring of the printing test in render_view_unittest. Mainly to reuse the new Image class. BUG=http://crbug.com/15250 TEST=Print pages with frames. Print selection when using multiple frames (one example in bug). Review URL: http://codereview.chromium.org/149644 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20876 0039d316-1c4b-4281-b951-d872f2087c98
* Add two ui_test suppressions, widen another, to make my home jaunty system happydkegel@google.com2009-07-161-2/+27
| | | | | | | | | BUG=16093,16908 TEST=run ui tests on Jaunty with ISP with highish packet loss, verify no getaddrinfo or cssyyparse valgrind warnings Review URL: http://codereview.chromium.org/155638 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20875 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify window_open_disposition.cc since we only require conversion fromdarin@chromium.org2009-07-162-30/+15
| | | | | | | | | | | | | WebNavigationPolicy. This is cleanup from the WebWidget patch that you reviewed yesterday. TEST=none BUG=none R=dglazkov Review URL: http://codereview.chromium.org/155637 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20874 0039d316-1c4b-4281-b951-d872f2087c98
* DevTools: Make 'toggle node search' mode work.pfeldman@chromium.org2009-07-1612-11/+84
| | | | | | | BUG=16824 Review URL: http://codereview.chromium.org/155639 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20873 0039d316-1c4b-4281-b951-d872f2087c98
* SingleSplitView: survive minimize / restore when resize changes leading bounds.pfeldman@chromium.org2009-07-161-8/+18
| | | | | | | | BUG=16855 Review URL: http://codereview.chromium.org/155629 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20872 0039d316-1c4b-4281-b951-d872f2087c98
* Update rebaselining tool to auto create new directory if it does not exist.victorw@chromium.org2009-07-161-2/+17
| | | | | | | | | TEST=rebaselining tool BUG=none Review URL: http://codereview.chromium.org/149718 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20871 0039d316-1c4b-4281-b951-d872f2087c98
* Patch from timurrrr: Re-factor valgrind scripts to add tsan supportdkegel@google.com2009-07-169-249/+521
| | | | | | | | | | First reviewed at http://codereview.chromium.org/125272 BUG=none TEST=none Review URL: http://codereview.chromium.org/155528 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20870 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a strict-aliasing issue in BlacklistStoreInput::ReadUInt.craig.schlenter@chromium.org2009-07-161-3/+3
| | | | | | Review URL: http://codereview.chromium.org/155622 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20869 0039d316-1c4b-4281-b951-d872f2087c98
* Linux omnibox suggestions, only force LTR on URLs.deanm@chromium.org2009-07-161-5/+15
| | | | | | | | | | | I had previously forced LTR on all match contents. These are usually URLs, but in the case of history they are the search query. Use the classification to only for LTR on match contents that are actually URLs. Review URL: http://codereview.chromium.org/155636 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20867 0039d316-1c4b-4281-b951-d872f2087c98
* Implement theme resetavi@chromium.org2009-07-162-13/+15
| | | | | | | | | BUG=none TEST=try clicking the "reset to default theme" button in the prefs; it'll reset things Review URL: http://codereview.chromium.org/155633 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20866 0039d316-1c4b-4281-b951-d872f2087c98
* Set theme immediately on change.avi@chromium.org2009-07-161-10/+15
| | | | | | | | | BUG=none TEST=reset theme; watch window immediately retheme Review URL: http://codereview.chromium.org/149752 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20865 0039d316-1c4b-4281-b951-d872f2087c98
* Properly let the SSL engine know when the underlying stream is closed.avi@chromium.org2009-07-161-2/+4
| | | | | | | | | BUG=http://crbug.com/14196 TEST=NONE (this fix is related to the bug but does not fix it) Review URL: http://codereview.chromium.org/155595 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20864 0039d316-1c4b-4281-b951-d872f2087c98
* Go back to using V8 1.2.14.2. Changing v8.gyp wasn't enough tokasperl@google.com2009-07-161-1/+1
| | | | | | | | | | fix the subtle code generation issues on Mac OS. BUG=none TEST=none Review URL: http://codereview.chromium.org/149751 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20863 0039d316-1c4b-4281-b951-d872f2087c98
* Work on RTL / BiDi Omnibox.deanm@chromium.org2009-07-166-27/+49
| | | | | | | | | Based on work from Evan Stade. Review URL: http://codereview.chromium.org/155567 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20862 0039d316-1c4b-4281-b951-d872f2087c98
* DevTools: Initial commit with docked devtools window support on Linux.pfeldman@chromium.org2009-07-164-13/+55
| | | | | | Review URL: http://codereview.chromium.org/149675 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20861 0039d316-1c4b-4281-b951-d872f2087c98
* Update V8 in Chromium to version 1.2.14.3. The only change is an kasperl@google.com2009-07-161-1/+1
| | | | | | | | | | | updated v8.gyp file that avoid some subtle code generation issue on Mac OS; see http://code.google.com/p/v8/issues/detail?id=404. BUG=none TEST=none Review URL: http://codereview.chromium.org/155627 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20860 0039d316-1c4b-4281-b951-d872f2087c98
* Reverting 20855.darin@chromium.org2009-07-1660-1155/+1061
| | | | | | Review URL: http://codereview.chromium.org/149744 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20859 0039d316-1c4b-4281-b951-d872f2087c98
* Workaround bogus GCC warningdarin@chromium.org2009-07-161-3/+4
| | | | | | | | | | | | | | | | | | | | | | I had a single parameter IPC message handler, which took an enumeration parameter. For some reason, GCC thought it was possible for the enum value to be used uninitialized: cc1plus: warnings being treated as errors /b/slave/sub-rel-linux/build/src/chrome/common/ipc_message_utils.h: In member function 'virtual void RenderWidget::OnMessageReceived(const IPC::Message&)': /b/slave/sub-rel-linux/build/src/chrome/common/ipc_message_utils.h:1244: warning: 'p.Tuple1<WebKit::WebTextDirection>::a' may be used uninitialized in this function /b/slave/sub-rel-linux/build/src/chrome/common/ipc_message_utils.h:1244: note: 'p.Tuple1<WebKit::WebTextDirection>::a' was declared here scons: *** [/b/slave/sub-rel-linux/build/src/sconsbuild/Release/obj/chrome/renderer/render_widget.o] Error 1 I am able to avoid this warning by adding an explicit branch. BUG=none TEST=none TBR=mpcomplete Review URL: http://codereview.chromium.org/155624 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20857 0039d316-1c4b-4281-b951-d872f2087c98
* Reverting 20854.darin@chromium.org2009-07-1660-1061/+1155
| | | | | | Review URL: http://codereview.chromium.org/155621 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20855 0039d316-1c4b-4281-b951-d872f2087c98
* Use WebWidget from the WebKit API. This change also makesdarin@chromium.org2009-07-1660-1155/+1061
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | use of WebKitClient (replacing WebWidgetDelegate from glue). The ripple effects of this change are rather large, but most of the impact is mechanical. The more interesting changes include: 1- Removing the WebWidget parameter from WebWidgetClient methods. This didn't matter at all to RenderWidget or RenderView, but it did cause some changes to be made to TestWebViewDelegate. Now, it is not possible to share a delegate implementation for both the WebView and a popup menu, so I have a second instance of the delegate owned by TestShell for use with popup menus. 2- Plumbing WebNavigationPolicy in place of WindowOpenDisposition was getting to be a pretty large change, so I stopped short of deleting WindowOpenDisposition. That way the Chrome side can remain mostly unmodified. I then added a mapping function to convert from WebNavigationPolicy to WindowOpenDisposition. 3- The IME methods on WebWidget were renamed (reviewed separately by hbono), and there is now an enum to specify the composition command (WebCompositionCommand). 4- I added IPC serialization for WebCompositionCommand and WebTextDirection, which cleaned up some code that was just using ints in IPC messages. R=jam BUG=16234 TEST=none Review URL: http://codereview.chromium.org/149620 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20854 0039d316-1c4b-4281-b951-d872f2087c98