Refactor BrowserProcessPlatformPart.

chrome/browser/browser_process_platform_part_base.h

Index: chrome/browser/browser_process_platform_part_base.h
diff --git a/chrome/browser/browser_process_platform_part.h b/chrome/browser/browser_process_platform_part_base.h
similarity index 51%
copy from chrome/browser/browser_process_platform_part.h
copy to chrome/browser/browser_process_platform_part_base.h
index 01000a0110ca0afd480a7596bd11d1c19d2054bc..f63a441654120278230a9ee35c0888290433537a 100644
--- a/chrome/browser/browser_process_platform_part.h
+++ b/chrome/browser/browser_process_platform_part_base.h
@@ -2,17 +2,19 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_BROWSER_PROCESS_PLATFORM_PART_H_
-#define CHROME_BROWSER_BROWSER_PROCESS_PLATFORM_PART_H_
+#ifndef CHROME_BROWSER_BROWSER_PROCESS_PLATFORM_PART_BASE_H_
+#define CHROME_BROWSER_BROWSER_PROCESS_PLATFORM_PART_BASE_H_
 
 #include "base/basictypes.h"
 
 class CommandLine;
 
-class BrowserProcessPlatformPart {
+// A base class for platform-specific BrowserProcessPlatformPart
+// implementations. This class itself should never be used verbatim.

[sky, 2013/05/17 15:42:33]

This last sentence is wrong since this is used on platforms that don't need a
platform specific part, right?

[gab, 2013/05/17 19:47:00]

What I meant is it can't be used "verbatim"; i.e. when you use it through the
typedef you use it as BrowserProcessPlatformPart, not verbatim as
BrowserProcessPlatformPartBase so that's fine.

I was thinking we could also add a presubmit check to make sure no one ever does
"new BrowserProcessPlatformPartBase()", but that might be overkill..?

+class BrowserProcessPlatformPartBase {
  public:
-  BrowserProcessPlatformPart();
-  virtual ~BrowserProcessPlatformPart();
+  BrowserProcessPlatformPartBase();
+  virtual ~BrowserProcessPlatformPartBase();
 
   // Called after creating the process singleton or when another chrome
   // rendez-vous with this one.
@@ -23,7 +25,7 @@ class BrowserProcessPlatformPart {
   virtual void StartTearDown();
 
  private:
-  DISALLOW_COPY_AND_ASSIGN(BrowserProcessPlatformPart);
+  DISALLOW_COPY_AND_ASSIGN(BrowserProcessPlatformPartBase);
 };
 
-#endif  // CHROME_BROWSER_BROWSER_PROCESS_PLATFORM_PART_H_
+#endif  // CHROME_BROWSER_BROWSER_PROCESS_PLATFORM_PART_BASE_H_