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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java

Issue 2767803002: Adds metric to help assess last_n impact on tab restores. (M58 merge) (Closed)
Patch Set: 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
Index: chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java b/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
index 1f8767d74d93e9eca2d6bf62d0abd64efc403c34..b6b3dac2e70a20c947d26792076dbba34d41ca24 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java
@@ -895,7 +895,9 @@ public class Tab
public void stopLoading() {
if (isLoading()) {
RewindableIterator<TabObserver> observers = getTabObservers();
- while (observers.hasNext()) observers.next().onPageLoadFinished(this);
+ while (observers.hasNext()) {
+ observers.next().onPageLoadFinished(this);
+ }
}
if (getWebContents() != null) getWebContents().stop();
}
@@ -1525,7 +1527,6 @@ public class Tab
* Called when a page has finished loading.
*/
protected void didFinishPageLoad() {
- mIsBeingRestored = false;
mIsTabStateDirty = true;
updateTitle();
updateFullscreenEnabledState();
@@ -1540,6 +1541,7 @@ public class Tab
if (mTabUma != null) mTabUma.onPageLoadFinished();
for (TabObserver observer : mObservers) observer.onPageLoadFinished(this);
+ mIsBeingRestored = false;
}
/**
@@ -1547,9 +1549,11 @@ public class Tab
* @param errorCode The error code causing the page to fail loading.
*/
protected void didFailPageLoad(int errorCode) {
- mIsBeingRestored = false;
if (mTabUma != null) mTabUma.onLoadFailed(errorCode);
- for (TabObserver observer : mObservers) observer.onPageLoadFailed(this, errorCode);
+ for (TabObserver observer : mObservers) {
+ observer.onPageLoadFailed(this, errorCode);
+ }
+ mIsBeingRestored = false;
}
/**
@@ -2561,9 +2565,15 @@ public class Tab
*/
void handleTabCrash() {
mIsLoading = false;
- mIsBeingRestored = false;
if (mTabUma != null) mTabUma.onRendererCrashed();
+
+ boolean sadTabShown = isShowingSadTab();
+ RewindableIterator<TabObserver> observers = getTabObservers();
+ while (observers.hasNext()) {
+ observers.next().onCrash(this, sadTabShown);
+ }
+ mIsBeingRestored = false;
}
/**

Powered by Google App Engine
This is Rietveld 408576698