Move forms/ out of webkit/.

content/public/common/password_form_fill_data.cc

Index: content/public/common/password_form_fill_data.cc
diff --git a/webkit/forms/password_form_dom_manager.cc b/content/public/common/password_form_fill_data.cc
similarity index 55%
rename from webkit/forms/password_form_dom_manager.cc
rename to content/public/common/password_form_fill_data.cc
index 93545982fd1def4535e8378db1d130e8169cb0e6..5d313daa4e4723490931e3021eb7289392a2b776 100644
--- a/webkit/forms/password_form_dom_manager.cc
+++ b/content/public/common/password_form_fill_data.cc
@@ -1,20 +1,13 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright 2012 The Chromium Authors. All rights reserved.

[stuartmorgan, 2012/10/02 12:17:51]

Keep 2011

[blundell, 2012/10/03 15:06:07]

Done.

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "webkit/forms/password_form_dom_manager.h"
+#include "content/public/common/password_form_fill_data.h"
 
 #include "base/logging.h"
-#include "third_party/WebKit/Source/WebKit/chromium/public/WebInputElement.h"
-#include "third_party/WebKit/Source/WebKit/chromium/public/WebPasswordFormData.h"
-#include "webkit/forms/form_field.h"
+#include "content/public/common/form_field.h"
 
-using WebKit::WebFormElement;
-using WebKit::WebInputElement;
-using WebKit::WebPasswordFormData;
-
-namespace webkit {
-namespace forms {
+namespace content {
 
 PasswordFormFillData::PasswordFormFillData() : wait_for_username(false) {
 }
@@ -22,21 +15,11 @@ PasswordFormFillData::PasswordFormFillData() : wait_for_username(false) {
 PasswordFormFillData::~PasswordFormFillData() {
 }
 
-scoped_ptr<PasswordForm> PasswordFormDomManager::CreatePasswordForm(
-    const WebFormElement& webform) {
-  WebPasswordFormData web_password_form(webform);
-  if (web_password_form.isValid())
-    return scoped_ptr<PasswordForm>(new PasswordForm(web_password_form));
-  return scoped_ptr<PasswordForm>();
-}
-
-// static
-void PasswordFormDomManager::InitFillData(
-    const PasswordForm& form_on_page,
-    const PasswordFormMap& matches,
-    const PasswordForm* const preferred_match,
-    bool wait_for_username_before_autofill,
-    PasswordFormFillData* result) {
+void initPasswordFormFillData(const PasswordForm& form_on_page,

[stuartmorgan, 2012/10/02 12:17:51]

s/init/Init/

[jam, 2012/10/02 16:18:14]

static methods defined in headers in content/public should be implemented in
content/common (just like the webkit api)

if you absolutely need to have a cc file in
content/public/common/password_form_fill_data.cc for the constructor/destructor
above (to avoid a clang error), then put the static method in
content/common/password_form_fill_data_impl.cc

[blundell, 2012/10/03 15:06:07]

Noticed on re-inspection that this code is also used only in chrome/. Moved to
chrome/common.

On 2012/10/02 16:18:14, John Abd-El-Malek wrote:

+                              const PasswordFormMap& matches,
+                              const PasswordForm* const preferred_match,
+                              bool wait_for_username_before_autofill,
+                              PasswordFormFillData* result) {
   // Note that many of the |FormField| members are not initialized for
   // |username_field| and |password_field| because they are currently not used
   // by the password autocomplete code.
@@ -62,5 +45,4 @@ void PasswordFormDomManager::InitFillData(
   }
 }
 
-}  // namespace forms
-}  // namespace webkit
+}  // namespace content