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

Side by Side Diff: third_party/WebKit/Source/core/page/Page.h

Issue 2949593003: Move navigation-induced hiding of form-validation-bubble to the browser process. (Closed)
Patch Set: Remove now unnecessary call from WebContentsImpl::RenderViewTerminated to delegate_->HideValidation… 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2013 Apple Inc. All rights 2 * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2013 Apple Inc. All rights
3 * reserved. 3 * reserved.
4 * Copyright (C) 2008 Torch Mobile Inc. All rights reserved. 4 * Copyright (C) 2008 Torch Mobile Inc. All rights reserved.
5 * (http://www.torchmobile.com/) 5 * (http://www.torchmobile.com/)
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 Frame* MainFrame() const { return main_frame_; } 145 Frame* MainFrame() const { return main_frame_; }
146 // Escape hatch for existing code that assumes that the root frame is 146 // Escape hatch for existing code that assumes that the root frame is
147 // always a LocalFrame. With OOPI, this is not always the case. Code that 147 // always a LocalFrame. With OOPI, this is not always the case. Code that
148 // depends on this will generally have to be rewritten to propagate any 148 // depends on this will generally have to be rewritten to propagate any
149 // necessary state through all renderer processes for that page and/or 149 // necessary state through all renderer processes for that page and/or
150 // coordinate/rely on the browser process to help dispatch/coordinate work. 150 // coordinate/rely on the browser process to help dispatch/coordinate work.
151 LocalFrame* DeprecatedLocalMainFrame() const { 151 LocalFrame* DeprecatedLocalMainFrame() const {
152 return ToLocalFrame(main_frame_); 152 return ToLocalFrame(main_frame_);
153 } 153 }
154 154
155 void WillUnloadDocument(const Document&);
156 void DocumentDetached(Document*); 155 void DocumentDetached(Document*);
157 156
158 bool OpenedByDOM() const; 157 bool OpenedByDOM() const;
159 void SetOpenedByDOM(); 158 void SetOpenedByDOM();
160 159
161 PageAnimator& Animator() { return *animator_; } 160 PageAnimator& Animator() { return *animator_; }
162 ChromeClient& GetChromeClient() const { return *chrome_client_; } 161 ChromeClient& GetChromeClient() const { return *chrome_client_; }
163 AutoscrollController& GetAutoscrollController() const { 162 AutoscrollController& GetAutoscrollController() const {
164 return *autoscroll_controller_; 163 return *autoscroll_controller_;
165 } 164 }
(...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after
369 #endif 368 #endif
370 369
371 int subframe_count_; 370 int subframe_count_;
372 }; 371 };
373 372
374 extern template class CORE_EXTERN_TEMPLATE_EXPORT Supplement<Page>; 373 extern template class CORE_EXTERN_TEMPLATE_EXPORT Supplement<Page>;
375 374
376 } // namespace blink 375 } // namespace blink
377 376
378 #endif // Page_h 377 #endif // Page_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/html/HTMLFormControlElementTest.cpp ('k') | third_party/WebKit/Source/core/page/Page.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698