Title: [92883] trunk/Source/WebCore
- Revision
- 92883
- Author
- [email protected]
- Date
- 2011-08-11 15:02:33 -0700 (Thu, 11 Aug 2011)
Log Message
The PageSerializer should always use absolute URL
https://bugs.webkit.org/show_bug.cgi?id=66006
Patch by Benjamin Poulain <[email protected]> on 2011-08-11
Reviewed by Adam Barth.
Page serializer resolves every URL for storing the resources.
The MarkupAccumulator used there should use the same rules in order to
create a valid file.
No new tests because the test infrastructure for this
does not exist with layout tests and the Chromium unit test for
the serializer are disabled.
* page/PageSerializer.cpp:
(WebCore::SerializerMarkupAccumulator::SerializerMarkupAccumulator):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (92882 => 92883)
--- trunk/Source/WebCore/ChangeLog 2011-08-11 21:59:11 UTC (rev 92882)
+++ trunk/Source/WebCore/ChangeLog 2011-08-11 22:02:33 UTC (rev 92883)
@@ -1,3 +1,21 @@
+2011-08-11 Benjamin Poulain <[email protected]>
+
+ The PageSerializer should always use absolute URL
+ https://bugs.webkit.org/show_bug.cgi?id=66006
+
+ Reviewed by Adam Barth.
+
+ Page serializer resolves every URL for storing the resources.
+ The MarkupAccumulator used there should use the same rules in order to
+ create a valid file.
+
+ No new tests because the test infrastructure for this
+ does not exist with layout tests and the Chromium unit test for
+ the serializer are disabled.
+
+ * page/PageSerializer.cpp:
+ (WebCore::SerializerMarkupAccumulator::SerializerMarkupAccumulator):
+
2011-08-01 Adrienne Walker <[email protected]>
Enable fast path scrolling for composited iframes
Modified: trunk/Source/WebCore/page/PageSerializer.cpp (92882 => 92883)
--- trunk/Source/WebCore/page/PageSerializer.cpp 2011-08-11 21:59:11 UTC (rev 92882)
+++ trunk/Source/WebCore/page/PageSerializer.cpp 2011-08-11 22:02:33 UTC (rev 92883)
@@ -106,9 +106,8 @@
Document* m_document;
};
-// FIXME: should that really use ResolveNonLocalURLs?
SerializerMarkupAccumulator::SerializerMarkupAccumulator(PageSerializer* serializer, Document* document, Vector<Node*>* nodes)
- : MarkupAccumulator(nodes, ResolveNonLocalURLs)
+ : MarkupAccumulator(nodes, ResolveAllURLs)
, m_serializer(serializer)
, m_document(document)
{
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
