Linux: move X operations from the IO to UI2 thread.

chrome/browser/browser_process_impl.cc

 // Copyright (c) 2006-2008 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 "chrome/browser/browser_process_impl.h"
 
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "base/thread.h"
 #include "base/waitable_event.h"
@@ skipping 153 matching lines @@
 
     // Cancel pending requests and prevent new requests.
     resource_dispatcher_host()->Shutdown();
   }
 
   // Shutdown DNS prefetching now to ensure that network stack objects
   // living on the IO thread get destroyed before the IO thread goes away.
   io_thread_->message_loop()->PostTask(FROM_HERE,
       NewRunnableFunction(chrome_browser_net::EnsureDnsPrefetchShutdown));
 
+#if defined(OS_LINUX)
+  // The IO thread must outlive the BACKGROUND_X11 thread.
+  background_x11_thread_.reset();
+#endif
+
   // Need to stop io_thread_ before resource_dispatcher_host_, since
   // io_thread_ may still deref ResourceDispatcherHost and handle resource
   // request before going away.
   io_thread_.reset();
 
   // Clean up state that lives on the file_thread_ before it goes away.
   if (resource_dispatcher_host_.get()) {
     resource_dispatcher_host()->download_file_manager()->Shutdown();
     resource_dispatcher_host()->save_file_manager()->Shutdown();
   }
@@ skipping 93 matching lines @@
 void BrowserProcessImpl::CreateIOThread() {
   DCHECK(!created_io_thread_ && io_thread_.get() == NULL);
   created_io_thread_ = true;
 
   // Prior to starting the io thread, we create the plugin service as
   // it is predominantly used from the io thread, but must be created
   // on the main thread. The service ctor is inexpensive and does not
   // invoke the io_thread() accessor.
   PluginService::GetInstance();
 
+#if defined(OS_LINUX)
+  // The lifetime of the BACKGROUND_X11 thread is a subset of the IO thread so
+  // we start it now.
+  scoped_ptr<base::Thread> background_x11_thread(
+      new BrowserProcessSubThread(ChromeThread::BACKGROUND_X11));
+  if (!background_x11_thread->Start())
+    return;
+  background_x11_thread_.swap(background_x11_thread);
+#endif
+
   scoped_ptr<base::Thread> thread(
       new BrowserProcessSubThread(ChromeThread::IO));
   base::Thread::Options options;
   options.message_loop_type = MessageLoop::TYPE_IO;
   if (!thread->StartWithOptions(options))
     return;
   io_thread_.swap(thread);
 }
 
 void BrowserProcessImpl::CreateFileThread() {
@@ skipping 78 matching lines @@
 #else
   // TODO(port): remove this completely, it has no business being here.
 #endif
 }
 
 void BrowserProcessImpl::CreateGoogleURLTracker() {
   DCHECK(google_url_tracker_.get() == NULL);
   scoped_ptr<GoogleURLTracker> google_url_tracker(new GoogleURLTracker);
   google_url_tracker_.swap(google_url_tracker);
 }