Update implementation following changes to URLRequest cache mode.

mojo/services/network/url_loader_impl.cc

Index: mojo/services/network/url_loader_impl.cc
diff --git a/mojo/services/network/url_loader_impl.cc b/mojo/services/network/url_loader_impl.cc
index 955a49c2e958484272fd7896de24912b26b723d1..1f372f3895b62945537f056f9cd41e22354da121 100644
--- a/mojo/services/network/url_loader_impl.cc
+++ b/mojo/services/network/url_loader_impl.cc
@@ -537,10 +537,15 @@ void URLLoaderImpl::StartInternal(URLRequestPtr request) {
         new net::ElementsUploadDataStream(element_readers.Pass(), 0)));
   }
   int load_flags = 0;
-  if (request->bypass_cache)
-    load_flags |= net::LOAD_BYPASS_CACHE;
-  if (request->only_from_cache)
-    load_flags |= net::LOAD_ONLY_FROM_CACHE;
+  switch (request->cache_mode) {
+    case URLRequest::CACHE_MODE_DEFAULT:
+      break;
+    case URLRequest::CACHE_MODE_BYPASS_CACHE:
+      load_flags |= net::LOAD_BYPASS_CACHE;
+    case URLRequest::CACHE_MODE_ONLY_FROM_CACHE:
+      load_flags |= net::LOAD_ONLY_FROM_CACHE;
+      break;
+  }

[viettrungluu, 2015/07/20 17:00:28]

Since we don't validate enums, you should have a default case (which presumably
treats it as an error, probably by deleting this, I think).

[qsr, 2015/07/20 18:30:50]

Hum... I'm kind of torn here. If I don't do this, I will get an error when
trying to update the network service.

Would you mind:
load_flags = -1;
swicth() {
...
}
if (load_flags == -1) { delete this; }
?

   if (load_flags)
     url_request_->SetLoadFlags(load_flags);