Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(824)

Unified Diff: Source/core/page/PageSerializer.h

Issue 68613003: Merges the two different page serializers (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/page/PageSerializer.h
diff --git a/Source/core/page/PageSerializer.h b/Source/core/page/PageSerializer.h
index 93a4dc10677b93a545968cfbb8d3209855e8e620..0c3af23c740c181646eaec5486492b315c016a31 100644
--- a/Source/core/page/PageSerializer.h
+++ b/Source/core/page/PageSerializer.h
@@ -55,11 +55,15 @@ class StyleRuleFontFace;
struct SerializedResource;
+// This hash map is used to map resource URL of original link to its local
+// file path.
+typedef HashMap<WTF::String, WTF::String> LinkLocalPathMap;
abarth-chromium 2013/11/14 16:55:03 WTF::String -> String. There's no need to use the
+
// This class is used to serialize a page contents back to text (typically HTML).
// It serializes all the page frames and retrieves resources such as images and CSS stylesheets.
class PageSerializer {
public:
- explicit PageSerializer(Vector<SerializedResource>*);
+ explicit PageSerializer(Vector<SerializedResource>*, HashMap<WTF::String, WTF::String>* = 0, WTF::String = "");
abarth-chromium 2013/11/14 16:55:03 Why not use the LinkLocalPathMap type here? No ne
// Initiates the serialization of the frame's page. All serialized content and retrieved
// resources are added to the Vector passed to the constructor. The first resource in that
@@ -87,6 +91,8 @@ private:
Vector<SerializedResource>* m_resources;
ListHashSet<KURL> m_resourceURLs;
HashMap<Frame*, KURL> m_blankFrameURLs;
+ LinkLocalPathMap* m_URLs;
+ WTF::String m_directory;
unsigned m_blankFrameCounter;
};

Powered by Google App Engine
This is Rietveld 408576698