Move multipart resource handling to core/fetch (2/2)

components/page_load_metrics/renderer/metrics_render_frame_observer.cc

Index: components/page_load_metrics/renderer/metrics_render_frame_observer.cc
diff --git a/components/page_load_metrics/renderer/metrics_render_frame_observer.cc b/components/page_load_metrics/renderer/metrics_render_frame_observer.cc
index 16256113ecae794b2c0200d41c100799ea02ea18..20f4d8a84f6834a35c33fe94138cea9e932d8d8b 100644
--- a/components/page_load_metrics/renderer/metrics_render_frame_observer.cc
+++ b/components/page_load_metrics/renderer/metrics_render_frame_observer.cc
@@ -87,9 +87,6 @@ bool MetricsRenderFrameObserver::ShouldSendMetrics() const {
     return false;
 
   const blink::WebURLResponse& url_response = frame->dataSource()->response();
-  // Ignore multipart responses (e.g. MHTML).

[Charlie Harrison, 2016/03/07 20:07:05]

Can you explain why this check isn't needed (along with the check in
page_load_histograms)?

[yhirano, 2016/03/07 20:18:27]

Previously isMultipartPayload is set only for multipart/x-replace-image
responses[1] and I'm removing the flag now. IIUC MHTML is not related to this
flag.

For multipart/x-replace-image responses the function should return false based
on the mime type as implemented below.

1:
https://code.google.com/p/chromium/codesearch#chromium/src/content/child/mult...

-  if (url_response.isMultipartPayload())
-    return false;
 
   // Ignore non-HTML documents (e.g. SVG). Note that images are treated by
   // Blink as HTML documents, so to exclude images, we must perform