0
Commit Graph

21 Commits

Author SHA1 Message Date
msarda@chromium.org
ef7557ba6d Build browser_process_sub_thread.cc on iOS.
Adds browser_process_sub_thread.cc to the build on iOS and ifdefs out
BrowserChildProcessHostImpl::TerminateAll() as Chrome on iOS is
single-process.

Review URL: https://chromiumcodereview.appspot.com/11066054

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@161091 0039d316-1c4b-4281-b951-d872f2087c98
2012-10-10 12:28:22 +00:00
pkasting@chromium.org
451fd90d92 Use ScopedCOMInitializer in more places. While this doesn't always simplify code, it does mean we do consistent logging and error-checking at all these sites.
BUG=none
TEST=none
Review URL: https://codereview.chromium.org/11050009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159908 0039d316-1c4b-4281-b951-d872f2087c98
2012-10-03 17:14:48 +00:00
pkasting@chromium.org
435756b898 Just because we want to destroy a pointer somewhere other than a destructor doesn't mean we can't use a scoped_ptr for it. This saves an explicit initializer and makes the code parallel to the ScopedCOMInitializers I'll be adding next.
BUG=none
TEST=none
Review URL: https://codereview.chromium.org/10993056

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159556 0039d316-1c4b-4281-b951-d872f2087c98
2012-10-01 21:19:36 +00:00
alecflett@chromium.org
5c085eda07 Remove all the indexeddb-related utility process code
BUG=129471


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152873

Review URL: https://chromiumcodereview.appspot.com/10834350

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153336 0039d316-1c4b-4281-b951-d872f2087c98
2012-08-24 22:57:29 +00:00
ricow@chromium.org
f6c57d4e73 Revert 152873 - Remove all the indexeddb-related utility process code
BUG=129471


Review URL: https://chromiumcodereview.appspot.com/10834350

This seems to be causing test failure on Linux Tests (dbg)(1), first hit here:
http://build.chromium.org/p/chromium/builders/Linux%20Tests%20%28dbg%29%281%29/builds/20655/steps/browser_tests/logs/stdio

TBR=alecflett@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10873028

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152980 0039d316-1c4b-4281-b951-d872f2087c98
2012-08-23 10:47:07 +00:00
alecflett@chromium.org
8374250ef7 Remove all the indexeddb-related utility process code
BUG=129471


Review URL: https://chromiumcodereview.appspot.com/10834350

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152873 0039d316-1c4b-4281-b951-d872f2087c98
2012-08-22 23:48:57 +00:00
akalin@chromium.org
3dc1bc4dd6 Move content::URLFetcher static functions to net::URLFetcher
Remove content::URLFetcher.

Update all references.

Add explicit dependency on content_common from browser, since template_url_fetcher.cc calls AssociateURLFetcherWithRenderView
(there are probably other direct uses of content_common).

BUG=118220
TEST=
TBR=satorux@chromium.org,mnissler@chromium.org,mal@chromium.org,jhawkins@chromium.org,sky@chromium.org,rlp@chromium.org

Review URL: https://chromiumcodereview.appspot.com/10554008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142926 0039d316-1c4b-4281-b951-d872f2087c98
2012-06-19 08:20:53 +00:00
willchan@chromium.org
ef2bf421b0 Stop refcounting URLRequestContext.
While doing so, fix a few issues with the code like ordering of URLRequestContext to ensure correct destruction order. Also fix const correctness in some places.

BUG=58859
TEST=none
TBR=willchan

Review URL: https://chromiumcodereview.appspot.com/10299002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136497 0039d316-1c4b-4281-b951-d872f2087c98
2012-05-11 03:27:09 +00:00
jam@chromium.org
3a7b66d5ef Disallow UI/IO thread blocking on any other thread.
By design, there's no ScopedAllowWait that is reachable by all code. From experience with ScopedAllowIO, it will be abused. So instead the existing callers (which should all be fixed other than two) are friends with ThreadRestrictions.
Review URL: https://chromiumcodereview.appspot.com/10151009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134114 0039d316-1c4b-4281-b951-d872f2087c98
2012-04-26 16:34:16 +00:00
jam@chromium.org
4c01d49992 Rename BrowserChildProcessHost implementation class to BrowserChildProcessHostImpl.
BUG=98716
Review URL: https://chromiumcodereview.appspot.com/9117006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118758 0039d316-1c4b-4281-b951-d872f2087c98
2012-01-23 23:33:01 +00:00
jam@chromium.org
4967f79293 Add a Content API around BrowserChildProcessHost, similar to what was done with ChildProcessHost. Now classes like PluginProcessHost don't derive from it, but instead use composition.
I've also moved the iterator class into its own file in the public directory. Since classes don't derive from BrowserChildProcessHost and so can't static_cast from it, I added a template helper that does this.

