Making PageSerializer operate on single frames only.

third_party/WebKit/Source/core/page/PageSerializer.h

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 40 matching lines @@
 class Document;
 class LocalFrame;
 class Page;
 class LayoutObject;
 class Resource;
 class SharedBuffer;
 class StylePropertySet;
 
 struct SerializedResource;
 
-// 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.
+// This class is used to serialize frame's contents back to text (typically HTML).
+// It serializes frame's document and resources such as images and CSS stylesheets.
+// TODO(lukasza): Rename this class to FrameSerializer.
 class CORE_EXPORT PageSerializer final {
     STACK_ALLOCATED();
 public:
     class Delegate {
         USING_FAST_MALLOC(Delegate);
     public:
         virtual ~Delegate() { }
         virtual bool shouldIgnoreAttribute(const Attribute&) = 0;
     };
 
     PageSerializer(Vector<SerializedResource>*, PassOwnPtr<Delegate>);
 
-    // 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
-    // vector is the top frame serialized content.
-    void serialize(Page*);
+    // Initiates the serialization of the frame. All serialized content and
+    // retrieved resources are added to the Vector passed to the constructor.
+    // The first resource in that vector is the frame's serialized content.
+    // Subsequent resources are images, css, etc.
+    void serializeFrame(LocalFrame&);

[tkent, 2015/11/18 22:32:55]

nit: Can we change |LocalFrame&| to |const LocalFrame&|?

[Łukasz Anforowicz, 2015/11/18 23:27:41]

Done (constness cascaded into urlForBlankFrame and BlankFrameURLMap)

I also changed parameter of urlForBlankFrame from pointer to a reference (since
nullptr won't ever get passed there from the 2 callsites).

 
     void registerRewriteURL(const String& from, const String& to);
     void setRewriteURLFolder(const String&);
 
     KURL urlForBlankFrame(LocalFrame*);
 
     Delegate* delegate();
 
     static String markOfTheWebDeclaration(const KURL&);
 
 private:
-    void serializeFrame(LocalFrame*);
-
     // Serializes the stylesheet back to text and adds it to the resources if URL is not-empty.
     // It also adds any resources included in that stylesheet (including any imported stylesheets and their own resources).
     void serializeCSSStyleSheet(CSSStyleSheet&, const KURL&);
 
     // Serializes the css rule (including any imported stylesheets), adding referenced resources.
     void serializeCSSRule(CSSRule*);
 
     bool shouldAddURL(const KURL&);
 
     void addToResources(Resource *, PassRefPtr<SharedBuffer>, const KURL&);
@@ skipping 11 matching lines @@
     HashMap<String, String> m_rewriteURLs;
     String m_rewriteFolder;
     unsigned m_blankFrameCounter;
 
     OwnPtr<Delegate> m_delegate;
 };
 
 } // namespace blink
 
 #endif // PageSerializer_h