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

Unified Diff: sky/engine/core/rendering/compositing/RenderLayerCompositor.h

Issue 721473002: Removed ScrollingCoordinator and a bunch of composited scrolling' (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: cr comments Created 6 years, 1 month 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: sky/engine/core/rendering/compositing/RenderLayerCompositor.h
diff --git a/sky/engine/core/rendering/compositing/RenderLayerCompositor.h b/sky/engine/core/rendering/compositing/RenderLayerCompositor.h
index 0a35a7ec8b94bfb2fff6bb3e92199471af3468c6..99ff8c824006f7b9025e661cedba8f3c40f506a3 100644
--- a/sky/engine/core/rendering/compositing/RenderLayerCompositor.h
+++ b/sky/engine/core/rendering/compositing/RenderLayerCompositor.h
@@ -37,7 +37,6 @@ class DocumentLifecycle;
class GraphicsLayer;
class GraphicsLayerFactory;
class Page;
-class ScrollingCoordinator;
enum CompositingUpdateType {
CompositingUpdateNone,
@@ -137,13 +136,8 @@ public:
void frameViewDidChangeSize();
void rootFixedBackgroundsChanged();
- bool scrollingLayerDidChange(RenderLayer*);
-
String layerTreeAsText(LayerTreeFlags);
- GraphicsLayer* layerForHorizontalScrollbar() const { return m_layerForHorizontalScrollbar.get(); }
- GraphicsLayer* layerForVerticalScrollbar() const { return m_layerForVerticalScrollbar.get(); }
-
void resetTrackedPaintInvalidationRects();
void setTracksPaintInvalidations(bool);
@@ -183,18 +177,12 @@ private:
void attachRootLayer(RootLayerAttachment);
void detachRootLayer();
- void updateOverflowControlsLayers();
-
Page* page() const;
GraphicsLayerFactory* graphicsLayerFactory() const;
- ScrollingCoordinator* scrollingCoordinator() const;
void enableCompositingModeIfNeeded();
- bool requiresHorizontalScrollbarLayer() const;
- bool requiresVerticalScrollbarLayer() const;
-
RenderView& m_renderView;
OwnPtr<GraphicsLayer> m_rootContentLayer;
OwnPtr<GraphicsLayer> m_rootTransformLayer;
@@ -224,10 +212,6 @@ private:
// Enclosing layer for overflow controls and the clipping layer
OwnPtr<GraphicsLayer> m_overflowControlsHostLayer;
-
- // Layers for overflow controls
- OwnPtr<GraphicsLayer> m_layerForHorizontalScrollbar;
- OwnPtr<GraphicsLayer> m_layerForVerticalScrollbar;
};
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698