Remove variable shadowing in blink/modules/cache_storage

In an effort to reduce or even ban variable shadowing, this renames
a variable to avoid such shadowing. I'm interested in prohibiting
shadowing because I think it might prevent potential jumbo problems.

The exact error this avoid is:

third_party/blink/renderer/modules/cache_storage/inspector_cache_storage_agent.cc:313:22: error: declaration shadows a local variable [-Werror,-Wshadow]
    for (const auto& request_response : responses_) {
                     ^
third_party/blink/renderer/modules/cache_storage/inspector_cache_storage_agent.cc:277:22: note: previous declaration is here
    RequestResponse& request_response =

Bug: 923510
Change-Id: I8f6b6cd11f80a254c2c926e934dacefaa6dfccf8
Reviewed-on: https://chromium-review.googlesource.com/c/1477285
Reviewed-by: Ben Kelly <wanderview@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#633715}
diff --git a/third_party/blink/renderer/modules/cache_storage/inspector_cache_storage_agent.cc b/third_party/blink/renderer/modules/cache_storage/inspector_cache_storage_agent.cc
index 5bb674d..e4ff216 100644
--- a/third_party/blink/renderer/modules/cache_storage/inspector_cache_storage_agent.cc
+++ b/third_party/blink/renderer/modules/cache_storage/inspector_cache_storage_agent.cc
@@ -276,23 +276,23 @@
       const mojom::blink::FetchAPIRequestPtr& request,
       const mojom::blink::FetchAPIResponsePtr& response) {
     DCHECK_GT(num_responses_left_, 0);
-    RequestResponse& request_response =
+    RequestResponse& next_request_response =
         responses_.at(responses_.size() - num_responses_left_);
 
-    request_response.request_url = request->url.GetString();
-    request_response.request_method = request->method;
+    next_request_response.request_url = request->url.GetString();
+    next_request_response.request_method = request->method;
     for (const auto& header : request->headers) {
-      request_response.request_headers.Set(AtomicString(header.key),
-                                           AtomicString(header.value));
+      next_request_response.request_headers.Set(AtomicString(header.key),
+                                                AtomicString(header.value));
     }
 
-    request_response.response_status = response->status_code;
-    request_response.response_status_text = response->status_text;
-    request_response.response_time = response->response_time.ToDoubleT();
-    request_response.response_type = response->response_type;
+    next_request_response.response_status = response->status_code;
+    next_request_response.response_status_text = response->status_text;
+    next_request_response.response_time = response->response_time.ToDoubleT();
+    next_request_response.response_type = response->response_type;
     for (const auto& header : response->headers) {
-      request_response.response_headers.Set(AtomicString(header.key),
-                                            AtomicString(header.value));
+      next_request_response.response_headers.Set(AtomicString(header.key),
+                                                 AtomicString(header.value));
     }
 
     if (--num_responses_left_ != 0)