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

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

Issue 878303002: Remove more scrolling code from Sky (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/dom/Element.h » ('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 e5c7c195c3973bf74aaeb1e96ce719df6e9d46f3..99639560dbc4046cc1e3b03dd2b5408a26598b9f 100644
--- a/sky/engine/core/dom/Document.cpp
+++ b/sky/engine/core/dom/Document.cpp
@@ -1814,14 +1814,6 @@ void Document::enqueueUniqueAnimationFrameEvent(PassRefPtr<Event> event)
ensureScriptedAnimationController().enqueuePerFrameEvent(event);
}
-void Document::enqueueScrollEventForNode(Node* target)
-{
- // Per the W3C CSSOM View Module only scroll events fired at the document should bubble.
- RefPtr<Event> scrollEvent = target->isDocumentNode() ? Event::createBubble(EventTypeNames::scroll) : Event::create(EventTypeNames::scroll);
- scrollEvent->setTarget(target);
- ensureScriptedAnimationController().enqueuePerFrameEvent(scrollEvent.release());
-}
-
void Document::enqueueResizeEvent()
{
RefPtr<Event> event = Event::create(EventTypeNames::resize);
@@ -1844,8 +1836,6 @@ void Document::addListenerTypeIfNeeded(const AtomicString& eventType)
addListenerType(ANIMATIONITERATION_LISTENER);
} else if (eventType == EventTypeNames::transitionend) {
addListenerType(TRANSITIONEND_LISTENER);
- } else if (eventType == EventTypeNames::scroll) {
- addListenerType(SCROLL_LISTENER);
}
}
@@ -2200,7 +2190,7 @@ IntSize Document::initialViewportSize() const
{
if (!view())
return IntSize();
- return view()->unscaledVisibleContentSize(IncludeScrollbars);
+ return view()->unscaledVisibleContentSize();
}
Node* eventTargetNodeForDocument(Document* doc)
@@ -2213,26 +2203,6 @@ Node* eventTargetNodeForDocument(Document* doc)
return node;
}
-void Document::adjustFloatQuadsForScroll(Vector<FloatQuad>& quads)
-{
- if (!view())
- return;
-
- LayoutRect visibleContentRect = view()->visibleContentRect();
- for (size_t i = 0; i < quads.size(); ++i) {
- quads[i].move(-FloatSize(visibleContentRect.x().toFloat(), visibleContentRect.y().toFloat()));
- }
-}
-
-void Document::adjustFloatRectForScroll(FloatRect& rect)
-{
- if (!view())
- return;
-
- LayoutRect visibleContentRect = view()->visibleContentRect();
- rect.move(-FloatSize(visibleContentRect.x().toFloat(), visibleContentRect.y().toFloat()));
-}
-
void Document::decrementActiveParserCount()
{
--m_activeParserCount;
« no previous file with comments | « sky/engine/core/dom/Document.h ('k') | sky/engine/core/dom/Element.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698