Start implementing an auth flow for platform apps to be able to do auth

chrome/browser/ui/extensions/web_auth_flow_window.h

Index: chrome/browser/ui/extensions/web_auth_flow_window.h
===================================================================
--- chrome/browser/ui/extensions/web_auth_flow_window.h	(revision 0)
+++ chrome/browser/ui/extensions/web_auth_flow_window.h	(revision 0)
@@ -0,0 +1,55 @@
+// Copyright (c) 2012 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_EXTENSIONS_WEB_AUTH_FLOW_WINDOW_H_
+#define CHROME_BROWSER_UI_EXTENSIONS_WEB_AUTH_FLOW_WINDOW_H_
+#pragma once
+
+#include "base/basictypes.h"
+
+class Profile;
+
+namespace content {
+class BrowserContext;
+class WebContents;
+}
+
+// Platform independent abstraction for a window that performs web auth flow.
+// Platform specific implementations implement this abstract class.
+class WebAuthFlowWindow {
+ public:
+  class Delegate {
+   public:
+    virtual void OnClose() = 0;
+  };

[sky, 2012/05/16 23:17:11]

Add a protected virtual destructor to make it clear WebAuthFlowWindow doesn't
own the Delegate.

+
+  // TODO(munjal): Allow customizing these?
+  static const int kDefaultWidth = 1024;
+  static const int kDefaultHeight = 768;
+
+  // Creates a new instance of WebAuthFlowWindow with the given parmaters.
+  // Delegate::OnClose will be called when the window is closed.
+  static WebAuthFlowWindow* Create(Delegate* delegate,
+                                   content::BrowserContext* browser_context,
+                                   content::WebContents* contents);
+
+  virtual ~WebAuthFlowWindow();
+
+  // Show the window.
+  virtual void Show() = 0;
+
+ protected:
+  explicit WebAuthFlowWindow(Delegate* delegate,

[sky, 2012/05/16 23:17:11]

no explicit

[Munjal (Google), 2012/05/16 23:33:44]

Done.

+                             content::BrowserContext* browser_context,
+                             content::WebContents* contents);
+
+  Delegate* delegate_;

[sky, 2012/05/16 23:17:11]

Style guide says no protected members.

[Munjal (Google), 2012/05/16 23:33:44]

Done.

+  content::BrowserContext* browser_context_;
+  content::WebContents* contents_;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(WebAuthFlowWindow);
+};
+
+#endif  // CHROME_BROWSER_UI_EXTENSIONS_WEB_AUTH_FLOW_WINDOW_H_