Implement a ResourceThrottle for URL overriding in Chrome on Android.

chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h"
 
 #include <string>
 
 #include "base/base64.h"
 #include "base/logging.h"
@@ skipping 25 matching lines @@
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/resource_context.h"
 #include "content/public/browser/resource_dispatcher_host.h"
 #include "content/public/browser/resource_request_info.h"
 #include "net/base/load_flags.h"
 #include "net/base/ssl_config_service.h"
 #include "net/url_request/url_request.h"
 #include "third_party/protobuf/src/google/protobuf/repeated_field.h"
 
 // TODO(oshima): Enable this for other platforms.

[joth, 2012/06/13 22:32:49]

nit: either move this TODO inside the #if, or split out the two sections to be
independent #if blocks.

[mkosiba (inactive), 2012/06/15 11:05:36]

Done.

 #if defined(OS_CHROMEOS)
 #include "chrome/browser/renderer_host/offline_resource_throttle.h"
+#elif defined(OS_ANDROID)
+#include "content/public/browser/intercept_navigation_resource_throttle.h"
 #endif
 
 using content::BrowserThread;
 using content::RenderViewHost;
 using content::ResourceDispatcherHostLoginDelegate;
 using content::ResourceRequestInfo;
 
 namespace {
 
 void NotifyDownloadInitiatedOnUI(int render_process_id, int render_view_id) {
@@ skipping 66 matching lines @@
   if (is_continuation_of_transferred_request)
     ChromeURLRequestUserData::Delete(request);
 
   ChromeURLRequestUserData* user_data =
       ChromeURLRequestUserData::Create(request);
   if (prerender_tracker_->IsPrerenderingOnIOThread(child_id, route_id)) {
     user_data->set_is_prerender(true);
     request->set_priority(net::IDLE);
   }
 
+  if (resource_type == ResourceType::MAIN_FRAME) {
 #if defined(OS_CHROMEOS)
-  if (resource_type == ResourceType::MAIN_FRAME) {
     // We check offline first, then check safe browsing so that we still can
     // block unsafe site after we remove offline page.
     throttles->push_back(new OfflineResourceThrottle(
         child_id, route_id, request, resource_context));
+#elif defined(OS_ANDROID)

[joth, 2012/06/13 22:32:49]

again would be tempted to do these as unconnected #if , unless there's a logical
reason we could never have both throttles together?

[mkosiba (inactive), 2012/06/15 11:05:36]

No logical reason to not have the 2 throttles together. It does however make
sense to have an elif since OS_CHROMEOS and OS_ANDROID won't be declared at the
same time.

+    throttles->push_back(
+        content::InterceptNavigationResourceThrottle::Create(request));
+#endif
   }
-#endif
 
   AppendChromeMetricsHeaders(request, resource_context, resource_type);
 
   AppendStandardResourceThrottles(request,
                                   resource_context,
                                   child_id,
                                   route_id,
                                   resource_type,
                                   throttles);
 }
@@ skipping 257 matching lines @@
   if (base::Base64Encode(serialized, &hashed)) {
     // If successful, swap the header value with the new one.
     // Note that the list of IDs and the header could be temporarily out of sync
     // if IDs are added as we are recreating the header, but we're OK with those
     // descrepancies.
     variation_ids_header_ = hashed;
   } else {
     DVLOG(1) << "Failed to base64 encode Variation IDs value: " << serialized;
   }
 }