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

Unified Diff: sky/engine/core/dom/Document.cpp

Issue 858713002: Remove viewportDefiningElement. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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
« no previous file with comments | « sky/engine/core/dom/Document.h ('k') | sky/engine/core/rendering/RenderBox.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/dom/Document.cpp
diff --git a/sky/engine/core/dom/Document.cpp b/sky/engine/core/dom/Document.cpp
index 72eba2fb419b8a362a5083bfeff90a32e91dca1c..684d82bbe0dd818e8a35b16ea3acf94dd368f47c 100644
--- a/sky/engine/core/dom/Document.cpp
+++ b/sky/engine/core/dom/Document.cpp
@@ -1289,25 +1289,6 @@ DocumentParser* Document::startParsing()
return m_parser.get();
}
-Element* Document::viewportDefiningElement(RenderStyle* rootStyle) const
-{
- // If a BODY element sets non-visible overflow, it is to be propagated to the viewport, as long
- // as the following conditions are all met:
- // (1) The root element is HTML.
- // (2) It is the primary BODY element (we only assert for this, expecting callers to behave).
- // (3) The root element has visible overflow.
- // Otherwise it's the root element's properties that are to be propagated.
- Element* rootElement = documentElement();
- if (!rootElement)
- return 0;
- if (!rootStyle) {
- rootStyle = rootElement->renderStyle();
- if (!rootStyle)
- return 0;
- }
- return rootElement;
-}
-
void Document::implicitClose()
{
ASSERT(!inStyleRecalc());
« no previous file with comments | « sky/engine/core/dom/Document.h ('k') | sky/engine/core/rendering/RenderBox.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698