Implement offline mode behind a flag

content/browser/loader/resource_dispatcher_host_impl.h

Index: content/browser/loader/resource_dispatcher_host_impl.h
diff --git a/content/browser/loader/resource_dispatcher_host_impl.h b/content/browser/loader/resource_dispatcher_host_impl.h
index 5dc36e643526406d43167a57b5fa4ce32b0ae6e0..90bed401397c7f5c8bae0192105d31e29c9a63b1 100644
--- a/content/browser/loader/resource_dispatcher_host_impl.h
+++ b/content/browser/loader/resource_dispatcher_host_impl.h
@@ -25,6 +25,7 @@
 #include "base/time.h"
 #include "base/timer.h"
 #include "content/browser/download/download_resource_handler.h"
+#include "content/browser/loader/offline_policy.h"
 #include "content/browser/loader/render_view_host_tracker.h"
 #include "content/browser/loader/resource_loader.h"
 #include "content/browser/loader/resource_loader_delegate.h"
@@ -237,6 +238,8 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
   FRIEND_TEST_ALL_PREFIXES(ResourceDispatcherHostTest,
                            CalculateApproximateMemoryCost);
 
+  typedef std::pair<int, int> ProcessRouteIDs;

[darin (slow to review), 2013/04/11 20:01:23]

nit: maybe call this GlobalRoutingID?  maybe it should be an actual struct since
"first" and "second" are really terrible member variable names.

[Randy Smith (Not in Mondays), 2013/04/12 14:06:44]

Happy to take your guidance on this; I was just copying (moving) what was
already present in this file.  Done.

+
   class ShutdownTask;
 
   friend class ShutdownTask;
@@ -409,7 +412,6 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
   bool is_shutdown_;
 
   typedef std::vector<linked_ptr<ResourceLoader> > BlockedLoadersList;
-  typedef std::pair<int, int> ProcessRouteIDs;
   typedef std::map<ProcessRouteIDs, BlockedLoadersList*> BlockedLoadersMap;
   BlockedLoadersMap blocked_loaders_map_;
 
@@ -452,6 +454,10 @@ class CONTENT_EXPORT ResourceDispatcherHostImpl
 
   RenderViewHostTracker tracker_;  // Lives on UI thread.
 
+  typedef std::map<ProcessRouteIDs, OfflinePolicy*> OfflineMap;
+
+  OfflineMap offline_map_;

[darin (slow to review), 2013/04/11 20:01:23]

nit: I think it'd read better in the .cc file if this variable were named
offline_policy_map_.

[Randy Smith (Not in Mondays), 2013/04/12 14:06:44]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(ResourceDispatcherHostImpl);
 };