[ImageFetcher] Return http response headers

components/image_fetcher/core/request_metadata.cc

 // Copyright 2017 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 "components/image_fetcher/core/request_metadata.h"
 
 namespace image_fetcher {
 
 RequestMetadata::RequestMetadata()
-    : http_response_code(RESPONSE_CODE_INVALID), from_http_cache(false) {}
+    : http_response_code(RESPONSE_CODE_INVALID),
+      from_http_cache(false),
+      http_response_headers(nullptr) {}
 
 bool operator==(const RequestMetadata& lhs, const RequestMetadata& rhs) {
+  // Do not compare |http_response_headers| as this is solely used in

[markusheintz_, 2017/04/04 13:33:36]

I don't understand this comment. Do we really only add the headers for the
tests?

[jkrcal, 2017/04/04 16:42:02]

I rephrased the comment.

[markusheintz_, 2017/04/06 08:52:03]

Oh so the equals operator is only used in test! I guess that's hard to ensure
for the future since this is not obvious to user. Let's quickly sync on this in
person

[jkrcal, 2017/04/06 14:18:23]

Okay, I added the check also for the headers.

+  // unit-tests. The pointer to headers cannot be expected, easily.
   return lhs.mime_type == rhs.mime_type &&
          lhs.http_response_code == rhs.http_response_code &&
          lhs.from_http_cache == rhs.from_http_cache;
 }
 
 bool operator!=(const RequestMetadata& lhs, const RequestMetadata& rhs) {
   return !(lhs == rhs);
 }
 
 }  // namespace image_fetcher