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

Unified Diff: content/browser/browser_main_loop.cc

Issue 1504033007: Move Indexed DB from dedicated thread to task scheduler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@timer
Patch Set: Update to TaskScheduler, modern sequence goodness Created 3 years, 6 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/browser/browser_main_loop.cc
diff --git a/content/browser/browser_main_loop.cc b/content/browser/browser_main_loop.cc
index 77b62d4ae03a6f8c9b98aadc66b0c68e9ad43f77..7c833b0412b8cc807a9219960ca7558fbd7fb197 100644
--- a/content/browser/browser_main_loop.cc
+++ b/content/browser/browser_main_loop.cc
@@ -363,12 +363,6 @@ NOINLINE void ResetThread_IO(std::unique_ptr<BrowserProcessSubThread> thread) {
thread.reset();
}
-NOINLINE void ResetThread_IndexedDb(std::unique_ptr<base::Thread> thread) {
- volatile int inhibit_comdat = __LINE__;
- ALLOW_UNUSED_LOCAL(inhibit_comdat);
- thread.reset();
-}
-
MSVC_POP_WARNING()
MSVC_ENABLE_OPTIMIZE();
@@ -1309,10 +1303,6 @@ void BrowserMainLoop::ShutdownThreadsAndCleanUp() {
break;
}
}
- {
- TRACE_EVENT0("shutdown", "BrowserMainLoop::Subsystem:IndexedDBThread");
- ResetThread_IndexedDb(std::move(indexed_db_thread_));
- }
// Close the blocking I/O pool after the other threads. Other threads such
// as the I/O thread may need to schedule work like closing files or
@@ -1397,9 +1387,6 @@ int BrowserMainLoop::BrowserThreadsStarted() {
}
#endif
- indexed_db_thread_.reset(new base::Thread("IndexedDB"));
- indexed_db_thread_->Start();
-
HistogramSynchronizer::GetInstance();
#if defined(OS_ANDROID) || defined(OS_CHROMEOS)
// Up the priority of the UI thread.

Powered by Google App Engine
This is Rietveld 408576698