Add Views implementation of ProfileSigninConfirmationDialog.

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

 // Copyright (c) 2013 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_PROFILE_SIGNIN_CONFIRMATION_DIALOG_H_
 #define CHROME_BROWSER_UI_WEBUI_SIGNIN_PROFILE_SIGNIN_CONFIRMATION_DIALOG_H_
 
 #include <string>
 
 #include "base/callback.h"
-#include "base/memory/weak_ptr.h"
 #include "base/string16.h"
 #include "chrome/browser/common/cancelable_request.h"
 #include "chrome/browser/history/history_types.h"
+#include "chrome/browser/ui/sync/profile_signin_confirmation_helper.h"
 #include "chrome/browser/ui/webui/constrained_web_dialog_ui.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/web_dialogs/web_dialog_delegate.h"
 
+class Browser;
 class Profile;
 class WebUIMessageHandler;
 
+namespace content {
+class WebContents;
+}
+
+namespace chrome {
+void ShowProfileSigninConfirmationDialog(

[Peter Kasting, 2013/05/15 00:11:03]

Why do we have this declaration both here and in browser_dialogs.h?  This
function should only be declared in one place.

[dconnelly, 2013/05/17 13:01:55]

It's declared in browser_dialogs.h.  It's re-declared here because tfarina asked
me to remove the browser_dialogs.h include, and the friend declaration requires
functions in a different namespace to be forward-declared.

See other comment below on the friend declaration.

[Peter Kasting, 2013/05/17 18:45:53]

The #include should not have been removed, but this is probably moot pending the
below comment.

+    Browser* browser, content::WebContents* web_contents,

[Peter Kasting, 2013/05/15 00:11:03]

Nit: While this wrapping style is legal, I think putting one arg per line is
more common and more readable.

Also, please name all arguments.

[dconnelly, 2013/05/17 13:01:55]

Done.

+    Profile*, const std::string&,
+    const base::Closure&, const base::Closure&, const base::Closure&);
+}
+
 // A tab-modal dialog to allow a user signing in with a managed account
 // to create a new Chrome profile.
 class ProfileSigninConfirmationDialog : public ui::WebDialogDelegate {
  public:
-  // Creates and shows the modal dialog.  |profile| is the current Chrome
-  // profile and |username| is the GAIA username that the user is signing
-  // in with.
-  static void ShowDialog(Profile* profile,
-                         const std::string& username,
-                         const base::Closure& cancel_signin,
-                         const base::Closure& signin_with_new_profile,
-                         const base::Closure& continue_signin);
-
   // Closes the dialog, which will delete itself.
   void Close() const;
 
  private:
+  friend void ::chrome::ShowProfileSigninConfirmationDialog(

[Peter Kasting, 2013/05/15 00:11:03]

Nit: It seems like we could avoid the need for this friend declaration by either
making the constructor public or making the creation function be a public static
method of this class instead of a global within the chrome namespace.  Either
way seems better; I'd rather avoid friend declarations for things other than
unit tests unless there's no other good way.  (I don't know offhand whether
perhaps the "public static method" route doesn't make sense, based on what
further cleanup you'd like to do after this patch.  I will note that it's still
OK to make the function a public static method on this class but actually define
it for TOOLKIT_VIEWS off in the *_views.cc file; we do that with lots of other
UI objects.)

[dconnelly, 2013/05/17 13:01:55]

Here's the idea: there is one top-level function (the friended one) that clients
can call.  That function is declared in browser_dialogs.h.  Its definition is in
the separate toolkit implementation files.

I dislike the friending too.  Based on your suggestion, how about I do the
following:

- create public static methods on both this webui class and on the views class
- keep the declaration of the global Show() function in browser_dialogs.h
- implement the global Show() function in both the webui impl and the views
impl, which just calls the respective static methods.

[Peter Kasting, 2013/05/17 18:45:53]

I'm OK with this.

+      Browser* browser, content::WebContents* web_contents,
+      Profile*, const std::string&,
+      const base::Closure&, const base::Closure&, const base::Closure&);
+
   ProfileSigninConfirmationDialog(
+      content::WebContents* web_contents,
       Profile* profile,
       const std::string& username,
       const base::Closure& cancel_signin,
       const base::Closure& signin_with_new_profile,
       const base::Closure& continue_signin);
   virtual ~ProfileSigninConfirmationDialog();
 
   // Shows the dialog and releases ownership of this object. It will
   // delete itself when the dialog is closed. If |prompt_for_new_profile|
   // is true, the dialog will offer to create a new profile before signin.
-  void Show(bool prompt_for_new_profile);
+  void Show(ui::DisplayCreateProfilePrompt prompt_for_new_profile);
 
   // WebDialogDelegate implementation.
   virtual ui::ModalType GetDialogModalType() const OVERRIDE;
   virtual string16 GetDialogTitle() const OVERRIDE;
   virtual GURL GetDialogContentURL() const OVERRIDE;
   virtual void GetWebUIMessageHandlers(
       std::vector<content::WebUIMessageHandler*>* handlers) const OVERRIDE;
   virtual void GetDialogSize(gfx::Size* size) const OVERRIDE;
   virtual std::string GetDialogArgs() const OVERRIDE;
   virtual void OnDialogClosed(const std::string& json_retval) OVERRIDE;
   virtual void OnCloseContents(content::WebContents* source,
                                bool* out_close_dialog) OVERRIDE;
   virtual bool ShouldShowDialogTitle() const OVERRIDE;
 
+  // The containing view.
+  content::WebContents* web_contents_;
+
   // Weak ptr to delegate.
   ConstrainedWebDialogDelegate* delegate_;
 
   // The GAIA username being signed in.
   std::string username_;
 
   // Whether to show the "Create a new profile" button.
-  bool prompt_for_new_profile_;
+  ui::DisplayCreateProfilePrompt prompt_for_new_profile_;
 
   // Dialog button callbacks.
   base::Closure cancel_signin_;
   base::Closure signin_with_new_profile_;
   base::Closure continue_signin_;
 
   // Weak pointer to the profile being signed-in.
   Profile* profile_;
 
   // Cleanup bookkeeping.  Labeled mutable to get around inherited const
   // label on GetWebUIMessageHandlers.
   mutable bool closed_by_handler_;
 
-  base::WeakPtrFactory<ProfileSigninConfirmationDialog> weak_pointer_factory_;
-
   DISALLOW_COPY_AND_ASSIGN(ProfileSigninConfirmationDialog);
 };
 
 #endif  // CHROME_BROWSER_UI_WEBUI_SIGNIN_PROFILE_SIGNIN_CONFIRMATION_DIALOG_H_