Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(227)

Side by Side Diff: content/renderer/renderer_main_thread.cc

Issue 23235002: Set up content in-process main threads via factory (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: don't add targets (were already in !ios block) Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "content/renderer/renderer_main_thread.h"
6
7 #include "content/renderer/render_process.h"
8 #include "content/renderer/render_process_impl.h"
9 #include "content/renderer/render_thread_impl.h"
10
11 namespace content {
12
13 RendererMainThread::RendererMainThread(const std::string& channel_id)
14 : Thread("Chrome_InProcRendererThread"), channel_id_(channel_id) {
15 }
16
17 RendererMainThread::~RendererMainThread() {
18 Stop();
19 }
20
21 void RendererMainThread::Init() {
22 render_process_.reset(new RenderProcessImpl());
23 new RenderThreadImpl(channel_id_);
24 }
25
26 void RendererMainThread::CleanUp() {
27 render_process_.reset();
28
29 // It's a little lame to manually set this flag. But the single process
30 // RendererThread will receive the WM_QUIT. We don't need to assert on
31 // this thread, so just force the flag manually.
32 // If we want to avoid this, we could create the InProcRendererThread
33 // directly with _beginthreadex() rather than using the Thread class.
34 // We used to set this flag in the Init function above. However there
35 // other threads like WebThread which are created by this thread
36 // which resets this flag. Please see Thread::StartWithOptions. Setting
37 // this flag to true in Cleanup works around these problems.
38 SetThreadWasQuitProperly(true);
39 }
40
41 base::Thread* CreateRendererMainThread(const std::string& channel_id) {
42 return new RendererMainThread(channel_id);
43 }
44
45 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698