Avi Drissman | 8ba1bad | 2022-09-13 19:22:36 | [diff] [blame] | 1 | // Copyright 2012 The Chromium Authors |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
[email protected] | 62885ab | 2013-01-23 03:55:16 | [diff] [blame] | 5 | #include "components/navigation_interception/intercept_navigation_delegate.h" |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 6 | |
Gyuyoung Kim | cb7965e | 2018-01-25 00:39:01 | [diff] [blame] | 7 | #include <memory> |
| 8 | |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 9 | #include "base/android/jni_android.h" |
| 10 | #include "base/android/jni_string.h" |
Avi Drissman | 12be031 | 2023-01-11 09:16:09 | [diff] [blame] | 11 | #include "base/functional/bind.h" |
| 12 | #include "base/functional/callback.h" |
Ryan Hamilton | 7f3bd3d | 2022-04-23 00:07:39 | [diff] [blame] | 13 | #include "base/strings/escape.h" |
Mohamed Heikal | bd64131 | 2019-06-22 00:14:37 | [diff] [blame] | 14 | #include "components/navigation_interception/jni_headers/InterceptNavigationDelegate_jni.h" |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 15 | #include "content/public/browser/browser_thread.h" |
David Bokan | 2a48f7bb | 2021-07-09 13:21:36 | [diff] [blame] | 16 | #include "content/public/browser/navigation_handle.h" |
clamy | 40c9e14 | 2015-09-29 11:18:47 | [diff] [blame] | 17 | #include "content/public/browser/navigation_throttle.h" |
jaekyun | 03890319 | 2015-03-31 14:15:59 | [diff] [blame] | 18 | #include "content/public/browser/render_frame_host.h" |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 19 | #include "content/public/browser/render_view_host.h" |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 20 | #include "content/public/browser/web_contents.h" |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 21 | #include "mojo/public/cpp/bindings/self_owned_receiver.h" |
| 22 | #include "net/http/http_status_code.h" |
| 23 | #include "net/http/http_util.h" |
| 24 | #include "net/url_request/redirect_info.h" |
| 25 | #include "net/url_request/redirect_util.h" |
| 26 | #include "services/network/public/cpp/parsed_headers.h" |
| 27 | #include "services/network/public/cpp/resource_request.h" |
| 28 | #include "services/network/public/cpp/single_request_url_loader_factory.h" |
| 29 | #include "services/network/public/mojom/url_response_head.mojom.h" |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 30 | #include "url/android/gurl_android.h" |
[email protected] | e3b599e | 2013-07-05 07:15:17 | [diff] [blame] | 31 | #include "url/gurl.h" |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 32 | |
| 33 | using base::android::ConvertUTF8ToJavaString; |
| 34 | using base::android::ScopedJavaLocalRef; |
| 35 | using content::BrowserThread; |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 36 | using content::RenderViewHost; |
| 37 | using content::WebContents; |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 38 | using ui::PageTransition; |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 39 | |
[email protected] | 8812e3d0 | 2013-05-22 12:38:53 | [diff] [blame] | 40 | namespace navigation_interception { |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 41 | |
| 42 | namespace { |
| 43 | |
thestig | 3b6a2f1 | 2015-09-25 08:17:20 | [diff] [blame] | 44 | const void* const kInterceptNavigationDelegateUserDataKey = |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 45 | &kInterceptNavigationDelegateUserDataKey; |
| 46 | |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 47 | bool CheckIfShouldIgnoreNavigationOnUIThread( |
| 48 | content::NavigationHandle* navigation_handle) { |
mostynb | ad1e8c96 | 2015-03-25 21:51:12 | [diff] [blame] | 49 | DCHECK_CURRENTLY_ON(BrowserThread::UI); |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 50 | DCHECK(navigation_handle); |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 51 | |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 52 | InterceptNavigationDelegate* intercept_navigation_delegate = |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 53 | InterceptNavigationDelegate::Get(navigation_handle->GetWebContents()); |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 54 | if (!intercept_navigation_delegate) |
| 55 | return false; |
| 56 | |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 57 | return intercept_navigation_delegate->ShouldIgnoreNavigation( |
| 58 | navigation_handle); |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 59 | } |
| 60 | |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 61 | class RedirectURLLoader : public network::mojom::URLLoader { |
| 62 | public: |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 63 | RedirectURLLoader(const network::ResourceRequest& resource_request, |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 64 | mojo::PendingRemote<network::mojom::URLLoaderClient> client) |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 65 | : client_(std::move(client)), request_(resource_request) {} |
| 66 | |
| 67 | void DoRedirect(std::unique_ptr<GURL> url) { |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 68 | net::HttpStatusCode response_code = net::HTTP_TEMPORARY_REDIRECT; |
| 69 | auto response_head = network::mojom::URLResponseHead::New(); |
| 70 | response_head->encoded_data_length = 0; |
| 71 | response_head->headers = base::MakeRefCounted<net::HttpResponseHeaders>( |
| 72 | net::HttpUtil::AssembleRawHeaders("HTTP/1.1 307 Temporary Redirect")); |
| 73 | |
| 74 | // Avoid a round-trip to the network service by pre-parsing headers. |
| 75 | // This doesn't violate: `docs/security/rule-of-2.md`, because the input is |
| 76 | // trusted, before appending the Location: <url> header. |
| 77 | response_head->parsed_headers = |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 78 | network::PopulateParsedHeaders(response_head->headers.get(), *url); |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 79 | |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 80 | response_head->headers->AddHeader("Location", url->spec()); |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 81 | |
| 82 | auto first_party_url_policy = |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 83 | request_.update_first_party_url_on_redirect |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 84 | ? net::RedirectInfo::FirstPartyURLPolicy::UPDATE_URL_ON_REDIRECT |
| 85 | : net::RedirectInfo::FirstPartyURLPolicy::NEVER_CHANGE_URL; |
| 86 | |
| 87 | client_->OnReceiveRedirect( |
| 88 | net::RedirectInfo::ComputeRedirectInfo( |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 89 | request_.method, request_.url, request_.site_for_cookies, |
| 90 | first_party_url_policy, request_.referrer_policy, |
| 91 | request_.referrer.spec(), response_code, *url, absl::nullopt, |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 92 | /*insecure_scheme_was_upgraded=*/false, |
| 93 | /*copy_fragment=*/false), |
| 94 | std::move(response_head)); |
| 95 | } |
| 96 | |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 97 | void OnNonRedirectAsyncAction() { |
| 98 | client_->OnComplete(network::URLLoaderCompletionStatus(net::ERR_ABORTED)); |
| 99 | } |
| 100 | |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 101 | RedirectURLLoader(const RedirectURLLoader&) = delete; |
| 102 | RedirectURLLoader& operator=(const RedirectURLLoader&) = delete; |
| 103 | |
| 104 | ~RedirectURLLoader() override = default; |
| 105 | |
| 106 | private: |
| 107 | // network::mojom::URLLoader overrides: |
| 108 | void FollowRedirect( |
| 109 | const std::vector<std::string>& removed_headers, |
| 110 | const net::HttpRequestHeaders& modified_headers, |
| 111 | const net::HttpRequestHeaders& modified_cors_exempt_headers, |
| 112 | const absl::optional<GURL>& new_url) override { |
| 113 | NOTREACHED(); |
| 114 | } |
| 115 | void SetPriority(net::RequestPriority priority, |
| 116 | int intra_priority_value) override {} |
| 117 | void PauseReadingBodyFromNet() override {} |
| 118 | void ResumeReadingBodyFromNet() override {} |
| 119 | |
| 120 | mojo::Remote<network::mojom::URLLoaderClient> client_; |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 121 | network::ResourceRequest request_; |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 122 | }; |
| 123 | |
[email protected] | a8e69a74 | 2013-10-15 10:58:55 | [diff] [blame] | 124 | } // namespace |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 125 | |
| 126 | // static |
| 127 | void InterceptNavigationDelegate::Associate( |
| 128 | WebContents* web_contents, |
dcheng | 84c358e | 2016-04-26 07:05:53 | [diff] [blame] | 129 | std::unique_ptr<InterceptNavigationDelegate> delegate) { |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 130 | web_contents->SetUserData(kInterceptNavigationDelegateUserDataKey, |
avi | 8945fc9 | 2017-05-02 16:03:23 | [diff] [blame] | 131 | std::move(delegate)); |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 132 | } |
| 133 | |
| 134 | // static |
| 135 | InterceptNavigationDelegate* InterceptNavigationDelegate::Get( |
| 136 | WebContents* web_contents) { |
dtrainor | 037df0d | 2014-10-08 18:05:24 | [diff] [blame] | 137 | return static_cast<InterceptNavigationDelegate*>( |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 138 | web_contents->GetUserData(kInterceptNavigationDelegateUserDataKey)); |
| 139 | } |
| 140 | |
| 141 | // static |
dcheng | 84c358e | 2016-04-26 07:05:53 | [diff] [blame] | 142 | std::unique_ptr<content::NavigationThrottle> |
David Bokan | 2a48f7bb | 2021-07-09 13:21:36 | [diff] [blame] | 143 | InterceptNavigationDelegate::MaybeCreateThrottleFor( |
Charlie Harrison | 3286ab7 | 2019-02-13 20:13:30 | [diff] [blame] | 144 | content::NavigationHandle* handle, |
| 145 | navigation_interception::SynchronyMode mode) { |
David Bokan | 2a48f7bb | 2021-07-09 13:21:36 | [diff] [blame] | 146 | // Navigations in a subframe or non-primary frame tree should not be |
| 147 | // intercepted. As examples of a non-primary frame tree, a navigation |
| 148 | // occurring in a Portal element or an unactivated prerendering page should |
| 149 | // not launch an app. |
| 150 | // TODO(bokan): This is a bit of a stopgap approach since we won't run |
| 151 | // throttles again when the prerender is activated which means links that are |
| 152 | // prerendered will avoid launching an app intent that a regular navigation |
| 153 | // would have. Longer term we'll want prerender activation to check for app |
| 154 | // intents, or have this throttle cancel the prerender if an intent would |
| 155 | // have been launched (without launching the intent). It's also not clear |
| 156 | // what the right behavior for <portal> elements is. |
| 157 | // https://crbug.com/1227659. |
| 158 | if (!handle->IsInPrimaryMainFrame()) |
| 159 | return nullptr; |
| 160 | |
Gyuyoung Kim | cb7965e | 2018-01-25 00:39:01 | [diff] [blame] | 161 | return std::make_unique<InterceptNavigationThrottle>( |
Ken Rockot | ae24ce9 | 2019-12-19 20:00:25 | [diff] [blame] | 162 | handle, base::BindRepeating(&CheckIfShouldIgnoreNavigationOnUIThread), |
| 163 | mode); |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 164 | } |
| 165 | |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 166 | InterceptNavigationDelegate::InterceptNavigationDelegate( |
Colin Blundell | 4695e814 | 2020-03-16 11:13:12 | [diff] [blame] | 167 | JNIEnv* env, |
| 168 | jobject jdelegate, |
| 169 | bool escape_external_handler_value) |
| 170 | : weak_jdelegate_(env, jdelegate), |
| 171 | escape_external_handler_value_(escape_external_handler_value) {} |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 172 | |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 173 | InterceptNavigationDelegate::~InterceptNavigationDelegate() = default; |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 174 | |
| 175 | bool InterceptNavigationDelegate::ShouldIgnoreNavigation( |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 176 | content::NavigationHandle* navigation_handle) { |
| 177 | GURL escaped_url = escape_external_handler_value_ |
Ryan Hamilton | 7f3bd3d | 2022-04-23 00:07:39 | [diff] [blame] | 178 | ? GURL(base::EscapeExternalHandlerValue( |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 179 | navigation_handle->GetURL().spec())) |
| 180 | : navigation_handle->GetURL(); |
Colin Blundell | 4695e814 | 2020-03-16 11:13:12 | [diff] [blame] | 181 | |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 182 | if (!escaped_url.is_valid()) |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 183 | return false; |
| 184 | |
| 185 | JNIEnv* env = base::android::AttachCurrentThread(); |
| 186 | ScopedJavaLocalRef<jobject> jdelegate = weak_jdelegate_.get(env); |
| 187 | |
| 188 | if (jdelegate.is_null()) |
| 189 | return false; |
| 190 | |
Michael Thiessen | 884e617 | 2023-02-13 17:50:31 | [diff] [blame^] | 191 | // Only main frame navigations use this path, so we only need to check if the |
| 192 | // navigation is cross-frame to the main frame. |
| 193 | bool cross_frame = navigation_handle->GetInitiatorFrameToken() && |
| 194 | navigation_handle->GetInitiatorFrameToken() != |
| 195 | navigation_handle->GetWebContents() |
| 196 | ->GetPrimaryMainFrame() |
| 197 | ->GetFrameToken(); |
| 198 | |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 199 | return Java_InterceptNavigationDelegate_shouldIgnoreNavigation( |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 200 | env, jdelegate, navigation_handle->GetJavaNavigationHandle(), |
Michael Thiessen | 884e617 | 2023-02-13 17:50:31 | [diff] [blame^] | 201 | url::GURLAndroid::FromNativeGURL(env, escaped_url), cross_frame); |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 202 | } |
| 203 | |
Michael Thiessen | 7cb129e | 2022-11-08 17:24:51 | [diff] [blame] | 204 | void InterceptNavigationDelegate::HandleSubframeExternalProtocol( |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 205 | const GURL& url, |
| 206 | ui::PageTransition page_transition, |
| 207 | bool has_user_gesture, |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 208 | const absl::optional<url::Origin>& initiating_origin, |
| 209 | mojo::PendingRemote<network::mojom::URLLoaderFactory>* out_factory) { |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 210 | // If there's a pending async subframe action, don't consider external |
| 211 | // navigation for the current navigation. |
| 212 | if (subframe_redirect_url_ || url_loader_) { |
| 213 | return; |
| 214 | } |
| 215 | |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 216 | GURL escaped_url = escape_external_handler_value_ |
Ryan Hamilton | 7f3bd3d | 2022-04-23 00:07:39 | [diff] [blame] | 217 | ? GURL(base::EscapeExternalHandlerValue(url.spec())) |
Michael Thiessen | ca245a38 | 2022-02-21 16:11:17 | [diff] [blame] | 218 | : url; |
| 219 | if (!escaped_url.is_valid()) |
| 220 | return; |
| 221 | |
| 222 | JNIEnv* env = base::android::AttachCurrentThread(); |
| 223 | ScopedJavaLocalRef<jobject> jdelegate = weak_jdelegate_.get(env); |
| 224 | |
| 225 | if (jdelegate.is_null()) |
| 226 | return; |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 227 | ScopedJavaLocalRef<jobject> j_gurl = |
| 228 | Java_InterceptNavigationDelegate_handleSubframeExternalProtocol( |
| 229 | env, jdelegate, url::GURLAndroid::FromNativeGURL(env, escaped_url), |
| 230 | page_transition, has_user_gesture, |
| 231 | initiating_origin ? initiating_origin->CreateJavaObject() : nullptr); |
| 232 | if (j_gurl.is_null()) |
| 233 | return; |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 234 | subframe_redirect_url_ = url::GURLAndroid::ToNativeGURL(env, j_gurl); |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 235 | |
| 236 | mojo::PendingReceiver<network::mojom::URLLoaderFactory> receiver = |
| 237 | out_factory->InitWithNewPipeAndPassReceiver(); |
| 238 | scoped_refptr<network::SharedURLLoaderFactory> loader_factory = |
| 239 | base::MakeRefCounted<network::SingleRequestURLLoaderFactory>( |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 240 | base::BindOnce(&InterceptNavigationDelegate::LoaderCallback, |
| 241 | weak_ptr_factory_.GetWeakPtr())); |
Michael Thiessen | 1a49e4d5 | 2022-12-02 21:54:40 | [diff] [blame] | 242 | loader_factory->Clone(std::move(receiver)); |
[email protected] | 4360ae7 | 2012-10-09 22:10:46 | [diff] [blame] | 243 | } |
| 244 | |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 245 | void InterceptNavigationDelegate::LoaderCallback( |
| 246 | const network::ResourceRequest& resource_request, |
| 247 | mojo::PendingReceiver<network::mojom::URLLoader> pending_receiver, |
| 248 | mojo::PendingRemote<network::mojom::URLLoaderClient> pending_client) { |
| 249 | url_loader_ = mojo::MakeSelfOwnedReceiver( |
| 250 | std::make_unique<RedirectURLLoader>(resource_request, |
| 251 | std::move(pending_client)), |
| 252 | std::move(pending_receiver)); |
| 253 | MaybeHandleSubframeAction(); |
| 254 | } |
| 255 | |
| 256 | void InterceptNavigationDelegate::MaybeHandleSubframeAction() { |
| 257 | // An empty subframe_redirect_url_ implies a pending async action. |
| 258 | if (!url_loader_ || |
| 259 | (subframe_redirect_url_ && subframe_redirect_url_->is_empty())) { |
| 260 | return; |
| 261 | } |
| 262 | RedirectURLLoader* loader = |
| 263 | static_cast<RedirectURLLoader*>(url_loader_->impl()); |
| 264 | if (!subframe_redirect_url_) { |
| 265 | loader->OnNonRedirectAsyncAction(); |
| 266 | } else { |
| 267 | loader->DoRedirect(std::move(subframe_redirect_url_)); |
| 268 | } |
| 269 | url_loader_.reset(); |
| 270 | } |
| 271 | |
Michael Thiessen | 332dadb6 | 2022-07-13 14:44:07 | [diff] [blame] | 272 | void InterceptNavigationDelegate::OnResourceRequestWithGesture() { |
| 273 | JNIEnv* env = base::android::AttachCurrentThread(); |
| 274 | ScopedJavaLocalRef<jobject> jdelegate = weak_jdelegate_.get(env); |
| 275 | if (jdelegate.is_null()) |
| 276 | return; |
| 277 | Java_InterceptNavigationDelegate_onResourceRequestWithGesture(env, jdelegate); |
Michael Thiessen | e566352 | 2022-05-25 21:23:28 | [diff] [blame] | 278 | } |
| 279 | |
Michael Thiessen | 7b53117 | 2023-01-28 05:25:59 | [diff] [blame] | 280 | void InterceptNavigationDelegate::OnSubframeAsyncActionTaken( |
| 281 | JNIEnv* env, |
| 282 | const base::android::JavaParamRef<jobject>& j_gurl) { |
| 283 | // subframe_redirect_url_ no longer empty indicates the async action has been |
| 284 | // taken. |
| 285 | subframe_redirect_url_ = |
| 286 | j_gurl.is_null() ? nullptr : url::GURLAndroid::ToNativeGURL(env, j_gurl); |
| 287 | MaybeHandleSubframeAction(); |
| 288 | } |
| 289 | |
[email protected] | 8812e3d0 | 2013-05-22 12:38:53 | [diff] [blame] | 290 | } // namespace navigation_interception |