summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Nate Fischer <ntfschr@google.com> 2024-01-19 20:49:10 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2024-01-19 20:49:10 +0000
commit31827e997a68f36a5f2f6ec61a044b2907a2885e (patch)
tree3a69347405734f3edafc32c6548ba174b48060c2
parent16677ccc2b90ed67fbc6c9fd55cf85a870e4bac9 (diff)
parentbb5ba4b22dcefdbc67462ecf89e7f2f44a63b537 (diff)
Merge "WebView: document when shouldOverrideUrlLoading happens" into main
-rw-r--r--core/java/android/webkit/WebViewClient.java8
1 files changed, 8 insertions, 0 deletions
diff --git a/core/java/android/webkit/WebViewClient.java b/core/java/android/webkit/WebViewClient.java
index 2dfeae3567e5..80aad60d8bc4 100644
--- a/core/java/android/webkit/WebViewClient.java
+++ b/core/java/android/webkit/WebViewClient.java
@@ -65,6 +65,14 @@ public class WebViewClient {
* {@code true} causes the current WebView to abort loading the URL, while returning
* {@code false} causes the WebView to continue loading the URL as usual.
*
+ * <p>This callback is not called for all page navigations. In particular, this is not called
+ * for navigations which the app initiated with {@code loadUrl()}: this callback would not serve
+ * a purpose in this case, because the app already knows about the navigation. This callback
+ * lets the app know about navigations initiated by the web page (such as navigations initiated
+ * by JavaScript code), by the user (such as when the user taps on a link), or by an HTTP
+ * redirect (ex. if {@code loadUrl("foo.com")} redirects to {@code "bar.com"} because of HTTP
+ * 301).
+ *
* <p class="note"><b>Note:</b> Do not call {@link WebView#loadUrl(String)} with the request's
* URL and then return {@code true}. This unnecessarily cancels the current load and starts a
* new load with the same URL. The correct way to continue loading a given URL is to simply