arc: ARC loading progress should not be shown when started from OOBE.

chrome/browser/chromeos/arc/optin/arc_terms_of_service_oobe_negotiator.h

Index: chrome/browser/chromeos/arc/optin/arc_terms_of_service_oobe_negotiator.h
diff --git a/chrome/browser/chromeos/arc/optin/arc_terms_of_service_oobe_negotiator.h b/chrome/browser/chromeos/arc/optin/arc_terms_of_service_oobe_negotiator.h
new file mode 100644
index 0000000000000000000000000000000000000000..a1028e7b3feb12b09dfcc9aa58c4568ed99e5f8e
--- /dev/null
+++ b/chrome/browser/chromeos/arc/optin/arc_terms_of_service_oobe_negotiator.h
@@ -0,0 +1,43 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.

[hidehiko, 2016/12/16 05:31:51]

Please add unittest.

[khmel, 2016/12/16 18:37:55]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_CHROMEOS_ARC_OPTIN_ARC_TERMS_OF_SERVICE_OOBE_NEGOTIATOR_H_
+#define CHROME_BROWSER_CHROMEOS_ARC_OPTIN_ARC_TERMS_OF_SERVICE_OOBE_NEGOTIATOR_H_
+
+#include "base/macros.h"
+#include "chrome/browser/chromeos/arc/optin/arc_terms_of_service_negotiator.h"
+#include "chrome/browser/chromeos/login/screens/arc_terms_of_service_screen_actor_observer.h"
+
+namespace chromeos {
+class ArcTermsOfServiceScreenActor;

[hidehiko, 2016/12/16 05:31:51]

Please remove. (Should be a part of ArcTermsOfServiceScreenActorObserver
interface.)

[khmel, 2016/12/16 18:37:55]

Done.

+}
+
+namespace arc {
+
+// Handles the Terms-of-service agreement action for Arc managed user via OOBE
+// OptIn UI.
+class ArcTermsOfServiceOOBENegotiator

[hidehiko, 2016/12/16 05:31:51]

This is no longer negotiator for all OOBE case.
Maybe ArcTermsOfServiceInitialOobeNegotiatorForManagedUser etc.? (hmm...
long...).

[khmel, 2016/12/16 18:37:55]

Done.

+    : public ArcTermsOfServiceNegotiator,
+      public chromeos::ArcTermsOfServiceScreenActorObserver {
+ public:
+  ArcTermsOfServiceOOBENegotiator();
+  ~ArcTermsOfServiceOOBENegotiator() override;
+
+  // ArcTermsOfServiceNegotiator:
+  void StartNegotiation(const NegotiationCallback& callback) override;
+
+  // chromeos::ArcTermsOfServiceScreenActorObserver:

[hidehiko, 2016/12/16 05:31:51]

Move to private?

[khmel, 2016/12/16 18:37:55]

Done.

+  void OnSkip() override;
+  void OnAccept() override;
+  void OnActorDestroyed(chromeos::ArcTermsOfServiceScreenActor* actor) override;
+
+ private:
+  void HandleTermsAccepted(bool accepted);
+
+  DISALLOW_COPY_AND_ASSIGN(ArcTermsOfServiceOOBENegotiator);
+};
+
+}  // namespace arc
+
+#endif  // CHROME_BROWSER_CHROMEOS_ARC_OPTIN_ARC_TERMS_OF_SERVICE_OOBE_NEGOTIATOR_H_