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

Unified Diff: chrome/browser/ui/tabs/tab_strip_model_unittest.cc

Issue 14197014: Add TestBrowserThreadBundle into RenderViewHostTestHarness. Kill some unnecessary real threads. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merged ToT Created 7 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: chrome/browser/ui/tabs/tab_strip_model_unittest.cc
diff --git a/chrome/browser/ui/tabs/tab_strip_model_unittest.cc b/chrome/browser/ui/tabs/tab_strip_model_unittest.cc
index d6befc4d952b7396e445de1123d1c117d246ee75..de42f8720d95684e524ec59c4d83fce01155c0e9 100644
--- a/chrome/browser/ui/tabs/tab_strip_model_unittest.cc
+++ b/chrome/browser/ui/tabs/tab_strip_model_unittest.cc
@@ -36,10 +36,8 @@
#include "content/public/browser/notification_source.h"
#include "content/public/browser/render_process_host.h"
#include "content/public/browser/web_contents.h"
-#include "content/public/test/test_browser_thread.h"
#include "testing/gtest/include/gtest/gtest.h"
-using content::BrowserThread;
using content::SiteInstance;
using content::WebContents;
using extensions::Extension;
@@ -117,9 +115,6 @@ class TabStripModelTestIDUserData : public base::SupportsUserData::Data {
class TabStripModelTest : public ChromeRenderViewHostTestHarness {
public:
- TabStripModelTest() : browser_thread_(BrowserThread::UI, &message_loop_) {
- }
-
WebContents* CreateWebContents() {
return WebContents::Create(WebContents::CreateParams(profile()));
}
@@ -206,9 +201,6 @@ class TabStripModelTest : public ChromeRenderViewHostTestHarness {
selection_model.set_active(selection_model.selected_indices()[0]);
model->SetSelectionFromModel(selection_model);
}
-
- private:
- content::TestBrowserThread browser_thread_;
};
class MockTabStripModelObserver : public TabStripModelObserver {

Powered by Google App Engine
This is Rietveld 408576698