Run RemoveArcData after a user has opted out

chrome/browser/chromeos/chrome_browser_main_chromeos.h

Index: chrome/browser/chromeos/chrome_browser_main_chromeos.h
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.h b/chrome/browser/chromeos/chrome_browser_main_chromeos.h
index eb33f493433b56dc84195f846a2b3902441676b2..efaa01745b559d6220c1df8e1130beca9ea76b02 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.h
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.h
@@ -19,6 +19,7 @@ class SessionManager;
 
 namespace arc {
 class ArcServiceLauncher;
+class ArcDataManager;

[Junichi Uekawa, 2016/05/12 08:44:15]

sort?

[dspaid, 2016/05/13 01:19:25]

Done.

 }
 
 namespace chromeos {
@@ -84,6 +85,7 @@ class ChromeBrowserMainPartsChromeos : public ChromeBrowserMainPartsLinux {
   scoped_refptr<chromeos::ExternalMetrics> external_metrics_;
 
   std::unique_ptr<arc::ArcServiceLauncher> arc_service_launcher_;
+  std::unique_ptr<arc::ArcDataManager> arc_data_manager_;

[Junichi Uekawa, 2016/05/12 08:44:15]

naive question: why does this not live inside ArcServiceLauncher?

[dspaid, 2016/05/13 01:19:25]

Service launcher right now only seems to be responsible for starting up and
shutting down the arc bridge.  I felt that adding in data management would be
out of scope.

 
   DISALLOW_COPY_AND_ASSIGN(ChromeBrowserMainPartsChromeos);
 };