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

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

Issue 2394663002: Blimp: Propagating page loading status to tab (Closed)
Patch Set: dtrainor@ comments Created 4 years, 2 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/TabObserver.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tab/TabObserver.java b/chrome/android/java/src/org/chromium/chrome/browser/tab/TabObserver.java
index d10d0fa20cfa3883bf6ccb5ccf9d238a5cc3436b..5b72fdbdf05aa8433fea54bb8765766bfa3fc4f3 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tab/TabObserver.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tab/TabObserver.java
@@ -211,17 +211,11 @@ public interface TabObserver {
/**
* Called when load is started for a given frame.
* @param tab The notifying {@link Tab}.
- * @param frameId A positive, non-zero integer identifying the navigating frame.
- * @param parentFrameId The frame identifier of the frame containing the navigating frame,
- * or -1 if the frame is not contained in another frame.
* @param isMainFrame Whether the load is happening for the main frame.
* @param validatedUrl The validated URL that is being navigated to.
- * @param isErrorPage Whether this is navigating to an error page.
- * @param isIframeSrcdoc Whether this is navigating to about:srcdoc.
*/
public void onDidStartProvisionalLoadForFrame(
- Tab tab, long frameId, long parentFrameId, boolean isMainFrame, String validatedUrl,
- boolean isErrorPage, boolean isIframeSrcdoc);
+ Tab tab, boolean isMainFrame, String validatedUrl);
/**
* Notifies that the provisional load was successfully committed. The RenderViewHost is now

Powered by Google App Engine
This is Rietveld 408576698