summaryrefslogtreecommitdiffstats
path: root/chrome/browser/browser_shutdown.cc
Commit message (Collapse)AuthorAgeFilesLines
* Remove a bunch of easy entries from chrome\browser\DEPS.jam@chromium.org2012-01-311-1/+0
| | | | | | | | | | | | -hide PluginProcessHost from chrome by having extensions code just tell PluginService which plugin to kill -hide RenderProcessHostImpl from a test by using the interface instead -use TestBrowserThread in the SyncListenNotifications tool instead of BrowserThreadImp -remove some unnecessary mocks from chrome\browser\DEPS BUG=98716 Review URL: https://chromiumcodereview.appspot.com/9159059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119906 0039d316-1c4b-4281-b951-d872f2087c98
* get rid of static initializer shutdown_started_tbreisacher@chromium.org2012-01-271-4/+5
| | | | | | | | | | BUG=94925 TEST=sizes.py should report one less SI Review URL: http://codereview.chromium.org/9235081 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119371 0039d316-1c4b-4281-b951-d872f2087c98
* Remove PrefService::ScheduleSavePersistentPrefs and change ↵bauerb@chromium.org2011-12-271-1/+1
| | | | | | | | | | | | | SavePersistentPrefs calls to CommitPendingWrites. TBR=ncarter@chromium.org BUG=99306 TEST=none Review URL: http://codereview.chromium.org/8198007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115829 0039d316-1c4b-4281-b951-d872f2087c98
* Store the "browser autorestarted, last session must be restored" information ↵marja@chromium.org2011-12-051-4/+0
| | | | | | | | | | | | | | in a preference. This way ChromiumOS can run the same session restore code as Chrome. BUG=1870 TEST=NONE Review URL: http://codereview.chromium.org/8745015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112985 0039d316-1c4b-4281-b951-d872f2087c98
* Have content/ create and destroy its own threads. (Re-land)joi@chromium.org2011-11-281-1/+5
| | | | | | | | | | | | | | | | | | | Change embedding API and embedders to allow for this. Push inheritance of base::Thread down to content::BrowserThreadImpl so that content::BrowserThread is just a namespace for API functions. This change temporarily disables chrome_frame_net_tests as agreed by the CF lead, see bug 105435. TBR=ben@chromium.org (IWYU change only) BUG=98716,104578,105435 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111695 Reverted (problems on official bot): r111698 Review URL: http://codereview.chromium.org/8477004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111705 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 111695 - Have content/ create and destroy its own threads.joi@chromium.org2011-11-281-5/+1
| | | | | | | | | | | | | | | | | | | | | Reason: Problems on official builders. Change embedding API and embedders to allow for this. Push inheritance of base::Thread down to content::BrowserThreadImpl so that content::BrowserThread is just a namespace for API functions. This change temporarily disables chrome_frame_net_tests as agreed by the CF lead, see bug 105435. TBR=ben@chromium.org (IWYU change only) BUG=98716,104578,105435 Review URL: http://codereview.chromium.org/8477004 TBR=joi@chromium.org Review URL: http://codereview.chromium.org/8718012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111698 0039d316-1c4b-4281-b951-d872f2087c98
* Have content/ create and destroy its own threads.joi@chromium.org2011-11-281-1/+5
| | | | | | | | | | | | | | | | Change embedding API and embedders to allow for this. Push inheritance of base::Thread down to content::BrowserThreadImpl so that content::BrowserThread is just a namespace for API functions. This change temporarily disables chrome_frame_net_tests as agreed by the CF lead, see bug 105435. TBR=ben@chromium.org (IWYU change only) BUG=98716,104578,105435 Review URL: http://codereview.chromium.org/8477004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111695 0039d316-1c4b-4281-b951-d872f2087c98
* Define the public interface for content browser RenderProcessHost. This ↵ananta@chromium.org2011-11-181-2/+3
| | | | | | | | | | | | | | | interface is implemented by the RenderProcessHostImpl class which lives in content\browser\renderer_host\render_process_host_impl.cc/.h. The RenderProcessHostImpl class is a consolidation of the RenderProcessHost and BrowserRenderProcessHost classes. The RenderProcessHost public interface was created from the now deleted RenderProcessHost class defined in content\browser\renderer_host\render_process_host.h. Additional methods have been added to the interface to ensure that it works correctly with the MockRenderProcessHost class used by unit tests. I had to implement a number of overrides in the MockRenderProcessHost class to ensure that tests work correctly. This was because of assumptions in the tests that the MockRPH class was a real RPH which it was till now. Added a TODO for the methods which could potentially be removed from this interface. Will revisit that in a subsequent CL. BUG=98716 TEST=No change in functionality. Hopefully it all compiles and works. Review URL: http://codereview.chromium.org/8515027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110615 0039d316-1c4b-4281-b951-d872f2087c98
* Replace NewRunnableFunction with Bind in BrowserShutdown.achuith@chromium.org2011-11-031-2/+2
| | | | | | | | BUG=chromium-os:22024. TEST=compiles, tests pass. Review URL: http://codereview.chromium.org/8437023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108522 0039d316-1c4b-4281-b951-d872f2087c98
* Move BrowserThread to content namespace.joi@chromium.org2011-11-021-0/+1
| | | | | | | | | TBR=owners BUG=98716 Review URL: http://codereview.chromium.org/8437002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108270 0039d316-1c4b-4281-b951-d872f2087c98
* Split BrowserThread into public API and private implementation, step 1.joi@chromium.org2011-10-281-2/+2
| | | | | | | | | | | | | | | | | | | | | Only content/ now has the ability to create BrowserThread objects, with the exception that tests can create the content::TestBrowserThread subclass, and (temporarily) code in chrome/ can create the DeprecatedBrowserThread subclass. A follow-up change will make content/ take care of its own thread creation, remove DeprecatedBrowserThread, and move all state and non-trivial constructors from BrowserThread down to BrowserThreadImpl. Also moved BrowserProcessSubThread into content/ namespace. As part of follow-up cleanup, chrome/ will stop using this class. BUG=98716 TEST=existing Review URL: http://codereview.chromium.org/8392042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107718 0039d316-1c4b-4281-b951-d872f2087c98
* Chrome OS: Shutdown without blocking when SIGTERM is received.oshima@chromium.org2011-10-241-5/+8
| | | | | | | | | | | | | | | | | | | | | | | Made ShuttingDownWithoutCloseBrowsers a explicit flag. Use END_SESSION shutdown type if chrome recieves SIGTERM AND there are tabs that may block shutdown. Make sure APP_TERMINATING is sent only once. This fixes SIGABORT crash in two shutdown scenarios: 1) powering off when chrome has beforeunload handler, or downloads in progress. 2) singout from screen locker when chrome has beforeunload handler or downloads in progress. This is simple version of fix to merge to release branch. I'm working on another CL that will (hopefully) cleanup a bit more. BUG=chromium-os:20460 TEST=see bug for repro step Review URL: http://codereview.chromium.org/8262009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106988 0039d316-1c4b-4281-b951-d872f2087c98
* chromeos: Remove unnecessary includes of login_library.hsatorux@chromium.org2011-10-131-1/+0
| | | | | | | | | | | | This is a preparation work for moving login manager D-Bus code from libcros to Chrome. We want to clean things up before doing the migration. BUG=chromium-os:16555 TEST=chrome and tests build as before Review URL: http://codereview.chromium.org/8273013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105384 0039d316-1c4b-4281-b951-d872f2087c98
* Make profile deletion really remove stale profile directoryjeremy@chromium.org2011-10-061-0/+5
| | | | | | | | | | | | | | Before this CL, after deleting a profile and restarting the browser - the old profile directory would persist and still contained some files. This happened because the code to physically delete the profile directory was running before the profile object had been deleted and some files were being written back out to disk recreating the directory. This patch moves the "nuking" code late in the shutdown process to fix the order of these events. BUG=95079 TEST=See bug Review URL: http://codereview.chromium.org/8133002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104297 0039d316-1c4b-4281-b951-d872f2087c98
* Explicitly intialize AudioHandler and SystemKeyEventListenerstevenjb@chromium.org2011-09-291-12/+0
| | | | | | | | | | BUG=chromium-os:14058 TEST=All tests should pass without shutdown crashes (in these classes). Review URL: http://codereview.chromium.org/8060041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103359 0039d316-1c4b-4281-b951-d872f2087c98
* Modifying prefetch to account for multi-profile.rlp@chromium.org2011-09-101-5/+0
| | | | | | | | | | | | | | | | | | | | Items of note: - predictor_api is gone. Most functions in predictor_api.cc have moved into the chrome_browser_net::Predictor class or the Profile class. - The predictor state is cleaned up in the Profile dtor. - Predictor is owned by the ProfileIOData of the profile. - InitialObserver class is gone since each profile keeps their own info, the non-OTR don't care if anyone is OTR. - Predictor is created by the profile and then passed to the ProfileIOData. Then its initialization is finished on the IOThread. - ConnectInterceptor now subclasses off of UrlRequestJobFactory::Interceptor. - Updated Profile to create a SimpleShutdownPredictor with limited functionality when in unittests. BUG=89937,90114 TEST=passes existing Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97446 Review URL: http://codereview.chromium.org/7467012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100555 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Modifying prefetch to account for multi-profile."avi@chromium.org2011-08-191-0/+5
| | | | | | | | | | | | | | This appears to make all the CF bots crash in chrome_frame_net_tests. This reverts commit r97446. BUG=89937 TEST=n/a TBR=rlp Review URL: http://codereview.chromium.org/7685009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97467 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 97465 - Revert 97446 - Modifying prefetch to account for multi-profile.avi@chromium.org2011-08-191-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | Items of note: - predictor_api is gone. Most functions in predictor_api.cc have moved into the chrome_browser_net::Predictor class or the Profile class. - The predictor state is cleaned up in the Profile dtor. - Predictor is owned by the ProfileIOData of the profile. - InitialObserver class is gone since each profile keeps their own info, the non-OTR don't care if anyone is OTR. - Predictor is created by the profile and then passed to the ProfileIOData. Then its initialization is finished on the IOThread. - ConnectInterceptor now subclasses off of UrlRequestJobFactory::Interceptor. - Updated Profile to create a SimpleShutdownPredictor with limited functionality when in unittests. BUG=89937 TEST=passes existing Review URL: http://codereview.chromium.org/7467012 TBR=rlp@chromium.org Review URL: http://codereview.chromium.org/7690006 TBR=avi@chromium.org Review URL: http://codereview.chromium.org/7688006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97466 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 97446 - Modifying prefetch to account for multi-profile.avi@chromium.org2011-08-191-0/+5
| | | | | | | | | | | | | | | | | | | | | Items of note: - predictor_api is gone. Most functions in predictor_api.cc have moved into the chrome_browser_net::Predictor class or the Profile class. - The predictor state is cleaned up in the Profile dtor. - Predictor is owned by the ProfileIOData of the profile. - InitialObserver class is gone since each profile keeps their own info, the non-OTR don't care if anyone is OTR. - Predictor is created by the profile and then passed to the ProfileIOData. Then its initialization is finished on the IOThread. - ConnectInterceptor now subclasses off of UrlRequestJobFactory::Interceptor. - Updated Profile to create a SimpleShutdownPredictor with limited functionality when in unittests. BUG=89937 TEST=passes existing Review URL: http://codereview.chromium.org/7467012 TBR=rlp@chromium.org Review URL: http://codereview.chromium.org/7690006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97465 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r96364: Rename PluginUpdater to PluginPrefs and make it per-profile.bauerb@chromium.org2011-08-191-3/+0
| | | | | | | | | BUG=80794 TEST=none Review URL: http://codereview.chromium.org/7639017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97452 0039d316-1c4b-4281-b951-d872f2087c98
* Modifying prefetch to account for multi-profile.rlp@chromium.org2011-08-191-5/+0
| | | | | | | | | | | | | | | | | | Items of note: - predictor_api is gone. Most functions in predictor_api.cc have moved into the chrome_browser_net::Predictor class or the Profile class. - The predictor state is cleaned up in the Profile dtor. - Predictor is owned by the ProfileIOData of the profile. - InitialObserver class is gone since each profile keeps their own info, the non-OTR don't care if anyone is OTR. - Predictor is created by the profile and then passed to the ProfileIOData. Then its initialization is finished on the IOThread. - ConnectInterceptor now subclasses off of UrlRequestJobFactory::Interceptor. - Updated Profile to create a SimpleShutdownPredictor with limited functionality when in unittests. BUG=89937 TEST=passes existing Review URL: http://codereview.chromium.org/7467012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97446 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r96364, it's breaking a NaCl test.mnissler@chromium.org2011-08-111-0/+3
| | | | | | | | | | TEST= BUG= TBR=bauerb@chromium.org Review URL: http://codereview.chromium.org/7627001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96430 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 96409 - Speculatively revert r96364, it's the best bet on the NaCL ↵mrossetti@chromium.org2011-08-111-3/+0
| | | | | | | | | | | | | | | breakage. TEST= BUG= TBR=bauerb@chromium.org,mnissler@chromium.org Review URL: http://codereview.chromium.org/7627001 TBR=mnissler@chromium.org Review URL: http://codereview.chromium.org/7618013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96411 0039d316-1c4b-4281-b951-d872f2087c98
* Speculatively revert r96364, it's the best bet on the NaCL breakage.mnissler@chromium.org2011-08-111-0/+3
| | | | | | | | | | TEST= BUG= TBR=bauerb@chromium.org Review URL: http://codereview.chromium.org/7627001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96409 0039d316-1c4b-4281-b951-d872f2087c98
* Rename PluginUpdater to PluginPrefs and make it per-profile.bauerb@chromium.org2011-08-111-3/+0
| | | | | | | | | BUG=80794 TEST=none Review URL: http://codereview.chromium.org/7564006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96364 0039d316-1c4b-4281-b951-d872f2087c98
* touch: Always expect XInput2 availability.sadrul@chromium.org2011-07-221-3/+0
| | | | | | | | | BUG=80790 TEST=touch compiles Review URL: http://codereview.chromium.org/6975045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93621 0039d316-1c4b-4281-b951-d872f2087c98
* Fix relaunches on the Mac.mark@chromium.org2011-06-241-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Relaunches that go through browser_shutdown and relaunches after install-from-disk-image should both be improved. browser_shutdown relaunches include those that occur when Chrome detects a new version and offers to relaunch itself, and those that occur when you change about:flags and click "Relaunch Now." When relaunching Chrome in the same location via browser_shutdown, the same Dock tile should now always be reused, provided that the tile is persistent, meaning that the user has chosen "Keep in Dock" from the tile's contextual menu. The relaunched process' activation status (foreground or background application) should now track the original process' activation status. BUG=42962, 85073 TEST=1a. Change something in about:flags and hit "Relaunch Now" at the bottom. If you had a persistent Dock tile, the same tile should be used for the relaunched browser every time, without fail. 1b. Same, but once you're running a version of Chrome (including a canary) with this fix, when an update is applied and you relaunch from within the application via the "Update Google Chrome" item in the wrench menu or the "Relaunch" button in the about window, the same tile should be used for the relaunched browser every time, without fail. 2. Relaunch after install from disk image. Remove Chrome from /Applications and double-click the Chrome icon in a read-only disk image carrying a version that has this fix. (Our official or canary disk images will work). Chrome will offer to install itself. Let it. Upon completion of the installation, Chrome should launch out of /Applications properly. If you had a persistent Dock tile pointing to a copy in /Applications, it will be used. Activation status (foreground or background) in the relaunched browser will match the status in the process that had run out of the disk image at the time that the installation completed. Review URL: http://codereview.chromium.org/7215040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90371 0039d316-1c4b-4281-b951-d872f2087c98
* Remove g_browser_process dependency from content. Also make all the ↵jam@chromium.org2011-06-131-6/+0
| | | | | | | | | SyncChannels in the browser process ChannelProxy instead, since no code in the browser should send synchronous IPC messages. The renderer one was done like that a long time ago to send sync IPCs for accessibility, and we learnt that that doesn't work. BUG=76697 Review URL: http://codereview.chromium.org/7046100 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88806 0039d316-1c4b-4281-b951-d872f2087c98
* Listen to XI_HierarchyChanged events and call setxkbmap when needed.yusukes@google.com2011-06-031-0/+8
| | | | | | | | | | | | | | BUG=chromium-os:15851 BUG=chromium-os:15516 BUG=84694 TEST=manually done: Log in, change the layout from US Qwerty (US) to US Extended (EXTD), focus Omnibox, press AltGr+a, confirm á is shown, plug a USB keyboard, press AltGr+a, confirm the character again, press ctrl+alt+f2 to show VT2, press ctrl+alt+f1 to go back to X, press AltGr+a, confirm the char again, close lid, open lid, press AltGr+a, confirm the char again. TEST=On login screen, change the keyboard layout to EXTD, add "sleep 30" to /etc/init/udev-*.conf files, reboot, on login screen, wait for ~30 seconds, press AltGr+a, confirm á is shown. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=87585 Review URL: http://codereview.chromium.org/6975057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87756 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Listen to XI_HierarchyChanged events and call setxkbmap when needed."yusukes@google.com2011-06-021-5/+0
| | | | | | | | | | This reverts commit 0e193a0cd5d3e40eb5e387c3dac1bd6dc2572dab. BUG=chromium-os:15851 TEST=None TBR=sadrul@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87587 0039d316-1c4b-4281-b951-d872f2087c98
* Listen to XI_HierarchyChanged events and call setxkbmap when needed.yusukes@google.com2011-06-021-0/+5
| | | | | | | | | | | BUG=chromium-os:15851 BUG=chromium-os:15516 TEST=manually done: Log in, change the layout from US Qwerty (US) to US Extended (EXTD), focus Omnibox, press AltGr+a, confirm á is shown, plug a USB keyboard, press AltGr+a, confirm the character again, press ctrl+alt+f2 to show VT2, press ctrl+alt+f1 to go back to X, press AltGr+a, confirm the char again, close lid, open lid, press AltGr+a, confirm the char again. TEST=On login screen, change the keyboard layout to EXTD, add "sleep 30" to /etc/init/udev-*.conf files, reboot, on login screen, wait for ~30 seconds, press AltGr+a, confirm á is shown. Review URL: http://codereview.chromium.org/6975057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87585 0039d316-1c4b-4281-b951-d872f2087c98
* The ServiceProcessControl class is now a singleton and is not keyed off the ↵sanjeevr@chromium.org2011-06-021-3/+3
| | | | | | | | | | | | profile. BUG=80170 TEST=Unit-tests, Cloud Print proxy. Review URL: http://codereview.chromium.org/7074023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87580 0039d316-1c4b-4281-b951-d872f2087c98
* iwyu: Include stringprintf.h where appropriate, part 2.jhawkins@chromium.org2011-05-111-7/+7
| | | | | | | | | | | BUG=82098 TEST=none R=thakis@chromium.org Review URL: http://codereview.chromium.org/7004007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85048 0039d316-1c4b-4281-b951-d872f2087c98
* Observe plug-in blacklist changes in user preferences instead of local state.bauerb@chromium.org2011-04-281-0/+3
| | | | | | | | | | | Previously, we would read the plug-in blacklist from user prefs at startup, but watch for changes (only!) in local state, which just seems wrong. BUG=80025,45856 TEST=none Review URL: http://codereview.chromium.org/6898050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83324 0039d316-1c4b-4281-b951-d872f2087c98
* first-run: Pull IsBrowserAlreadyRunning() out of upgrade_util_win.h.tfarina@chromium.org2011-04-151-1/+2
| | | | | | | | | | | | | | This concept is not specific to upgrade path and thus shouldn't belong there. Move it to a new header file called browser_util_win.h as it's more appropriate. BUG=79203 TEST=None R=jhawkins@chromium.org Review URL: http://codereview.chromium.org/6861001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81691 0039d316-1c4b-4281-b951-d872f2087c98
* first-run: Eliminate the ifdefs in upgrade_util.htfarina@chromium.org2011-04-141-0/+1
| | | | | | | | | | | BUG=79203 TEST=None R=jhawkins@chromium.org Review URL: http://codereview.chromium.org/6853022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81659 0039d316-1c4b-4281-b951-d872f2087c98
* first-run: Reland refactor of Upgrade class into upgrade_util API.tfarina@chromium.org2011-04-131-4/+4
| | | | | | | | | | | | | This fix linux_chromeos build by adding an appropriate #ifdef on upgrade_util_linux.cc. BUG=79203 TEST=None TBR=jhawkins@chromium.org Review URL: http://codereview.chromium.org/6839021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81431 0039d316-1c4b-4281-b951-d872f2087c98
* Broke ARM compile.siggi@chromium.org2011-04-131-4/+4
| | | | | | | | Revert 81409 - first-run: Refactor Upgrade class into a common upgrade_util API.BUG=79203TEST=NoneR=jhawkins@chromium.orgReview URL: http://codereview.chromium.org/6840003 TBR=tfarina@chromium.org Review URL: http://codereview.chromium.org/6835022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81412 0039d316-1c4b-4281-b951-d872f2087c98
* first-run: Refactor Upgrade class into a common upgrade_util API.tfarina@chromium.org2011-04-131-4/+4
| | | | | | | | | | | BUG=79203 TEST=None R=jhawkins@chromium.org Review URL: http://codereview.chromium.org/6840003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81409 0039d316-1c4b-4281-b951-d872f2087c98
* first-run: Refactor Upgrade class.tfarina@chromium.org2011-04-121-1/+1
| | | | | | | | | | | | | | | - Split Upgrade class implementation per platform files outside of first_run implementations. - Move TryChromeDialog class into its own header file. - Rename TryChromeDialog to TryChromeDialogView. BUG=None TEST=None R=jhawkins@chromium.org Review URL: http://codereview.chromium.org/6824029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81313 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify MetricsService API for shutting downstuartmorgan@chromium.org2011-03-231-3/+1
| | | | | | | | | | | Infer clean shutdown from the fact that shutdown methods are being called, so clients don't have to make two MetricsService shutdown calls at each stage of shutdown. BUG=None TEST=Existing tests Review URL: http://codereview.chromium.org/6712020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79182 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify ResolveProxyMsgHelper. Make it not special case renderer/plugin, ↵jam@chromium.org2011-03-151-0/+1
| | | | | | | | and derive from BrowserMessageFilter for easier filtering and replying. Review URL: http://codereview.chromium.org/6695009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78225 0039d316-1c4b-4281-b951-d872f2087c98
* Remove Gears from Chrome.aa@chromium.org2011-03-111-6/+0
| | | | | | | | | | There are probably a few more bits and pieces that can be removed, but I think this is the majority of it. BUG=51934 Review URL: http://codereview.chromium.org/6576020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77888 0039d316-1c4b-4281-b951-d872f2087c98
* Delete the temporary headers left behind in chrome\browser. I've left ↵jam@chromium.org2011-03-011-5/+5
| | | | | | | | | browser_thread.h for another change, since 450 files include it. TBR=avi Review URL: http://codereview.chromium.org/6596060 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76326 0039d316-1c4b-4281-b951-d872f2087c98
* WebUI: Move the remaining files from chrome/browser/webui to ↵tfarina@chromium.org2011-02-271-1/+1
| | | | | | | | | | | chrome/browser/ui/webui. Final Part. BUG=59946 TEST=trybots Review URL: http://codereview.chromium.org/6594035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76191 0039d316-1c4b-4281-b951-d872f2087c98
* Reland "WebUI: Move more files from chrome/browser/dom_ui to ↵tfarina@chromium.org2011-02-191-3/+3
| | | | | | | | | | | | | | | | chrome/browser/webui. Part 5." This reverts commit 1088003ee7026f2b830d2451f79ce772c44a7cea. BUG=59945, 59946 TEST=trybots TBR=avi@chromium.org Original Review: http://codereview.chromium.org/6538053/ Review URL: http://codereview.chromium.org/6532069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75478 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 75453 - WebUI: Move more files from chrome/browser/dom_ui to ↵scherkus@chromium.org2011-02-181-3/+3
| | | | | | | | | | | | | chrome/browser/webui. Part 5. BUG=59945, 59946 TEST=trybots Review URL: http://codereview.chromium.org/6538053 TBR=tfarina@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75456 0039d316-1c4b-4281-b951-d872f2087c98
* WebUI: Move more files from chrome/browser/dom_ui to chrome/browser/webui. ↵tfarina@chromium.org2011-02-181-3/+3
| | | | | | | | | | | Part 5. BUG=59945, 59946 TEST=trybots Review URL: http://codereview.chromium.org/6538053 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75453 0039d316-1c4b-4281-b951-d872f2087c98
* Attempt 3 at: Splits ChromeURLDataManager into 2 chunks:sky@chromium.org2011-02-101-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | . ChromeURLDataManager is no longer a singleton and is always used on the UI thread. ChromeURLDataManager is now profile specific (you get from the profile). . ChromeURLDataManagerBackend handles the URLRequests and the DataSources. ChromeURLDataManagerBackend is created by ChromeURLRequestContext. All DataSources are now profile specific. There were two that wanted to be global, but have been converted. This differs from the last version in that DataSource::SendResponse does nothing if the DataSource is schedule for deletion. This is necessary otherwise SendResponse will up the ref count and by the time the request is processed on the IO thread chances are the DataSource will have been deleted. And if it wasn't deleted, it'll get scheduled for deletion again. <insert comment here about the perils of using delayed deletion> This has always been possible, it just appears to be more likely with my patch. BUG=52022 71868 TEST=none Review URL: http://codereview.chromium.org/6479007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74432 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 74340 - Attempt 2 at: Splits ChromeURLDataManager into 2 chunks:sky@chromium.org2011-02-101-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | . ChromeURLDataManager is no longer a singleton and is always used on the UI thread. ChromeURLDataManager is now profile specific (you get from the profile). . ChromeURLDataManagerBackend handles the URLRequests and the DataSources. ChromeURLDataManagerBackend is created by ChromeURLRequestContext. All DataSources are now profile specific. There were two that wanted to be global, but have been converted. This differs from the version I landed in that there is now a NULL check in ChromeURLDataManager::DeleteDataSources. BUG=52022 71868 TEST=none TBR=willchan, evan, ahendrickson Review URL: http://codereview.chromium.org/6462036 TBR=sky@chromium.org Review URL: http://codereview.chromium.org/6478010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74384 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-07 20:33:35 +0000