summaryrefslogtreecommitdiffstats
path: root/chrome/browser/service
Commit message (Collapse)AuthorAgeFilesLines
* Memory leak from cloud printing proxy UI.scottbyer@google.com2010-10-251-9/+14
| | | | | | | | | | | | | First, stop calling some service things from the tabbed options UI support code when cloud printing isn't enabled. Second, have better ownership of the leaked task. BUG=60008, 60313 TEST=OptionsUITest.FLAKY_CommandAgainGoesBackToOptionsTab under valgrind. Review URL: http://codereview.chromium.org/4033003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63791 0039d316-1c4b-4281-b951-d872f2087c98
* Thread IO safety: annotate file_util, and block IO thread from doing IOevan@chromium.org2010-10-231-1/+9
| | | | | | | | | | | | | | | - Mark functions in file_util_posix as requiring permission to perform disk actions. - Mark the IO thread as disallowed from performing disk actions. - Temporarily work around the protections in places where we currently have bugs. BUG=59847,59849,60207,60211,60394 TEST=no dchecks in debug builds Review URL: http://codereview.chromium.org/3872002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63636 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 63600 - Thread IO safety: annotate file_util, and block IO thread ↵isherman@chromium.org2010-10-231-9/+1
| | | | | | | | | | | | | | | | | | | | | | from doing IO Reverting because this breaks the ChromiumOS (dbg) builder. - Mark functions in file_util_posix as requiring permission to perform disk actions. - Mark the IO thread as disallowed from performing disk actions. - Temporarily work around the protections in places where we currently have bugs. BUG=59847,59849,60207,60211 TEST=no dchecks in debug builds Review URL: http://codereview.chromium.org/3872002 TBR=evan@chromium.org Review URL: http://codereview.chromium.org/3983005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63630 0039d316-1c4b-4281-b951-d872f2087c98
* Thread IO safety: annotate file_util, and block IO thread from doing IOevan@chromium.org2010-10-231-1/+9
| | | | | | | | | | | | | | | - Mark functions in file_util_posix as requiring permission to perform disk actions. - Mark the IO thread as disallowed from performing disk actions. - Temporarily work around the protections in places where we currently have bugs. BUG=59847,59849,60207,60211 TEST=no dchecks in debug builds Review URL: http://codereview.chromium.org/3872002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63600 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Thread IO safety: annotate file_util, and block IO thread from doing IO"evan@chromium.org2010-10-221-9/+1
| | | | | | This reverts commit r63580, build breakage. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63586 0039d316-1c4b-4281-b951-d872f2087c98
* Thread IO safety: annotate file_util, and block IO thread from doing IOevan@chromium.org2010-10-221-1/+9
| | | | | | | | | | | | | | | - Mark functions in file_util_posix as requiring permission to perform disk actions. - Mark the IO thread as disallowed from performing disk actions. - Temporarily work around the protections in places where we currently have bugs. BUG=59847,59849,60207,60211 TEST=no dchecks in debug builds Review URL: http://codereview.chromium.org/3872002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63580 0039d316-1c4b-4281-b951-d872f2087c98
* Convert LOG(INFO) to VLOG(1) - misc. chrome/browser/*/*.pkasting@chromium.org2010-10-211-1/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/4009001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63420 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed ServiceProcessControl::Launch to remember the Task supplied to it in a ↵sanjeevr@chromium.org2010-10-183-45/+144
| | | | | | | | | | | list so that multiple successive Launch calls will work. Also added separate success and failure tasks to Launch. Modified the browser test to test this new functionality. Modified callers. BUG=58802 TEST=Browser tests, Cloud Print Proxy UI. Review URL: http://codereview.chromium.org/3851001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62978 0039d316-1c4b-4281-b951-d872f2087c98
* Used process_util methods to wait for the service process to die in the ↵sanjeevr@chromium.org2010-10-121-33/+23
| | | | | | | | | | | ServiceProcessControlBrowserTest. BUG=None TEST=browser_tests. Review URL: http://codereview.chromium.org/3653005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62339 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash on launch when remoting lab is turned on on Windows.scottbyer@chromium.org2010-10-121-0/+8
| | | | | | | | | BUG=58888 TEST=turn on remoting in labs, quit chrome, make sure service process isn't running, launch chrome. Shouldn't crash. Review URL: http://codereview.chromium.org/3704003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62294 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ChromeThread to BrowserThread Part21:tfarina@chromium.org2010-10-122-2/+2
| | | | | | | | | | | - Include browser_thread.h instead of chrome_thread.h in more 100 files. BUG=56926 TEST=trybots Review URL: http://codereview.chromium.org/3691006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62286 0039d316-1c4b-4281-b951-d872f2087c98
* Added the service process pid to the shared memory data. This will be used ↵sanjeevr@chromium.org2010-10-122-3/+12
| | | | | | | | | | | for tests. Also cleaned up some of the method names in service_process_utils.cc to make them clearer. BUG=None. TEST=Browser tests. Review URL: http://codereview.chromium.org/3727002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62223 0039d316-1c4b-4281-b951-d872f2087c98
* The IPC channel names and event names used by the service process now append ↵sanjeevr@chromium.org2010-10-092-16/+49
| | | | | | | | | | | | | the version string so that a browser with the same version only can talk to it. In addition, the service process writes its version into shared memory (the name of this shared mem does not use a version) and also listens on a shutdown event. This can be used by a newer service process to shutdown an older version when it starts. BUG=None TEST=Unittests, with an existing service process running, let chrome autoupdate and then run the service process again. It should kill the old one. Review URL: http://codereview.chromium.org/3603016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62066 0039d316-1c4b-4281-b951-d872f2087c98
* All communication with the cloud print proxy service from the browser now ↵sanjeevr@chromium.org2010-10-073-4/+70
| | | | | | | | | | | happens in the CloudPrintProxyService class. Added code to start the service process if the cloud print proxy was enabled. Also, when detect an auto-update, we send an IPC to the service process. The service process then shuts down when the browser disconnects. BUG=None TEST=Unit-tests, test Cloud Print proxy UI, test that when an update is available, the service process shuts down when the browser does. TBR=phajdan.jr Review URL: http://codereview.chromium.org/3617008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61871 0039d316-1c4b-4281-b951-d872f2087c98
* Rename ChromeThread to BrowserThread Part4:tfarina@chromium.org2010-10-072-14/+14
| | | | | | | | | | | - Rename entries under safe_browsing, search_engines, service and speech. BUG=56926 TEST=trybots Review URL: http://codereview.chromium.org/3552013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61763 0039d316-1c4b-4281-b951-d872f2087c98
* Removed the ServiceProcessType enum because a single service process should ↵sanjeevr@chromium.org2010-10-055-22/+10
| | | | | | | | | | | host all types of services. Also implemeneted a rudimentary singleton mechanism for the service process on Windows. BUG=None TEST=Test cloud print proxy and remoting UI. Review URL: http://codereview.chromium.org/3521012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61549 0039d316-1c4b-4281-b951-d872f2087c98
* Cloud print proxy management UI.scottbyer@chromium.org2010-09-272-6/+46
| | | | | | | | | | | Based in the Under-the-Hood section of options, this UI allows mere mortals to turn on and off the cloud print proxy. Currently working on Windows only, behind a flag. BUG=none TEST=Open up the options, turn CPP on and off. Review URL: http://codereview.chromium.org/3450021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60684 0039d316-1c4b-4281-b951-d872f2087c98
* Enable ServiceProcessControlTest.LaunchTwicehclam@chromium.org2010-09-031-3/+5
| | | | | | | | | | | Quit the message loop by posting a quit task instead of calling quit directly. BUG= TEST= Review URL: http://codereview.chromium.org/3342012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58510 0039d316-1c4b-4281-b951-d872f2087c98
* Build fix for browser_testshclam@chromium.org2010-09-021-5/+1
| | | | | | | | | | | | | ServiceProcesControlTest.LaunchTwice is timing out. Disable the test for now. TBR=tc@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/3349005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58291 0039d316-1c4b-4281-b951-d872f2087c98
* Don't launch service process if it's already runninghclam@chromium.org2010-09-013-25/+42
| | | | | | | | | | | | | | BUG=52891 TEST=browser_tests --gtest_filter=ServiceProcess* Removed ServiceProcessControl::Connect() and just have a Launch() method. The Launch() method will connect to the service process if the it has already been launched but not connected. The method will simply return if the process is already been launched connected. Review URL: http://codereview.chromium.org/3306001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58265 0039d316-1c4b-4281-b951-d872f2087c98
* Registered the service process to AutoRun (for Windows) if any service ↵sanjeevr@chromium.org2010-09-011-2/+1
| | | | | | | | | | | (CloudPrint or Remoting) is enabled. When all services are disabled, we delete the AutoRun key. Also remembered the enabled state of the Cloud Print Proxy in the service prefs. BUG=None TEST=After enabling cloud print or chromoting on Windows, the service process should auto start on user login. Review URL: http://codereview.chromium.org/3257011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58246 0039d316-1c4b-4281-b951-d872f2087c98
* Don't shutdown service process when browser shuts downhclam@chromium.org2010-09-011-6/+2
| | | | | | | | | | | | Keep the service process running when browser shuts down. BUG=52891 TBR=sanjeevr TEST=none Review URL: http://codereview.chromium.org/3307001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58116 0039d316-1c4b-4281-b951-d872f2087c98
* Use named events instead of lock files for service processhclam@chromium.org2010-08-311-6/+0
| | | | | | | | | | | Use named events to signal that a service process is running. BUG=52891 TEST=browser_tests --gtest_filter=ServiceProcess* Review URL: http://codereview.chromium.org/3268003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58041 0039d316-1c4b-4281-b951-d872f2087c98
* Resubmit r56600 - Start/stop service process when browser starts and stophclam@chromium.org2010-08-232-3/+22
| | | | | | | | | | | | | | | | | Save the information that the setup of remoting has completed. After setup has been completed we start and stop service process when --enable-remoting presents when browser starts. Also save the information in the service process that host is registered and ready to be used. Again start chromoting host automatically once the host registration is done. BUG=50244, 50242, 50243, 50249 Review URL: http://codereview.chromium.org/3153029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57120 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 6)thestig@chromium.org2010-08-191-0/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3093013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56641 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 56600 - ServiceProcessControlBrowserTest.LaunchAndIPC crashes.rvargas@google.com2010-08-182-16/+3
| | | | | | | | | | | | | | | | | | | | Start/stop service process when browser starts and stop Save the information that the setup of remoting has completed. After setup has been completed we start and stop service process when --enable-remoting presents when browser starts. Also save the information in the service process that host is registered and ready to be used. Again start chromoting host automatically once the host registration is done. Review URL: http://codereview.chromium.org/3185015 TBR=hclam@chromium.org Review URL: http://codereview.chromium.org/3107024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56610 0039d316-1c4b-4281-b951-d872f2087c98
* Start/stop service process when browser starts and stophclam@chromium.org2010-08-182-3/+16
| | | | | | | | | | | | | | | Save the information that the setup of remoting has completed. After setup has been completed we start and stop service process when --enable-remoting presents when browser starts. Also save the information in the service process that host is registered and ready to be used. Again start chromoting host automatically once the host registration is done. Review URL: http://codereview.chromium.org/3185015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56600 0039d316-1c4b-4281-b951-d872f2087c98
* Gaia authentication for access remoting directory and talkhclam@chromium.org2010-08-172-0/+15
| | | | | | | | | | | | | | | | We are now able to access remoting directory token and talk token through the chromoting setup dialog. These information are sent to the service process for enabling the chromoting host and perform the host registration. All the plumbing for starting the service process, enabling the chromoting host through through IPC commands are in place but host registration is stubbed out and will be completed in next patch. Review URL: http://codereview.chromium.org/3176014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56278 0039d316-1c4b-4281-b951-d872f2087c98
* ServiceProcessControl to launch a service process and communicate through IPC.hclam@chromium.org2010-08-125-0/+488
Added two class for use with service process: ServiceProcessControl Used by the browser to launch and connect to the service process, also used to receive messages from the service process. ServiceProcessControlManager A singleton to manage multiple ServicProcessControl. BUG=50244 TEST=browser_tests --gtest_filter=ServiceProcess* Review URL: http://codereview.chromium.org/3032061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55826 0039d316-1c4b-4281-b951-d872f2087c98