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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/ToolbarSceneLayer.java

Issue 2787983002: [Home] Disable composited progress bar (Closed)
Patch Set: address comment Created 3 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/ToolbarSceneLayer.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/ToolbarSceneLayer.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/ToolbarSceneLayer.java
index 3c4c3cb6747908b1e1aaade655b6004f7a5ac730..6361b5d5a49d631554c18e8063759b24ae4d6ff4 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/ToolbarSceneLayer.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/scene_layer/ToolbarSceneLayer.java
@@ -66,7 +66,7 @@ public class ToolbarSceneLayer extends SceneOverlayLayer implements SceneOverlay
* @param fullscreenManager A ChromeFullscreenManager instance.
* @param resourceManager A ResourceManager for loading static resources.
* @param forceHideAndroidBrowserControls True if the Android browser controls are being hidden.
- * @param sizingFlags The sizing flags for the toolbar.
+ * @param viewportMode The sizing mode of the viewport being drawn in.
* @param isTablet If the device is a tablet.
* @param windowHeight The height of the window.
*/
@@ -78,7 +78,8 @@ public class ToolbarSceneLayer extends SceneOverlayLayer implements SceneOverlay
if (fullscreenManager == null) return;
ControlContainer toolbarContainer = fullscreenManager.getControlContainer();
- if (!isTablet && toolbarContainer != null) {
+ if (!isTablet && toolbarContainer != null
+ && !fullscreenManager.areBrowserControlsAtBottom()) {
if (mProgressBarDrawingInfo == null) mProgressBarDrawingInfo = new DrawingInfo();
toolbarContainer.getProgressBarDrawingInfo(mProgressBarDrawingInfo);
} else {
@@ -102,8 +103,7 @@ public class ToolbarSceneLayer extends SceneOverlayLayer implements SceneOverlay
fullscreenManager.areBrowserControlsAtBottom());
if (mProgressBarDrawingInfo == null) return;
- nativeUpdateProgressBar(mNativePtr,
- mProgressBarDrawingInfo.progressBarRect.left,
+ nativeUpdateProgressBar(mNativePtr, mProgressBarDrawingInfo.progressBarRect.left,
mProgressBarDrawingInfo.progressBarRect.top,
mProgressBarDrawingInfo.progressBarRect.width(),
mProgressBarDrawingInfo.progressBarRect.height(),
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698