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

Unified Diff: third_party/WebKit/Source/core/loader/EmptyClients.h

Issue 2383403002: Reflow comments in core/loader (Closed)
Patch Set: yhirano comments Created 4 years, 2 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: third_party/WebKit/Source/core/loader/EmptyClients.h
diff --git a/third_party/WebKit/Source/core/loader/EmptyClients.h b/third_party/WebKit/Source/core/loader/EmptyClients.h
index 5aae64f30dd5ff56f451d6d69998e7b5dc88edd8..109964416f8b780833bfa11b1f2cc61c27549905 100644
--- a/third_party/WebKit/Source/core/loader/EmptyClients.h
+++ b/third_party/WebKit/Source/core/loader/EmptyClients.h
@@ -53,15 +53,18 @@
/*
This file holds empty Client stubs for use by WebCore.
- Viewless element needs to create a dummy Page->LocalFrame->FrameView tree for use in parsing or executing JavaScript.
- This tree depends heavily on Clients (usually provided by WebKit classes).
- This file was first created for SVGImage as it had no way to access the current Page (nor should it,
- since Images are not tied to a page).
- See http://bugs.webkit.org/show_bug.cgi?id=5971 for the original discussion about this file.
+ Viewless element needs to create a dummy Page->LocalFrame->FrameView tree for
+ use in parsing or executing JavaScript. This tree depends heavily on Clients
+ (usually provided by WebKit classes).
+
+ This file was first created for SVGImage as it had no way to access the current
+ Page (nor should it, since Images are not tied to a page). See
+ http://bugs.webkit.org/show_bug.cgi?id=5971 for the original discussion about
+ this file.
Ideally, whenever you change a Client class, you should add a stub here.
- Brittle, yes. Unfortunate, yes. Hopefully temporary.
+ Brittle, yes. Unfortunate, yes. Hopefully temporary.
*/
namespace blink {

Powered by Google App Engine
This is Rietveld 408576698