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

chrome/browser/ui/extensions/web_auth_flow_window.cc

+// 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.
+
+#include "chrome/browser/ui/extensions/web_auth_flow_window.h"
+
+#include "content/public/browser/browser_context.h"
+
+using content::BrowserContext;
+using content::WebContents;
+
+WebAuthFlowWindow* WebAuthFlowWindow::Create(

[Mihai Parparita -not on Chrome, 2012/05/14 20:59:39]

If all this does is call WebAuthFlowWindow::CreateWebAuthFlowWindow, then you
don't need to separate the two, you can just have WebAuthFlowWindow::Create be
the function that's implemented per-platform.

The reason why ShellWindow has the separation is because ::Create does some
massaging of parameters before calling ::CreateShellWindow

[Munjal (Google), 2012/05/15 00:55:20]

Done.

Yeah, thought about it when I did this but at the time I kept it thinking we
might have some non-trivial logic in future. But no point ni doing what we don't
need now. We can always add it later. So removed the redundant method.

+    Delegate* delegate,
+    BrowserContext* browser_context,
+    WebContents* contents) {
+  // This object will delete itself when the window is closed.
+  return WebAuthFlowWindow::CreateWebAuthFlowWindow(
+      delegate, browser_context, contents);
+}
+
+WebAuthFlowWindow::WebAuthFlowWindow(
+    Delegate* delegate,
+    BrowserContext* browser_context,
+    WebContents* contents)
+    : delegate_(delegate),
+      browser_context_(browser_context),
+      contents_(contents) {
+}
+
+WebAuthFlowWindow::~WebAuthFlowWindow() {
+}
+
+// TODO(munjal): Remove this once we have mac/linux implementations.
+#ifndef TOOLKIT_VIEWS
+// static
+WebAuthFlowWindow* WebAuthFlowWindow::CreateWebAuthFlowWindow(
+    Delegate* delegate,
+    BrowserContext* browser_context,
+    WebContents* contents) {
+  return NULL;
+}
+#endif