summaryrefslogtreecommitdiffstats
path: root/chrome
Commit message (Collapse)AuthorAgeFilesLines
* LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL ↵mirandac@chromium.org2009-08-031-1/+1
| | | | | | | | | | | | | | | image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases. BUG= http://crbug.com/18140 TEST= none. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22213 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22254 Review URL: http://codereview.chromium.org/159711 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22267 0039d316-1c4b-4281-b951-d872f2087c98
* Implement the NSTextInput protocol.hbono@chromium.org2009-08-034-2/+315
| | | | | | | | | | | | | | | | This change implements the NSTextInput protocol to integrate dead-keys and IME support into Mac Chromium. Same as Linux, to improve compatibility with Windows Chrome, this change emulates IPC messages sent on Windows when we input characters to fix Issue 11952 and Issue 11981. Even though I notice we need more work for fixing edge cases (e.g. disabling IMEs on a password input) also on Mac, it is the good starting point. (Supporting edge-cases requires complicated code and it makes hard to review.) BUG=11952 "IME support is not implemented" BUG=11981 "Deadkeys do not work" BUG=16393 "Mac: Not able to insert any letter using "Special Characters" pallet" TEST=Open a web page which contains an <input> form (e.g. <http://www.google.com/>), type a '[{' key and an 'A' key on a Canadian-French keyboard, and see a Latin character "U+00E2" is displayed in the <input> form. TEST=Open a web page which contains an <input> form (e.g. <http://www.google.com/>), enable an Chinese Pinyin IME, type a 'W' key, type an 'O' key, and see a Chinese character is displayed in the <input> form. TEST=Open a web page which contains a <textarea> form, type a return key, and see a new line is inserted. Review URL: http://codereview.chromium.org/150206 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22262 0039d316-1c4b-4281-b951-d872f2087c98
* When a content script has a match pattern of http://*/*, weaa@chromium.org2009-08-032-5/+14
| | | | | | | | | | | | should run on web pages that have an IP address for their host. BUG=18256 TEST=Added unit test Review URL: http://codereview.chromium.org/159767 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22260 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Check the return value of file_util::ReadFileToString. It's ↵jhawkins@chromium.org2009-08-031-4/+13
| | | | | | | | | | | possible that the file will be removed from the filesystem inbetween the enumeration and reading of the file. In this instance, the script is removed from the script list. CID=3888 BUG=none TEST=none Review URL: http://codereview.chromium.org/159680 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22258 0039d316-1c4b-4281-b951-d872f2087c98
* Adds an alternate set of frame images.sky@chromium.org2009-08-027-6/+17
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/159716 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22255 0039d316-1c4b-4281-b951-d872f2087c98
* LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL ↵mirandac@chromium.org2009-08-021-1/+3
| | | | | | | | | | | | | image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases. BUG= http://crbug.com/18140 TEST= none. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22213 Review URL: http://codereview.chromium.org/159711 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22254 0039d316-1c4b-4281-b951-d872f2087c98
* Move alternate error page loading out of WebFrame.darin@chromium.org2009-08-026-69/+244
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the RenderView be in charge of loading alternate error pages. While working on this change, I noticed several related bugs: 1- Loading an URL with an invalid host name from the new tab page results in an error page. If you hit back and then forward, you will be left with an empty location bar. In a debug build this trips an assertion in ClassifyNavigation because the given page_id is -1. This problem is caused by not duplicating the NavigationState of the failed load when creating a load for the error page. Hence, the pending_page_id of the forward navigation is lost. 2- Loading an URL with an invalid host name as a subframe results in an extra navigation in session history. One navigation for the main frame and one navigation for the error page load. This is another symptom of the problem described in #1. However, the solution is different. Here, we need to know that the subframe load is an AUTO_SUBFRAME load so that we load the error page using 'replace' semantics (so that WebCore does not generate a new session history entry). Finally, I decided to restrict alternative DNS error pages to only work for the main frame to match what we do for alternative 404 error pages. It doesn't seem worth it to show link doctor results for subframes since their primary purpose is to assist people who mis-type an URL. R=tony,brettw BUG=15648 TEST=covered by errorpage_uitest.cc Review URL: http://codereview.chromium.org/159575 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22253 0039d316-1c4b-4281-b951-d872f2087c98
* Hunspell 1.2.8 update caused valgrind warnings.mhm@chromium.org2009-08-021-0/+2
| | | | | | | | | BUG=18254 TEST=None Review URL: http://codereview.chromium.org/159766 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22252 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r22080 and re-enable tcmalloc.huanr@chromium.org2009-08-022-2/+4
| | | | | | Review URL: http://codereview.chromium.org/159771 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22251 0039d316-1c4b-4281-b951-d872f2087c98
* Add renderer histograms for late binding of sockets.willchan@chromium.org2009-08-021-0/+22
| | | | | | Review URL: http://codereview.chromium.org/159759 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22250 0039d316-1c4b-4281-b951-d872f2087c98
* Rolling back change 22245.chrome-bot@google.com2009-08-0225-715/+128
| | | | | | | | | | | | Too much red :-( BUG=None TEST=None TBR=erikkay Review URL: http://codereview.chromium.org/159763 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22247 0039d316-1c4b-4281-b951-d872f2087c98
* mole expand/collapse API, callable from a toolstrip for manual operationerikkay@chromium.org2009-08-0125-128/+715
| | | | | | Review URL: http://codereview.chromium.org/160276 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22245 0039d316-1c4b-4281-b951-d872f2087c98
* Try again to land: http://codereview.chromium.org/160483aa@chromium.org2009-08-0112-190/+377
| | | | | | | | | | | | | | | | | | Ever closer. Extract a client interface out of CrxInstaller and use it to implement ExtensionInstallUI. There is (still) no dialog here. But the next CL will, um, definitely have it. Also, fixed the issue with theme preview infobars not updating if you install a theme while another one is already previewed. TBR=mpcomplete@chromium.org BUG=17932 Review URL: http://codereview.chromium.org/160501 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22244 0039d316-1c4b-4281-b951-d872f2087c98
* Update Hunspell to the latest stable version to use the latest bdict format.mhm@chromium.org2009-08-0132-2777/+5233
| | | | | | | | | | Updated Hunspell to version 1.2.8 which properly deals with UTF8 and fixes many bugs. This CL will use the BDict format and remove the usage of FileMgr. Removed the unwanted "key" parameter constructors from hunspell since we are managing them through bdict. Removed all line numbers from the errors since we don't support that. BUG= 14756 (http://crbug.com/14756) TEST= Compiled Hunspell, Compiled Chromium, Ran Chromium, Fixed some of my spelling mistakes. Ran unit tests for SpellCheckTest.* Review URL: http://codereview.chromium.org/155841 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22243 0039d316-1c4b-4281-b951-d872f2087c98
* GTK Themes: Fix painting issues where GTK theme engine was painting over ↵erg@chromium.org2009-08-011-12/+12
| | | | | | | | | | | | | | buttons. So it looks like some themeing engines in distros more recent than Hardy paint random boxes of the default background if you don't pass an explicit clip area to gtk_paint_shadow(). http://crbug.com/18129 Review URL: http://codereview.chromium.org/159754 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22242 0039d316-1c4b-4281-b951-d872f2087c98
* retry r2226, with a fix for a case where we could double free.estade@chromium.org2009-08-016-29/+102
| | | | | | Review URL: http://codereview.chromium.org/160495 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22241 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: Fix Valgrind error in VisitedLinkEventsTest.Coalescense.derat@chromium.org2009-08-013-1/+11
| | | | | | | | | | | | | | I introduced the error in r22193; it shows up when there's no X server to connect to. I get a "GLib-GObject-CRITICAL **: g_object_get: assertion `G_IS_OBJECT (object)' failed" message with this change, but I'm not sure if it's from my code or if it was already there, and the tests still pass. TEST=ran VisitedLinkEventsTest.Coalescense under Valgrind with DISPLAY=:6 Review URL: http://codereview.chromium.org/159753 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22240 0039d316-1c4b-4281-b951-d872f2087c98
* Replace the guts of AudioRendererBase with calls to scaling algorithm.kylep@chromium.org2009-08-012-13/+7
| | | | | | | | BUG=16011 TEST=audio_renderer_base_unittest.cc and buffer_queue_unittest.cc Review URL: http://codereview.chromium.org/155695 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22237 0039d316-1c4b-4281-b951-d872f2087c98
* Fix compile error.nsylvain@chromium.org2009-08-011-1/+1
| | | | | | Review URL: http://codereview.chromium.org/159755 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22236 0039d316-1c4b-4281-b951-d872f2087c98
* revert 22226 as it caused some ui test crashesestade@chromium.org2009-08-016-103/+29
| | | | | | | | TBR=nsylvain Review URL: http://codereview.chromium.org/160491 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22235 0039d316-1c4b-4281-b951-d872f2087c98
* Fix unrelated bustage... ALLOW_THIS...ben@chromium.org2009-08-011-1/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22233 0039d316-1c4b-4281-b951-d872f2087c98
* revert 22215 which was testing an increased stack size.jam@chromium.org2009-08-011-1/+0
| | | | | | Review URL: http://codereview.chromium.org/159749 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22232 0039d316-1c4b-4281-b951-d872f2087c98
* Rollback 22228aa@chromium.org2009-08-0112-376/+189
| | | | | | TBR=mpcomplete@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22231 0039d316-1c4b-4281-b951-d872f2087c98
* Fix leak of GtkWidgets in NativeViewHost, and in doing so fix a crash on ↵ben@chromium.org2009-08-011-0/+6
| | | | | | | | | | | | | shutdown due to a check in OwnedWidgetGtk for unbalanced refcnt. The NativeViewHostGtk now owns the GtkWidget attached to it for the lifetime of its attachment. This removes my crazy ownership games from before and makes it a lot simpler. BUG=none TEST=none Review URL: http://codereview.chromium.org/159751 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22230 0039d316-1c4b-4281-b951-d872f2087c98
* Status bubble limping in TOOLKIT_VIEWS.ben@chromium.org2009-08-012-63/+25
| | | | | | | | | | | Add CreateTransparentFloatingWidget method to Widget and add Init/SetContentsView methods. BUG=none TEST=none Review URL: http://codereview.chromium.org/160474 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22229 0039d316-1c4b-4281-b951-d872f2087c98
* Ever closer. Extract a client interface out of CrxInstaller andaa@chromium.org2009-07-3112-189/+376
| | | | | | | | | | | | | | | use it to implement ExtensionInstallUI. There is (still) no dialog here. But the next CL will, um, definitely have it. Also, fixed the issue with theme preview infobars not updating if you install a theme while another one is already previewed. BUG=17932 Review URL: http://codereview.chromium.org/160483 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22228 0039d316-1c4b-4281-b951-d872f2087c98
* Typo fixestade@chromium.org2009-07-311-1/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22227 0039d316-1c4b-4281-b951-d872f2087c98
* GTK: Theme our link button.estade@chromium.org2009-07-316-29/+103
| | | | | | | | BUG=17772 Review URL: http://codereview.chromium.org/159748 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22226 0039d316-1c4b-4281-b951-d872f2087c98
* Fix some character literals to use the right escape string.pkasting@chromium.org2009-07-311-2/+2
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/159714 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22222 0039d316-1c4b-4281-b951-d872f2087c98
* Since bookmark copy/paste is not impl on non-Windows platforms, don't remove ↵estade@chromium.org2009-07-311-0/+5
| | | | | | | | | | | | | nodes when cutting. BUG=18152 TEST=none TBR=erg Review URL: http://codereview.chromium.org/160488 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22221 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing rpm.include file to build output.mmoss@chromium.org2009-07-311-1/+2
| | | | | | | Review URL: http://codereview.chromium.org/159747 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22219 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a bug where we tried to get the position of the tabstriptc@google.com2009-07-312-9/+18
| | | | | | | | | | | | | but the tabstrip wasn't in our widget hierarchy for app mode windows. Now we always put the tabstrip in the hierarchy, even if we don't show it. BUG=18202 Review URL: http://codereview.chromium.org/159745 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22218 0039d316-1c4b-4281-b951-d872f2087c98
* Fix or/|| mistake in ifdefstuartmorgan@google.com2009-07-311-1/+1
| | | | | | | | | | TBR=mark BUG=none TEST=none Review URL: http://codereview.chromium.org/160487 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22216 0039d316-1c4b-4281-b951-d872f2087c98
* Try increasing the stack size of the main thread, to see if it changes ↵jam@chromium.org2009-07-311-0/+1
| | | | | | | | chromebot crashes. Just a temporary change, I'll revert after a chromebot run. Review URL: http://codereview.chromium.org/160486 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22215 0039d316-1c4b-4281-b951-d872f2087c98
* CHECK that IOBuffer::data_ in AsyncResourceHandler is not NULL before ↵willchan@chromium.org2009-07-311-0/+2
| | | | | | | | | | caching it. BUG=http://crbug.com/16371 Review URL: http://codereview.chromium.org/159730 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22214 0039d316-1c4b-4281-b951-d872f2087c98
* LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL ↵mirandac@chromium.org2009-07-311-0/+2
| | | | | | | | | | | image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases. BUG= http://crbug.com/18140 TEST= none. Review URL: http://codereview.chromium.org/159711 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22213 0039d316-1c4b-4281-b951-d872f2087c98
* Convert parts of SavePage UI tests to more reliable browser tests.phajdan.jr@chromium.org2009-07-315-50/+134
| | | | | | | | | | | Use a real notification for download completion - that's the main unflakying point. TEST=none http://crbug.com/3791 Review URL: http://codereview.chromium.org/160480 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22212 0039d316-1c4b-4281-b951-d872f2087c98
* This CL adds accelerators to the Linux toolkit views.jcampan@chromium.org2009-07-317-13/+14
| | | | | | | | | | | The MessageLoop had to be modified to support Dispatchers on Linux. BUG=None TEST=On Windows and Linux, make sure the accelerators still work as expected. On Linux toolkit views, build and run the unit-tests. Review URL: http://codereview.chromium.org/159046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22210 0039d316-1c4b-4281-b951-d872f2087c98
* Switch on plugins by default for the Macstuartmorgan@google.com2009-07-311-1/+1
| | | | | | | | | BUG=10809 TEST=Visit a sit with Flash content. Playback should work, interacting with the plugin is still buggy. Review URL: http://codereview.chromium.org/159729 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22209 0039d316-1c4b-4281-b951-d872f2087c98
* Revert changes from r22198 to r22204 and r22194. None of these changes werekuchhal@chromium.org2009-07-3116-326/+367
| | | | | | | | | supposed to be commited. Seems like git took all the temporary commits from the base branch and uploaded them to svn. Review URL: http://codereview.chromium.org/159736 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22206 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: Fix cast breakage from 22193 (hopefully).derat@chromium.org2009-07-311-3/+6
| | | | | | Review URL: http://codereview.chromium.org/159737 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22205 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing space to command line that registers Chrome on Windows.kuchhal@chromium.org2009-07-311-1/+1
| | | | | | | | | | | So instead of --register-chrome-browser="<path>\chrome.exe"--register-chrome-browser-suffix=".<user>" it is actually --register-chrome-browser="<path>\chrome.exe" --register-chrome-browser-suffix=".<user>". BUG=3641 TBR=huanr Review URL: http://codereview.chromium.org/160481 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22204 0039d316-1c4b-4281-b951-d872f2087c98
* fix unit testskuchhal@chromium.org2009-07-311-27/+50
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22203 0039d316-1c4b-4281-b951-d872f2087c98
* Fix distribution key bug.kuchhal@chromium.org2009-07-313-26/+40
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22202 0039d316-1c4b-4281-b951-d872f2087c98
* eols cleanupkuchhal@chromium.org2009-07-312-10/+10
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22201 0039d316-1c4b-4281-b951-d872f2087c98
* cleanupkuchhal@chromium.org2009-07-316-120/+93
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22200 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "add second icon"kuchhal@chromium.org2009-07-311-1/+0
| | | | | | This reverts commit 03094992f89be349b19477596b82bfc165fd9825. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22199 0039d316-1c4b-4281-b951-d872f2087c98
* Refactoring of master preferences parsing before adding a new preference.kuchhal@chromium.org2009-07-3114-291/+240
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22198 0039d316-1c4b-4281-b951-d872f2087c98
* Enable url request mocks in browser tests.phajdan.jr@chromium.org2009-07-311-0/+4
| | | | | | | | | | | They will be used by my new browser test. TEST=none BUG=none Review URL: http://codereview.chromium.org/160464 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22197 0039d316-1c4b-4281-b951-d872f2087c98
* add second iconkuchhal@chromium.org2009-07-311-0/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22194 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-16 00:49:01 +0000