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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java

Issue 2482233002: Remove visible viewport dependency in CompositorView (Closed)
Patch Set: address comments Created 4 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: chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java
index 09c01f82075c0c41f1c05997bbd9e455943eb457..a5731bf57d4bdcc84c2c0813c854cd05c862674c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManager.java
@@ -273,10 +273,10 @@ public abstract class LayoutManager implements LayoutUpdateHost, LayoutProvider,
}
@Override
- public SceneLayer getUpdatedActiveSceneLayer(Rect viewport, Rect contentViewport,
- LayerTitleCache layerTitleCache, TabContentManager tabContentManager,
- ResourceManager resourceManager, ChromeFullscreenManager fullscreenManager) {
- return mActiveLayout.getUpdatedSceneLayer(viewport, contentViewport, layerTitleCache,
+ public SceneLayer getUpdatedActiveSceneLayer(Rect viewport, LayerTitleCache layerTitleCache,
+ TabContentManager tabContentManager, ResourceManager resourceManager,
+ ChromeFullscreenManager fullscreenManager) {
+ return mActiveLayout.getUpdatedSceneLayer(viewport, getViewportPixel(null), layerTitleCache,
tabContentManager, resourceManager, fullscreenManager);
}

Powered by Google App Engine
This is Rietveld 408576698