Run RemoveArcData after a user has opted out

components/arc/arc_bridge_service.h

Index: components/arc/arc_bridge_service.h
diff --git a/components/arc/arc_bridge_service.h b/components/arc/arc_bridge_service.h
index 9fbc6b6183feb6bae94ace2782178213e9775560..bd254797c6c2e295dc1e52d94593bb43a39d4a5e 100644
--- a/components/arc/arc_bridge_service.h
+++ b/components/arc/arc_bridge_service.h
@@ -68,6 +68,8 @@ class ArcBridgeService : public mojom::ArcBridgeHost {
    public:
     // Called whenever the state of the bridge has changed.
     virtual void OnStateChanged(State state) {}
+    virtual void OnBridgeReady() {}

[Daniel Erat, 2016/05/20 17:41:05]

adding these seems strange. why can't observers just use OnStateChanged() to get
the same information?

[dspaid, 2016/05/23 01:22:36]

Luis commented that OnStateChanged was originally intended for testing and
requested that I add these new methods instead. 

[Daniel Erat, 2016/05/24 14:51:22]

luis, can you add some more details here? is the issue that the only states that
non-test consumers should care about are STOPPED and READY? if so, could you
also rename OnStateChanged to OnStateChangedForTest (if you weren't planning to
do so already)?

[Luis Héctor Chávez, 2016/05/24 16:18:28]

changing to OnStateChangedForTest SGTM.

[dspaid, 2016/05/25 00:06:48]

Luis, is this something you'll do in the cleanup CL you mentioned earlier, or do
you want me to put together another CL for that?  I'd rather not increase the
scope of this CL.

[Luis Héctor Chávez, 2016/05/25 00:11:01]

it's fine to defer for later, just add a TODO for me.

[dspaid, 2016/05/25 00:13:27]

Done.

+    virtual void OnBridgeStopped() {}
 
     // Called whenever ARC's availability has changed for this system.
     virtual void OnAvailableChanged(bool available) {}