Removing FRE UI for unicorn accounts.

chrome/browser/supervised_user/child_accounts/child_account_status_observer.h

Index: chrome/browser/supervised_user/child_accounts/child_account_status_observer.h
diff --git a/chrome/browser/supervised_user/child_accounts/child_account_service_android.h b/chrome/browser/supervised_user/child_accounts/child_account_status_observer.h
similarity index 50%
copy from chrome/browser/supervised_user/child_accounts/child_account_service_android.h
copy to chrome/browser/supervised_user/child_accounts/child_account_status_observer.h
index f9b09600cdf72d05bb0ef1edf4d2a44abb5ef0e3..985743fbb3ee4ca6a499dc7c391863eecb484f38 100644
--- a/chrome/browser/supervised_user/child_accounts/child_account_service_android.h
+++ b/chrome/browser/supervised_user/child_accounts/child_account_status_observer.h
@@ -1,13 +1,14 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
+// Copyright 2015 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.
 
-#ifndef CHROME_BROWSER_SUPERVISED_USER_CHILD_ACCOUNTS_CHILD_ACCOUNT_SERVICE_ANDROID_H_
-#define CHROME_BROWSER_SUPERVISED_USER_CHILD_ACCOUNTS_CHILD_ACCOUNT_SERVICE_ANDROID_H_
+#ifndef CHROME_BROWSER_SUPERVISED_USER_CHILD_ACCOUNTS_CHILD_ACCOUNT_STATUS_OBSERVER_H_
+#define CHROME_BROWSER_SUPERVISED_USER_CHILD_ACCOUNTS_CHILD_ACCOUNT_STATUS_OBSERVER_H_
 
-#include <jni.h>
+class ChildAccountStatusObserver {
+  public:

[Bernhard Bauer, 2015/03/10 09:55:12]

"public:" is indented one space, then the method declarations are indented two
spaces (i.e. one more than the previous line).

[merkulova, 2015/03/10 11:53:57]

Acknowledged.

+    // Observes changes from Unknown to other states.
+    virtual void OnChildAccountStatusChanged(bool is_child) {}

[Bernhard Bauer, 2015/03/10 09:55:12]

If this is only called when transitioning from unknown state to known state,
maybe reflect that in the name (right now it looks like this method is called on
any state change).

Also, are you planning to add more methods to this interface? If not, consider
using a base::Callback instead.

[Marc Treib, 2015/03/10 10:43:34]

I think it should be called on any state change, otherwise there's really no
need for an Observer interface.
Hm, I guess that's the alternative: A callback that's called exactly once, when
we get the child state for the first time.

[merkulova, 2015/03/10 11:53:57]

Removed the class completely.

+};
 
-// Register native methods
-bool RegisterChildAccountService(JNIEnv* env);
-
-#endif  // CHROME_BROWSER_SUPERVISED_USER_CHILD_ACCOUNTS_CHILD_ACCOUNT_SERVICE_ANDROID_H_
+#endif  // CHROME_BROWSER_SUPERVISED_USER_CHILD_ACCOUNTS_CHILD_ACCOUNT_STATUS_OBSERVER_H_