BUG=98716
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=118415
Review URL: https://chromiumcodereview.appspot.com/9150017

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118516 0039d316-1c4b-4281-b951-d872f2087c98
2012-01-20 22:14:40 +00:00
jam@chromium.org
f177ffd84b Revert 118415 - Add a Content API around BrowserChildProcessHost, similar to what was done with ChildProcessHost. Now classes like PluginProcessHost don't derive from it, but instead use composition.
I've also moved the iterator class into its own file in the public directory. Since classes don't derive from BrowserChildProcessHost and so can't static_cast from it, I added a template helper that does this.

BUG=98716
Review URL: https://chromiumcodereview.appspot.com/9150017

TBR=jam@chromium.org

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118420 0039d316-1c4b-4281-b951-d872f2087c98
2012-01-20 07:51:54 +00:00
jam@chromium.org
3b0032a7ab Add a Content API around BrowserChildProcessHost, similar to what was done with ChildProcessHost. Now classes like PluginProcessHost don't derive from it, but instead use composition.
I've also moved the iterator class into its own file in the public directory. Since classes don't derive from BrowserChildProcessHost and so can't static_cast from it, I added a template helper that does this.

BUG=98716
Review URL: https://chromiumcodereview.appspot.com/9150017

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118415 0039d316-1c4b-4281-b951-d872f2087c98
2012-01-20 06:29:23 +00:00
jam@chromium.org
af6699372a Move cleanup that happens on IO thread destruction to content, so that not every embedder has to know to do it. This also allows us to hide some of these functions from embedders.
BUG=98716
Review URL: https://chromiumcodereview.appspot.com/9221008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117934 0039d316-1c4b-4281-b951-d872f2087c98
2012-01-17 19:26:58 +00:00
jhawkins@chromium.org
5abe630d97 Coverity: Initialize member variables.
CID=100356,100360,100368,100373,100374,100562,101529,101530,101536,101541,
    101542,101595,101630,101685,101687,101689,101776,101812,101813,101854,
    101861,101862,101871,101883,101886,101887,102034
BUG=none
TEST=none
R=binji

Review URL: http://codereview.chromium.org/8965054

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115225 0039d316-1c4b-4281-b951-d872f2087c98
2011-12-20 23:44:32 +00:00
joi@chromium.org
2e5b60a280 Have content/ create and destroy its own threads. (Re-land)
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
2011-11-28 15:56:41 +00:00
joi@chromium.org
14f79fec5f Revert 111695 - Have content/ create and destroy its own threads.
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
2011-11-28 14:00:36 +00:00
joi@chromium.org
042666f262 Have content/ create and destroy its own threads.
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
2011-11-28 12:51:39 +00:00
joi@chromium.org
c38831a108 Split BrowserThread into public API and private implementation, step 1.
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
2011-10-28 12:44:49 +00:00
jam@chromium.org
ad50def5e1 Make NotificationService an interface in the content namespace, and switch callers to use it. Move the implementation to content/browser. Stop creating it in all child processes since it's only used in the browser.
BUG=98716
Review URL: http://codereview.chromium.org/8342048

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106403 0039d316-1c4b-4281-b951-d872f2087c98
2011-10-19 23:17:07 +00:00
jam@chromium.org
b0f146ff51 Create a very simple TabContentsView (and not fully implemented yet) and add more supporting code to be able to load a page. Right now it's not rendering, but I suspect it's something small, and the patch has gotten large so I figure it's time to send it for review.
BUG=90445
Review URL: http://codereview.chromium.org/7906008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101395 0039d316-1c4b-4281-b951-d872f2087c98
2011-09-15 22:14:25 +00:00