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

Unified Diff: chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java

Issue 1170843002: [Andorid] Migrate to ClipDrawable progress bar. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ChromeShell test fix 2 Created 5 years, 5 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
Index: chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java
diff --git a/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java b/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java
index ab193ff9b4aea46b6119ba171c79ba0ec89ce829..fd9bdc9e75a4beb074c4b0fa28f3af62e4b596e2 100644
--- a/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java
+++ b/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellToolbar.java
@@ -26,7 +26,7 @@ import org.chromium.chrome.browser.TabObserver;
import org.chromium.chrome.browser.UrlUtilities;
import org.chromium.chrome.browser.appmenu.AppMenuButtonHelper;
import org.chromium.chrome.browser.appmenu.AppMenuHandler;
-import org.chromium.chrome.browser.widget.SmoothProgressBar;
+import org.chromium.chrome.browser.widget.ToolbarProgressBar;
import org.chromium.chrome.shell.omnibox.SuggestionPopup;
import org.chromium.content.common.ContentSwitches;
@@ -34,31 +34,25 @@ import org.chromium.content.common.ContentSwitches;
* A Toolbar {@link View} that shows the URL and navigation buttons.
*/
public class ChromeShellToolbar extends LinearLayout {
- private static final long COMPLETED_PROGRESS_TIMEOUT_MS = 200;
-
- private final Runnable mClearProgressRunnable = new Runnable() {
- @Override
- public void run() {
- mProgressBar.setProgress(0);
- }
- };
private final Runnable mUpdateProgressRunnable = new Runnable() {
@Override
public void run() {
- mProgressBar.setProgress(mProgress);
- if (mLoading) {
- mStopReloadButton.setImageResource(
- R.drawable.btn_close);
+ mStopReloadButton.setImageResource(
+ mLoading ? R.drawable.btn_close : R.drawable.btn_toolbar_reload);
+
+ if (mProgress == 100.0f) {
+ if (mProgressBar.getProgress() != 1.0f) mProgressBar.setProgress(1.0f);
+ mProgressBar.finish(true);
} else {
- mStopReloadButton.setImageResource(R.drawable.btn_toolbar_reload);
- postOnAnimationDelayed(mClearProgressRunnable, COMPLETED_PROGRESS_TIMEOUT_MS);
+ mProgressBar.start();
+ mProgressBar.setProgress(mProgress / 100.0f);
}
}
};
private EditText mUrlTextView;
- private SmoothProgressBar mProgressBar;
+ private ToolbarProgressBar mProgressBar;
private ChromeShellTab mTab;
private final TabObserver mTabObserver;
@@ -121,7 +115,6 @@ public class ChromeShellToolbar extends LinearLayout {
}
private void onLoadProgressChanged(int progress) {
- removeCallbacks(mClearProgressRunnable);
removeCallbacks(mUpdateProgressRunnable);
mProgress = progress;
mLoading = progress != 100;
@@ -139,7 +132,7 @@ public class ChromeShellToolbar extends LinearLayout {
protected void onFinishInflate() {
super.onFinishInflate();
- mProgressBar = (SmoothProgressBar) findViewById(R.id.progress);
+ mProgressBar = (ToolbarProgressBar) findViewById(R.id.progress);
initializeUrlField();
initializeTabSwitcherButton();
initializeMenuButton();

Powered by Google App Engine
This is Rietveld 408576698