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

Unified 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: ios 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 side-by-side diff with in-line comments
Download patch
Index: content/renderer/renderer_main_thread.cc
diff --git a/content/renderer/renderer_main_thread.cc b/content/renderer/renderer_main_thread.cc
new file mode 100644
index 0000000000000000000000000000000000000000..c6704c83c5d38263c7c2c6f53e6154371f21803b
--- /dev/null
+++ b/content/renderer/renderer_main_thread.cc
@@ -0,0 +1,45 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/renderer/renderer_main_thread.h"
+
+#include "content/renderer/render_process.h"
+#include "content/renderer/render_process_impl.h"
+#include "content/renderer/render_thread_impl.h"
+
+namespace content {
+
+RendererMainThread::RendererMainThread(const std::string& channel_id)
+ : Thread("Chrome_InProcRendererThread"), channel_id_(channel_id) {
+}
+
+RendererMainThread::~RendererMainThread() {
+ Stop();
+}
+
+void RendererMainThread::Init() {
+ render_process_.reset(new RenderProcessImpl());
+ new RenderThreadImpl(channel_id_);
+}
+
+void RendererMainThread::CleanUp() {
+ render_process_.reset();
+
+ // It's a little lame to manually set this flag. But the single process
+ // RendererThread will receive the WM_QUIT. We don't need to assert on
+ // this thread, so just force the flag manually.
+ // If we want to avoid this, we could create the InProcRendererThread
+ // directly with _beginthreadex() rather than using the Thread class.
+ // We used to set this flag in the Init function above. However there
+ // other threads like WebThread which are created by this thread
+ // which resets this flag. Please see Thread::StartWithOptions. Setting
+ // this flag to true in Cleanup works around these problems.
+ SetThreadWasQuitProperly(true);
+}
+
+base::Thread* CreateRendererMainThread(const std::string& channel_id) {
+ return new RendererMainThread(channel_id);
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698