Implement the new Sync Confirmation dialog on Linux and Windows.

chrome/browser/ui/webui/signin/sync_confirmation_handler.h

Index: chrome/browser/ui/webui/signin/sync_confirmation_handler.h
diff --git a/chrome/browser/ui/webui/signin/sync_confirmation_handler.h b/chrome/browser/ui/webui/signin/sync_confirmation_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..ba44e904da2d648ed83b6b6d4209d549fdec45dd
--- /dev/null
+++ b/chrome/browser/ui/webui/signin/sync_confirmation_handler.h
@@ -0,0 +1,35 @@
+// 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_UI_WEBUI_SIGNIN_SYNC_CONFIRMATION_HANDLER_H_
+#define CHROME_BROWSER_UI_WEBUI_SIGNIN_SYNC_CONFIRMATION_HANDLER_H_
+
+#include "base/macros.h"
+#include "base/values.h"

[Dan Beam, 2016/01/06 22:57:58]

forward declare base::ListValue instead of including base/values.h

[anthonyvd, 2016/01/08 22:39:42]

Done.

+#include "components/signin/core/browser/account_tracker_service.h"
+#include "content/public/browser/web_ui_message_handler.h"
+
+class SyncConfirmationHandler : public content::WebUIMessageHandler,
+                                public AccountTrackerService::Observer {
+ public:
+  SyncConfirmationHandler();
+  ~SyncConfirmationHandler() override;
+

[Dan Beam, 2016/01/06 22:57:58]

// content::WebUIMessageHandler:

[anthonyvd, 2016/01/08 22:39:42]

Done.

+  void RegisterMessages() override;
+
+  // AccountTrackerService::Observer:
+  void OnAccountUpdated(const AccountInfo& info) override;
+
+ private:
+  void HandleConfirmMessage(const base::ListValue* args);

[Dan Beam, 2016/01/06 22:57:58]

these should be Handle + MessageName (without "Message" at the end), i.e.
HandleConfirm

[anthonyvd, 2016/01/08 22:39:42]

Done.

+  void HandleUndoMessage(const base::ListValue* args);
+  void HandleInitializedMessage(const base::ListValue* args);
+  void HandleGoToSettingsMessage(const base::ListValue* args);

[Dan Beam, 2016/01/06 22:57:58]

doc comments about what these do, i.e.

  // Handles "confirm" message from the page. No arguments.

[anthonyvd, 2016/01/08 22:39:42]

Done.

+
+  void SetUserImageURL(std::string url);

[Dan Beam, 2016/01/06 22:57:58]

doc comment, shouldn't this be const std::string& url or a GURL()?

[anthonyvd, 2016/01/08 22:39:42]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(SyncConfirmationHandler);
+};
+
+#endif  // CHROME_BROWSER_UI_WEBUI_SIGNIN_SYNC_CONFIRMATION_HANDLER_H_