Changeset 83252 in webkit

Timestamp:

Apr 7, 2011 9:59:33 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2011-04-07 Magnus Danielsson <​public@fuzzac.com>

Reviewed by Darin Fisher.

[chromium] WebPageSerializerImpl doesn't serialize sub-frames correctly
​https://bugs.webkit.org/show_bug.cgi?id=53897

When serializing a web page using 'save page as', sub-frames and resources gets
saved in a sub-directory. However, frame elements didn't get updated to reference
these saved sub-frames, but were still referencing the original url. So when opening
a saved web page, any sub-frames would get pulled in from the original url rather than
what was saved.

In addition to this, sub-frames in the sub-directory erroneously had the name of the
sub-directory prepended to the path of resources located in the same sub-directory.

• src/WebPageSerializerImpl.cpp: (WebKit::WebPageSerializerImpl::openTagToStrne: Fixed resource paths in sub-frames. Also made sure sub-frames are referenced correctly from parent frame. (WebKit::WebPageSerializerImpl::endTagToString): Removed constness from argument. (WebKit::WebPageSerializerImpl::buildContentForNode): Ditto.
• src/WebPageSerializerImpl.h:

Location:

trunk/Source/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/WebPageSerializerImpl.cpp (modified) (5 diffs)
• src/WebPageSerializerImpl.h (modified) (1 diff)

trunk/Source/WebKit/chromium/ChangeLog

@@ +1 @@
+2011-04-07  Magnus Danielsson  <public@fuzzac.com>
+
+        Reviewed by Darin Fisher.
+
+        [chromium] WebPageSerializerImpl doesn't serialize sub-frames correctly
+        https://bugs.webkit.org/show_bug.cgi?id=53897
+
+        When serializing a web page using 'save page as', sub-frames and resources gets
+        saved in a sub-directory. However, frame elements didn't get updated to reference
+        these saved sub-frames, but were still referencing the original url. So when opening
+        a saved web page, any sub-frames would get pulled in from the original url rather than 
+        what was saved.
+
+        In addition to this, sub-frames in the sub-directory erroneously had the name of the
+        sub-directory prepended to the path of resources located in the same sub-directory.
+
+        * src/WebPageSerializerImpl.cpp:
+        (WebKit::WebPageSerializerImpl::openTagToStrne: Fixed resource paths in sub-frames. 
+        Also made sure sub-frames are referenced correctly from parent frame.
+        (WebKit::WebPageSerializerImpl::endTagToString): Removed constness from argument.
+        (WebKit::WebPageSerializerImpl::buildContentForNode): Ditto.
+        * src/WebPageSerializerImpl.h:
+
 2011-04-07  Nat Duca  <nduca@chromium.org>
 

trunk/Source/WebKit/chromium/src/WebPageSerializerImpl.cpp

@@ -296 +296 @@
 }
 
-void WebPageSerializerImpl::openTagToString(const Element* element,
+void WebPageSerializerImpl::openTagToString(Element* element,
                                             SerializeDomParam* param)
 {
@@ -329 +329 @@
                     else {
                         // Get the absolute link
-                        String completeURL = param->document->completeURL(attrValue);
+                        WebFrameImpl* subFrame = WebFrameImpl::fromFrameOwnerElement(element);
+                        String completeURL = subFrame ? subFrame->frame()->document()->url() : 
+                                                        param->document->completeURL(attrValue);
                         // Check whether we have local files for those link.
                         if (m_localLinks.contains(completeURL)) {
-                            if (!m_localDirectoryName.isEmpty())
-                                result += "./" + m_localDirectoryName + "/";
+                            if (!param->directoryName.isEmpty())
+                                result += "./" + param->directoryName + "/";
                             result += m_localLinks.get(completeURL);
                         } else
@@ -361 +363 @@
 
 // Serialize end tag of an specified element.
-void WebPageSerializerImpl::endTagToString(const Element* element,
+void WebPageSerializerImpl::endTagToString(Element* element,
                                            SerializeDomParam* param)
 {
@@ -398 +400 @@
 }
 
-void WebPageSerializerImpl::buildContentForNode(const Node* node,
+void WebPageSerializerImpl::buildContentForNode(Node* node,
                                                 SerializeDomParam* param)
 {
@@ -404 +406 @@
     case Node::ELEMENT_NODE:
         // Process open tag of element.
-        openTagToString(static_cast<const Element*>(node), param);
+        openTagToString(static_cast<Element*>(node), param);
         // Walk through the children nodes and process it.
-        for (const Node *child = node->firstChild(); child; child = child->nextSibling())
+        for (Node *child = node->firstChild(); child; child = child->nextSibling())
             buildContentForNode(child, param);
         // Process end tag of element.
-        endTagToString(static_cast<const Element*>(node), param);
+        endTagToString(static_cast<Element*>(node), param);
         break;
     case Node::TEXT_NODE:

trunk/Source/WebKit/chromium/src/WebPageSerializerImpl.h

@@ -175 +175 @@
                               FlushOption);
     // Serialize open tag of an specified element.
-    void openTagToString(const WebCore::Element* element,
+    void openTagToString(WebCore::Element*,
                          SerializeDomParam* param);
     // Serialize end tag of an specified element.
-    void endTagToString(const WebCore::Element* element,
+    void endTagToString(WebCore::Element*,
                         SerializeDomParam* param);
     // Build content for a specified node
-    void buildContentForNode(const WebCore::Node* node,
+    void buildContentForNode(WebCore::Node*,
                              SerializeDomParam* param);
 };