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

Side by Side Diff: content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java

Issue 1155713005: Use a resource throttle to implement shouldOverrideUrlLoading. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add flag showing that shouldOverrideUrl cancelled the navigation Created 5 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.content.browser.webcontents; 5 package org.chromium.content.browser.webcontents;
6 6
7 import org.chromium.base.CalledByNative; 7 import org.chromium.base.CalledByNative;
8 import org.chromium.base.JNINamespace; 8 import org.chromium.base.JNINamespace;
9 import org.chromium.base.ObserverList; 9 import org.chromium.base.ObserverList;
10 import org.chromium.base.ObserverList.RewindableIterator; 10 import org.chromium.base.ObserverList.RewindableIterator;
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 @CalledByNative 87 @CalledByNative
88 public void didStopLoading(String url) { 88 public void didStopLoading(String url) {
89 for (mObserversIterator.rewind(); mObserversIterator.hasNext();) { 89 for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
90 mObserversIterator.next().didStopLoading(url); 90 mObserversIterator.next().didStopLoading(url);
91 } 91 }
92 } 92 }
93 93
94 @Override 94 @Override
95 @CalledByNative 95 @CalledByNative
96 public void didFailLoad(boolean isProvisionalLoad, boolean isMainFrame, int errorCode, 96 public void didFailLoad(boolean isProvisionalLoad, boolean isMainFrame, int errorCode,
97 String description, String failingUrl) { 97 String description, String failingUrl, boolean wasIgnoredByHandler) {
98 for (mObserversIterator.rewind(); mObserversIterator.hasNext();) { 98 for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
99 mObserversIterator.next().didFailLoad( 99 mObserversIterator.next().didFailLoad(isProvisionalLoad, isMainFrame , errorCode,
100 isProvisionalLoad, isMainFrame, errorCode, description, fail ingUrl); 100 description, failingUrl, wasIgnoredByHandler);
101 } 101 }
102 } 102 }
103 103
104 @Override 104 @Override
105 @CalledByNative 105 @CalledByNative
106 public void didNavigateMainFrame(String url, String baseUrl, 106 public void didNavigateMainFrame(String url, String baseUrl,
107 boolean isNavigationToDifferentPage, boolean isFragmentNavigation, i nt statusCode) { 107 boolean isNavigationToDifferentPage, boolean isFragmentNavigation, i nt statusCode) {
108 for (mObserversIterator.rewind(); mObserversIterator.hasNext();) { 108 for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
109 mObserversIterator.next().didNavigateMainFrame( 109 mObserversIterator.next().didNavigateMainFrame(
110 url, baseUrl, isNavigationToDifferentPage, isFragmentNavigat ion, statusCode); 110 url, baseUrl, isNavigationToDifferentPage, isFragmentNavigat ion, statusCode);
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
228 228
229 if (mNativeWebContentsObserverProxy != 0) { 229 if (mNativeWebContentsObserverProxy != 0) {
230 nativeDestroy(mNativeWebContentsObserverProxy); 230 nativeDestroy(mNativeWebContentsObserverProxy);
231 mNativeWebContentsObserverProxy = 0; 231 mNativeWebContentsObserverProxy = 0;
232 } 232 }
233 } 233 }
234 234
235 private native long nativeInit(WebContentsImpl webContents); 235 private native long nativeInit(WebContentsImpl webContents);
236 private native void nativeDestroy(long nativeWebContentsObserverProxy); 236 private native void nativeDestroy(long nativeWebContentsObserverProxy);
237 } 237 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698