summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Enable onhttpequiv again. The difference this time around is that wetommi@chromium.org2010-02-053-136/+83
| | | | | | | | | | | | | perform the switch in onhttpequiv when done==false. Also, we don't rely on having access to the already existing moniker but instead create our own and DCHECK that it is IsRunning() (see comments). TEST=The double loading effect we were seeing before should be gone. BUG=33332 Review URL: http://codereview.chromium.org/576017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38232 0039d316-1c4b-4281-b951-d872f2087c98
* Don't call View::DidChangeBounds() when the bounds didn't changedavemoore@chromium.org2010-02-051-6/+7
| | | | | | | | | Bug: None Test: Views based UI should still layout correctly Review URL: http://codereview.chromium.org/574013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38231 0039d316-1c4b-4281-b951-d872f2087c98
* Adding the URLRequestTest.CookiePolicy_ForceSession test to the list of ↵ananta@chromium.org2010-02-051-0/+1
| | | | | | | | | | | | disabled tests as it does not apply to the host network stack. TBR=amit Review URL: http://codereview.chromium.org/572040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38230 0039d316-1c4b-4281-b951-d872f2087c98
* Don't need ipc_message.h any morecmasone@google.com2010-02-052-16/+0
| | | | | | Review URL: http://codereview.chromium.org/577025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38229 0039d316-1c4b-4281-b951-d872f2087c98
* This CL adds support for displaying Chrome InfoBars in pages rendered via ↵ananta@chromium.org2010-02-056-26/+88
| | | | | | | | | | | | | | | | | | ChromeFrame. To achieve this the ExternalTabContainer now creates a view which comprises of the InfoBarContainer and the TabContentsContainer view. It uses the GridLayout manager to layout this view. The InfoBarContainer no longer depends on the BrowserView. It now passes notifications to the InfoBarContainerDelegate interface which is implemented by the BrowserView and the ExternalTabContainer. Fixes bug http://code.google.com/p/chromium/issues/detail?id=24051 Bug=24051 Review URL: http://codereview.chromium.org/573022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38228 0039d316-1c4b-4281-b951-d872f2087c98
* Add Single Sign On support to HTTP Authentication handlers.cbentzel@chromium.org2010-02-0521-185/+457
| | | | | | | | | | | | | | | Currently this is implemented on Windows for the NTLM and Negotiate schemes. This CL does not introduce the hooks to actually use Single Sign On in response to a 401/407 request - that will come in a later CL. This behavior is disabled for now as well. BUG=29862 TEST=Ran unittests, and Chrome against a server with authentication challenges. Review URL: http://codereview.chromium.org/555174 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38227 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crasher in LocationBarView when browser is closed with page action ↵mirandac@chromium.org2010-02-054-19/+31
| | | | | | | | | | install bubble showing. BUG= 34282 TEST= in release version, create a page action. close browser window while page action install bubble is showing. no crash, ever. Review URL: http://codereview.chromium.org/561028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38225 0039d316-1c4b-4281-b951-d872f2087c98
* Tweak the crash dump flags to make sure we get PEB/TEB and the unloaded ↵siggi@chromium.org2010-02-051-1/+20
| | | | | | module list where available. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38224 0039d316-1c4b-4281-b951-d872f2087c98
* Show the filebrowse ui rather than the download shelf in chromeos.xiyuan@chromium.org2010-02-058-26/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This cl displays the filebrowse ui rather than download shelf for downloads in chrom(e|ium) os. It conditionally replaces (with preprocessor macros) the Browser::OnStartDownload method to do this. The cl adds a static FileBrowseUI::OpenPopup(profile, hashArgument), which opens the file browse ui and passes it the provided hash argument. This is invoked directly from Browser::OnStartDownload. The USBMountObserver code was changed to call this static method, rather than open the popup by hand as it had been doing. I'm not sure about ownership of the Browser* returned by OpenPopup, but based on other code in the tree I assume chrome will deal with freeing it when appropriate. Before this change, USBMountObserver would add the window to registrar_ *before* showing it. Now that FileBrowseUI::OpenPopup returns a which which is already visible, this is no longer the case. I assume this won't be a problem. Commit this for rginda. Original review: http://codereview.chromium.org/555167 BUG=none TEST=none Review URL: http://codereview.chromium.org/564022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38222 0039d316-1c4b-4281-b951-d872f2087c98
* Prepare for running nacl_ui_test on Mac and Linux.gregoryd@google.com2010-02-051-0/+6
| | | | | | | | This is the NaCl-Chrome integration test that runs in Chrome waterfall, currently on Windows only. I am working on starting new bots that will run it on Mac and Linux. Review URL: http://codereview.chromium.org/563021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38218 0039d316-1c4b-4281-b951-d872f2087c98
* Update my watchlist.finnur@chromium.org2010-02-051-0/+9
| | | | | | | | | BUG=None TEST=None (no code change) Review URL: http://codereview.chromium.org/577023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38217 0039d316-1c4b-4281-b951-d872f2087c98
* Make Carbon plugin idle event source robust against changes during iterationstuartmorgan@chromium.org2010-02-051-57/+62
| | | | | | | | | | | | | Ensure that removing any plugin from the idle event source during iteration is safe (not just the currently-firing plugin). Also bullet-proofs against the possibility of nested iteration. Refactors the idle event source to use a helper class, to reduce duplicate code. BUG=33467 TEST=Scroll rapidly on a page with multiple instances of a Cabon plugin. Review URL: http://codereview.chromium.org/575023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38216 0039d316-1c4b-4281-b951-d872f2087c98
* Makes example.com resolve to localhost for test.sky@chromium.org2010-02-051-0/+2
| | | | | | | | | | BUG=32845 TEST=none TBR=erikkay Review URL: http://codereview.chromium.org/573029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38215 0039d316-1c4b-4281-b951-d872f2087c98
* Mark inspector\console-log-before-inspector-open.html as flaky.yurys@chromium.org2010-02-051-0/+1
| | | | | | | | | BUG=34759 TBR=pfeldman Review URL: http://codereview.chromium.org/577022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38214 0039d316-1c4b-4281-b951-d872f2087c98
* Split out of reverted change ↵joth@chromium.org2010-02-053-13/+23
| | | | | | | | | | | | | http://src.chromium.org/viewvc/chrome?view=rev&revision=38207 Commit fixes to code under test, whilst tests themselves are still pending. BUG=http://crbug.com/11246 TEST=see http://codereview.chromium.org/578006 Review URL: http://codereview.chromium.org/571014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38213 0039d316-1c4b-4281-b951-d872f2087c98
* Wires TabContents to app extensions.sky@chromium.org2010-02-0511-58/+174
| | | | | | | | | BUG=32845 TEST=none Review URL: http://codereview.chromium.org/566032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38212 0039d316-1c4b-4281-b951-d872f2087c98
* Add bugs, update, and remove TODOs that are no longer validpinkerton@chromium.org2010-02-0517-77/+20
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/577021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38211 0039d316-1c4b-4281-b951-d872f2087c98
* Avoid out-of-bounds array access.cevans@chromium.org2010-02-051-1/+2
| | | | | | | | | TEST=NONE BUG=34710 Review URL: http://codereview.chromium.org/572031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38209 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 38207 - bah. still can't work out why this passes fine on local ↵joth@chromium.org2010-02-055-416/+13
| | | | | | | | | | | | | | | | | | | | | machine & try bots. Reattempt at http://src.chromium.org/viewvc/chrome?view=rev&revision=37989 (asserts tidy up slit out into its own change http://codereview.chromium.org/578013/show) Add tests for the geolocation network provider. Also some small tidy up a few other files. BUG=http://crbug.com/11246 TEST=unit_tests.exe gtest_filter=NetworkLocationProvider* gtest_break_on_failure Review URL: http://codereview.chromium.org/578006 TBR=joth@chromium.org Review URL: http://codereview.chromium.org/575021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38208 0039d316-1c4b-4281-b951-d872f2087c98
* Re-attempt at http://src.chromium.org/viewvc/chrome?view=rev&revision=37989 joth@chromium.org2010-02-055-13/+416
| | | | | | | | | | | | | | | (asserts tidy up slit out into its own change http://codereview.chromium.org/578013/show) Add tests for the geolocation network provider. Also some small tidy up a few other files. BUG=http://crbug.com/11246 TEST=unit_tests.exe --gtest_filter=NetworkLocationProvider* --gtest_break_on_failure Review URL: http://codereview.chromium.org/578006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38207 0039d316-1c4b-4281-b951-d872f2087c98
* WebKit side is in https://bugs.webkit.org/show_bug.cgi?id=34524jeremy@chromium.org2010-02-055-1/+105
| | | | | | | | | BUG=NONE TEST=Right clicking on a text input field in Web content on OS X should now show a Writing Direction submenu. Clicking on "Left To Right" should align the input to the left, and "Right To Left" should do the opposit. The "Default" menu item should always be disabled. Review URL: http://codereview.chromium.org/566031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38204 0039d316-1c4b-4281-b951-d872f2087c98
* Split out of reverted change ↵joth@chromium.org2010-02-055-9/+10
| | | | | | | | | | | | | | http://src.chromium.org/viewvc/chrome?view=rev&revision=37989 Tidy up assertions to use CHECKs (Already reviewed as part of above change) BUG=none TEST=none Review URL: http://codereview.chromium.org/578013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38203 0039d316-1c4b-4281-b951-d872f2087c98
* Updating trunk VERSION from 318.0 to 319.0chrome-release@google.com2010-02-051-1/+1
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38202 0039d316-1c4b-4281-b951-d872f2087c98
* Remove context accessor crash from known crashes after pushing V8ager@chromium.org2010-02-051-3/+0
| | | | | | | fix to chromium. Review URL: http://codereview.chromium.org/578012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38201 0039d316-1c4b-4281-b951-d872f2087c98
* Enable JAVASCRIPT_DEBUGGER flag. JSC-dependent parts of WebCore were guarded ↵mnaganov@chromium.org2010-02-051-1/+0
| | | | | | | | | | | | | with USE(JSC). TBR=pfeldman@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org/572032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38200 0039d316-1c4b-4281-b951-d872f2087c98
* Update V8 to version 2.1.0.ager@chromium.org2010-02-056-6/+21
| | | | | | | | | This version contains a number of bugfixes including wrapping of values when calling user-defined functions and a stricter JSON parser. Review URL: http://codereview.chromium.org/572033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38199 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enable a unittest which failed on valgrind thread sanitizer.eroman@chromium.org2010-02-051-1/+5
| | | | | | | | | | | | | TSAN had reported a race on the bool |should_block_|, between BlockableProxyResolver::GetProxyForURL() and BlockableProxyResolver::Unblock(). I avoided the problem by calling WaitUntilBlocked() before Unblock(), which is the same pardigm used by the other tests. Really, the best fix is to remove that limitation from BlockableProxyResolver... left that as a future TODO. TBR=willchan Review URL: http://codereview.chromium.org/561083 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38195 0039d316-1c4b-4281-b951-d872f2087c98
* Disable a test which is causing an error under valgrind TSAN.eroman@chromium.org2010-02-051-1/+1
| | | | | | | | | | It looks like there is a problem with the test setup (BlockableProxyResolver tests a bool outside of a lock). I will fix that next. TBR=willchan Review URL: http://codereview.chromium.org/578011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38194 0039d316-1c4b-4281-b951-d872f2087c98
* Remove size_t from audio IPC code.jam@chromium.org2010-02-0540-404/+401
| | | | | | | The change got to this size because I had to modify the surrounding code (I didn't want to just cast at the last minute). Review URL: http://codereview.chromium.org/577006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38192 0039d316-1c4b-4281-b951-d872f2087c98
* Tweak some things to be compatible with libpng 1.4.craig.schlenter@chromium.org2010-02-051-9/+4
| | | | | | | | | | | libpng 1.2 should still work too. BUG=32805 TEST=compiles on tryservers and locally against libpng 1.4 Review URL: http://codereview.chromium.org/569009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38188 0039d316-1c4b-4281-b951-d872f2087c98
* Support app menu and double-click close on upper left corner of window, part ↵pkasting@chromium.org2010-02-0517-192/+31
| | | | | | | | | | | | one. This makes things work right for windows that have an icon. The behavior of the main window is unchanged because right now it never returns HTSYSMENU anywhere. Part two of this change will be figuring out how to return that at the right place, which will make things Just Work. BUG=6961 TEST=Open an app or popup window. Single click the icon. The app menu should appear. Close it, then click-and-drag into the app menu. The item you release on should execute. Double-click the icon. The window should close. Review URL: http://codereview.chromium.org/577018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38187 0039d316-1c4b-4281-b951-d872f2087c98
* PGL was relying on pthread_key_create not returning zero, which does notsehr@google.com2010-02-051-6/+9
| | | | | | | | | seem to conform to the documented interface. I have added an allocated flag. This is needed to fix NaCl PGL. Review URL: http://codereview.chromium.org/576002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38186 0039d316-1c4b-4281-b951-d872f2087c98
* Implement launch disposition for extension-apps.rafaelw@chromium.org2010-02-0513-19/+150
| | | | | | | | | | | | This change adds an --app-id command switch that signifies that the extension-app with the given id should be launched according to its configuration. It also adds parsing for app.window_type in the manifest and the behavior that when installed and a desktop shortcut is created, the --app-id switch is used rather than the --app switch. BUG=32361 Review URL: http://codereview.chromium.org/573016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38184 0039d316-1c4b-4281-b951-d872f2087c98
* Add a timing measurement to LoadLog that shows how long a proxy resolve ↵eroman@chromium.org2010-02-053-3/+90
| | | | | | | | | request was stalled waiting to be scheduled to a thread. TEST=SingleThreadedProxyResolverTest.UpdatesLoadLogWithThreadWait Review URL: http://codereview.chromium.org/570019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38181 0039d316-1c4b-4281-b951-d872f2087c98
* Remove some tests that are now living upstream, and eliminatedarin@chromium.org2010-02-055-876/+33
| | | | | | | | | | | | some usage of glue_util.h. R=yaar BUG=none TEST=none Review URL: http://codereview.chromium.org/572013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38180 0039d316-1c4b-4281-b951-d872f2087c98
* Implement backend support for forcing cookies to be saved asdarin@chromium.org2010-02-057-12/+91
| | | | | | | | | | | | | | session cookies. Introduces a new CookiePolicy result code OK_FOR_SESSION_ONLY. R=eroman BUG=34571 TEST=none Review URL: http://codereview.chromium.org/577013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38179 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash when extensions try to set cookies. It is valid fordarin@chromium.org2010-02-051-15/+21
| | | | | | | | | | | | | the cookie policy to be NULL, in which case we should default to allowing cookies to be set. R=eroman BUG=34649 TEST=Try to install extension AniWeather from official gallery site Review URL: http://codereview.chromium.org/578007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38178 0039d316-1c4b-4281-b951-d872f2087c98
* Add unittest for r37566 (on PAC failure, should fallback to DIRECT).eroman@chromium.org2010-02-054-51/+26
| | | | | | | | | | | | To make it more testable, I had to move the fallback code in question from HttpNetworkTransaction to ProxyService. Although I think this is a better fit for that code anway, so it should be an overall readability improvement. BUG=32316 TEST=ProxyServiceTest.ProxyFallback_BadConfig, ProxyServiceTest.ProxyResolverFails Review URL: http://codereview.chromium.org/556087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38177 0039d316-1c4b-4281-b951-d872f2087c98
* More work on diagnostics modecpu@chromium.org2010-02-054-43/+106
| | | | | | | | | | | | | | | | - Clean up a bit, reduce windows specific code - Add disk space test - Add Chrome version test - Make OS version readable, cross platform mostly This concludes the minimal sanity tests. BUG=27885 TEST= run chrome with --diagnostics , no test should fail. Review URL: http://codereview.chromium.org/561078 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38176 0039d316-1c4b-4281-b951-d872f2087c98
* Relaxes the suppression rule for a leak in WorkerThreadableWebSocketChannel ↵hbono@chromium.org2010-02-051-2/+0
| | | | | | | | | | | (Take 2). BUG=32644 TBR=ukai TEST=none Review URL: http://codereview.chromium.org/572028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38175 0039d316-1c4b-4281-b951-d872f2087c98
* Now showing the browser action image (including badge) when dragging andfinnur@chromium.org2010-02-053-9/+36
| | | | | | | | | | | | showing the badge in the overflow menu. BUG=26990 TEST=Browser actions in container/overflow should show image and badge and also while dragging back and forth. Review URL: http://codereview.chromium.org/575016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38174 0039d316-1c4b-4281-b951-d872f2087c98
* System toast was trying to launch the per-user chromecpu@chromium.org2010-02-056-7/+11
| | | | | | | | | | | | | - forgot to pass along the system_install flag. BUG=32474 TEST=see bug for instructions, you need to simulate a system level install. TBR=huanr Review URL: http://codereview.chromium.org/561080 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38172 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 38166 - Added Star button to compact navigation mode.pkasting@chromium.org2010-02-0511-220/+240
| | | | | | | | | | | | | BUG=32326 TEST=Switch Chrome to compact mode by pressing CtrlShiftC. Test that Star icon works. Review URL: http://codereview.chromium.org/543215 TBR=avayvod@chromium.org Review URL: http://codereview.chromium.org/561081 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38171 0039d316-1c4b-4281-b951-d872f2087c98
* Add specific error codes for when SOCKS connect fails.eroman@chromium.org2010-02-056-21/+46
| | | | | | | | | | | | | This also fixes HttpNetworkTransaction to fallback to the next proxy after a failure is encountered using a SOCKS proxy server. Note that I decided against using finer granularity error codes, since it just adds a higher maintenance cost to ReconsiderProxyAfterError(). Power users can get better information on the cause of the failure by looking at the LoadLog anyway. BUG=34386 Review URL: http://codereview.chromium.org/567030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38170 0039d316-1c4b-4281-b951-d872f2087c98
* linux: Use gtk-cursor-blink from GtkSettings to set the caret blink intervaljoel@chromium.org2010-02-054-1/+31
| | | | | | | | | BUG=20772 TEST=Open gnome-keyboard-properties. Un-set the "Cursor Blinking" checkbox. Open a new tab in chromium and the cursor should not blink. Do the same with the slider, and the rate of blinking should vary. Review URL: http://codereview.chromium.org/398003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38168 0039d316-1c4b-4281-b951-d872f2087c98
* Fix regression that broke the --disable-ipv6 command line flag.eroman@chromium.org2010-02-053-32/+298
| | | | | | | | | | | | BUG=34632 TEST= HostResolverImplTest.SetDefaultAddressFamily_IPv4 HostResolverImplTest.SetDefaultAddressFamily_IPv6 HostResolverImplTest.SetDefaultAddressFamily_Synchronous Review URL: http://codereview.chromium.org/577010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38167 0039d316-1c4b-4281-b951-d872f2087c98
* Added Star button to compact navigation mode.avayvod@chromium.org2010-02-0511-240/+220
| | | | | | | | | | BUG=32326 TEST=Switch Chrome to compact mode by pressing Ctrl-Shift-C. Test that Star icon works. Review URL: http://codereview.chromium.org/543215 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38166 0039d316-1c4b-4281-b951-d872f2087c98
* Fix 33130 by checking whether the tab is closed whenhuanr@chromium.org2010-02-051-2/+3
| | | | | | | | | | | closing login propmt BUG=33130 TEST=existing test passes Review URL: http://codereview.chromium.org/576010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38165 0039d316-1c4b-4281-b951-d872f2087c98
* Bug 34167: GPU plugins' output initially placed incorrectly on pagekbr@google.com2010-02-054-0/+40
| | | | | | | | | | | | | | | | Added call to RenderView::DidMovePlugin upon creating the command buffer so the nested delegate is initially placed properly on the page; thanks to jam for this suggestion. Added necessary accessor to fetch the plugin window handle from the nested delegate, which is the "fake" GPU plugin window handle on Mac OS X. Verified that Pepper 3D test plugin initially shows up in the correct place. BUG=34167 TEST=none (ran Pepper test plugin on Mac and Linux) Review URL: http://codereview.chromium.org/561068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38164 0039d316-1c4b-4281-b951-d872f2087c98
* Mac: Set fixed maximum line height in Omnibox.viettrungluu@chromium.org2010-02-052-1/+22
| | | | | | | | | | | (Otherwise, e.g., Arabic text can make the line shift downwards.) BUG=31115 TEST=Make sure Omnibox works/looks/feels normal (and that normal text is displayed in it correctly). Then paste in something containing appropriate Arabic characters (see bug report for some samples); make sure text doesn't shift down. Review URL: http://codereview.chromium.org/573002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38163 0039d316-1c4b-4281-b951-d872f2087c98