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

Unified Diff: trunk/src/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

Issue 23597024: Revert 222257 "Revert 222235 ""Redirecting URLs to Packaged Apps..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 3 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: trunk/src/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc
===================================================================
--- trunk/src/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc (revision 222257)
+++ trunk/src/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc (working copy)
@@ -65,6 +65,8 @@
#if defined(OS_ANDROID)
#include "chrome/browser/android/intercept_download_resource_throttle.h"
#include "components/navigation_interception/intercept_navigation_delegate.h"
+#else
+#include "chrome/browser/apps/app_url_redirector.h"
#endif
#if defined(OS_CHROMEOS)
@@ -263,12 +265,23 @@
request->SetPriority(net::IDLE);
}
+ ProfileIOData* io_data = ProfileIOData::FromResourceContext(
+ resource_context);
+
+ if (!is_prerendering && resource_type == ResourceType::MAIN_FRAME) {
#if defined(OS_ANDROID)
- if (!is_prerendering && resource_type == ResourceType::MAIN_FRAME) {
throttles->push_back(
InterceptNavigationDelegate::CreateThrottleFor(request));
+#else
+ // Redirect some navigations to apps that have registered matching URL
+ // handlers ('url_handlers' in the manifest).
+ content::ResourceThrottle* url_to_app_throttle =
+ AppUrlRedirector::MaybeCreateThrottleFor(request, io_data);
+ if (url_to_app_throttle)
+ throttles->push_back(url_to_app_throttle);
+#endif
}
-#endif
+
#if defined(OS_CHROMEOS)
if (resource_type == ResourceType::MAIN_FRAME) {
// We check offline first, then check safe browsing so that we still can
@@ -289,8 +302,6 @@
if (!request->is_pending()) {
net::HttpRequestHeaders headers;
headers.CopyFrom(request->extra_request_headers());
- ProfileIOData* io_data = ProfileIOData::FromResourceContext(
- resource_context);
bool incognito = io_data->is_incognito();
chrome_variations::VariationsHttpHeaderProvider::GetInstance()->
AppendHeaders(request->url(),
@@ -311,7 +322,6 @@
resource_type,
throttles);
- ProfileIOData* io_data = ProfileIOData::FromResourceContext(resource_context);
if (io_data->resource_prefetch_predictor_observer()) {
io_data->resource_prefetch_predictor_observer()->OnRequestStarted(
request, resource_type, child_id, route_id);

Powered by Google App Engine
This is Rietveld 408576698