Ensure loading PDFs doesn't generate 2 http requests

extensions/renderer/guest_view/mime_handler_view/mime_handler_view_container.cc

 // Copyright 2014 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 "extensions/renderer/guest_view/mime_handler_view/mime_handler_view_container.h"
 
 #include <map>
 #include <set>
 
 #include "base/macros.h"
@@ skipping 99 matching lines @@
       original_url_(original_url),
       guest_proxy_routing_id_(-1),
       guest_loaded_(false),
       weak_factory_(this) {
   DCHECK(!mime_type_.empty());
   is_embedded_ = !render_frame->GetWebFrame()->document().isPluginDocument();
   g_mime_handler_view_container_map.Get()[render_frame].insert(this);
 }
 
 MimeHandlerViewContainer::~MimeHandlerViewContainer() {
-  if (loader_)
+  if (loader_) {
+    DCHECK(is_embedded_);
     loader_->cancel();
+  }
 
   if (render_frame()) {
     g_mime_handler_view_container_map.Get()[render_frame()].erase(this);
     if (g_mime_handler_view_container_map.Get()[render_frame()].empty())
       g_mime_handler_view_container_map.Get().erase(render_frame());
   }
 }
 
 // static
 std::vector<MimeHandlerViewContainer*>
 MimeHandlerViewContainer::FromRenderFrame(content::RenderFrame* render_frame) {
   auto it = g_mime_handler_view_container_map.Get().find(render_frame);
   if (it == g_mime_handler_view_container_map.Get().end())
     return std::vector<MimeHandlerViewContainer*>();
 
   return std::vector<MimeHandlerViewContainer*>(it->second.begin(),
                                                 it->second.end());
 }
 
 void MimeHandlerViewContainer::OnReady() {
-  if (!render_frame())
+  if (!render_frame() || !is_embedded_)
     return;
 
   blink::WebFrame* frame = render_frame()->GetWebFrame();
   blink::WebURLLoaderOptions options;
   // The embedded plugin is allowed to be cross-origin and we should always
   // send credentials/cookies with the request.
   options.crossOriginRequestPolicy =
       blink::WebURLLoaderOptions::CrossOriginRequestPolicyAllow;
   options.allowCredentials = true;
   DCHECK(!loader_);
@@ skipping 165 matching lines @@
   v8::HandleScope handle_scope(isolate);
   v8::Context::Scope context_scope(frame->mainWorldScriptContext());
   for (const auto& pending_message : pending_messages_)
     PostMessage(isolate, v8::Local<v8::Value>::New(isolate, *pending_message));
 
   pending_messages_.clear();
 }
 
 void MimeHandlerViewContainer::CreateMimeHandlerViewGuest() {
   // The loader has completed loading |view_id_| so we can dispose it.
-  loader_.reset();
+  if (loader_) {
+    DCHECK(is_embedded_);
+    loader_.reset();
+  }
 
   DCHECK_NE(element_instance_id(), guest_view::kInstanceIDNone);
 
   if (!render_frame())
     return;
 
   render_frame()->Send(
       new ExtensionsGuestViewHostMsg_CreateMimeHandlerViewGuest(
           render_frame()->GetRoutingID(), view_id_, element_instance_id(),
           element_size_));
 }
 
 }  // namespace extensions