Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(457)

Unified Diff: components/autofill/content/renderer/form_autofill_util.h

Issue 2928033002: Move GetDocument method from WebFrame to WebLocalFrame. (Closed)
Patch Set: Split a DCHECK in two as suggested by boliu@. Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/autofill/content/renderer/form_autofill_util.h
diff --git a/components/autofill/content/renderer/form_autofill_util.h b/components/autofill/content/renderer/form_autofill_util.h
index 998d5b0459aa90ed6a2a52c8f852df244be56f31..eac0abc26e7b6875a60caa325994177ab12519b2 100644
--- a/components/autofill/content/renderer/form_autofill_util.h
+++ b/components/autofill/content/renderer/form_autofill_util.h
@@ -25,8 +25,8 @@ class WebDocument;
class WebElement;
class WebFormControlElement;
class WebFormElement;
-class WebFrame;
class WebInputElement;
+class WebLocalFrame;
class WebNode;
}
@@ -87,7 +87,7 @@ bool ExtractFormData(const blink::WebFormElement& form_element, FormData* data);
// equals |form_element|. If |form_element| is null, checks if forms action
// equals |action|. Returns true if so. For forms with empty or unspecified
// actions, all form data are used for comparison.
-bool IsFormVisible(blink::WebFrame* frame,
+bool IsFormVisible(blink::WebLocalFrame* frame,
const blink::WebFormElement& form_element,
const GURL& action,
const GURL& origin,
@@ -255,7 +255,7 @@ bool ClearPreviewedFormWithElement(const blink::WebFormControlElement& element,
// <body/>
// <html/>
// Meta, script and title tags don't influence the emptiness of a webpage.
-bool IsWebpageEmpty(const blink::WebFrame* frame);
+bool IsWebpageEmpty(const blink::WebLocalFrame* frame);
// This function checks whether the children of |element|
// are of the type <script>, <meta>, or <title>.
« no previous file with comments | « chrome/renderer/worker_content_settings_client.cc ('k') | components/autofill/content/renderer/form_autofill_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698