Save-Page-As-Complete-HTML: Better handling of <object> elements.

content/renderer/savable_resources.cc

 // Copyright (c) 2013 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 "content/renderer/savable_resources.h"
 
 #include <set>
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
@@ skipping 111 matching lines @@
   } else if (element.hasHTMLTagName("body") ||
              element.hasHTMLTagName("table") ||
              element.hasHTMLTagName("tr") ||
              element.hasHTMLTagName("td")) {
     attribute_name = "background";
   } else if (element.hasHTMLTagName("blockquote") ||
              element.hasHTMLTagName("q") ||
              element.hasHTMLTagName("del") ||
              element.hasHTMLTagName("ins")) {
     attribute_name = "cite";
+  } else if (element.hasHTMLTagName("object")) {
+    // TODO(lukasza): When <object> contains a html document, it should be
+    // reported as a subframe, not as a savable resource (reporting as a
+    // savable resource works, but will save original html contents, not
+    // current html contents of the frame).
+    attribute_name = "data";
   } else if (element.hasHTMLTagName("link")) {
     // If the link element is not linked to css, ignore it.
     if (base::LowerCaseEqualsASCII(
             base::StringPiece16(element.getAttribute("type")), "text/css") ||
         base::LowerCaseEqualsASCII(
             base::StringPiece16(element.getAttribute("rel")), "stylesheet")) {
       // TODO(jnd): Add support for extracting links of sub-resources which
       // are inside style-sheet such as @import, url(), etc.
       // See bug: http://b/issue?id=1111667.
       attribute_name = "href";
     }
   }
   if (!attribute_name)
     return WebString();
   WebString value = element.getAttribute(WebString::fromUTF8(attribute_name));
   // If value has content and not start with "javascript:" then return it,
   // otherwise return NULL.
   if (!value.isNull() && !value.isEmpty() &&
       !base::StartsWith(value.utf8(), "javascript:",
                         base::CompareCase::INSENSITIVE_ASCII))
     return value;
 
   return WebString();
 }
 
 }  // namespace content