Splits ChromeURLDataManager into 2 chunks:...

chrome/browser/dom_ui/sync_internals_ui_unittest.cc

 // Copyright (c) 2011 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/dom_ui/sync_internals_ui.h"
 
 #include <cstddef>
 #include <string>
 
 #include "base/message_loop.h"
@@ skipping 92 matching lines @@
       return NULL;
     }
     return GetTestSyncInternalsUIAddress();
   }
 
   // Used for passing into EXPECT_CALL().
   TestSyncInternalsUI* GetTestSyncInternalsUIAddress() {
     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_;
 };
 
 TEST_F(SyncInternalsUITest, HandleJsEvent) {
   ExpectSetupTeardownCalls();
 
   ConstructTestSyncInternalsUI();
 
@@ skipping 76 matching lines @@
 
   ConstructTestSyncInternalsUI();
 
   EXPECT_CALL(*GetTestSyncInternalsUI(),
               ExecuteJavascript(std::wstring(kAboutInfoCall)));
 
   GetTestSyncInternalsUI()->ProcessDOMUIMessage(params);
 }
 
 }  // namespace