Splits ChromeURLDataManager into 2 chunks:...

chrome/browser/dom_ui/sync_internals_ui_unittest.cc

Index: chrome/browser/dom_ui/sync_internals_ui_unittest.cc
===================================================================
--- chrome/browser/dom_ui/sync_internals_ui_unittest.cc	(revision 73990)
+++ chrome/browser/dom_ui/sync_internals_ui_unittest.cc	(working copy)
@@ -110,15 +110,19 @@
     return static_cast<TestSyncInternalsUI*>(test_sync_internals_ui_buf_);
   }
 
+ private:

[willchan no longer on Chromium, 2011/02/08 04:57:52]

Can these just be added to the protected section? Or can the stuff in the
protected section be moved to the private section? Looks like |mock_js_backend_|
needs to be protected, but maybe everything else can go private?

[sky, 2011/02/08 05:35:26]

Moved to protected.

+  // These need to be created before profile sync service.
+  // Needed by |ui_thread_|.
+  MessageLoopForUI ui_loop_;
+  // Needed by |test_tab_contents_|.
+  BrowserThread ui_thread_;
+
+ protected:
   NiceMock<ProfileMock> profile_mock_;
   StrictMock<ProfileSyncServiceMock> profile_sync_service_mock_;
   StrictMock<browser_sync::MockJsFrontend> mock_js_backend_;
 
  private:
-  // Needed by |ui_thread_|.
-  MessageLoopForUI ui_loop_;
-  // Needed by |test_tab_contents_|.
-  BrowserThread ui_thread_;
   TestTabContents test_tab_contents_;
   void* test_sync_internals_ui_buf_;
   bool test_sync_internals_ui_constructor_called_;