Adds a feature to MHTML serialization that omits subframes and subresources marked no-store.

third_party/WebKit/Source/core/frame/FrameSerializer.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 67 matching lines @@
         // return true and populate |rewrittenLink| with a desired value of the
         // html attribute value to be used in place of the original link.
         // (i.e. in place of img.src or iframe.src or object.data).
         //
         // If no link rewriting is desired, this method should return false.
         virtual bool rewriteLink(const Element&, String& rewrittenLink)
         {
             return false;
         }
 
-        // Tells whether to skip serialization of a subresource with a given URI.
-        // Used to deduplicate resources across multiple frames.
-        virtual bool shouldSkipResource(const KURL&)
+        // Tells whether to skip serialization of a subresource or CSSStyleSheet
+        // with a given URI. Used to deduplicate resources across multiple frames.
+        virtual bool shouldSkipResourceWithURL(const KURL&)

[Łukasz Anforowicz, 2016/05/19 17:00:13]

Nice name change.  Long time ago this was also used to check if a subframe
should be skipped, but right now (as the name change correctly highlights) this
is only used for subresources.

[dewittj, 2016/05/19 18:18:36]

Acknowledged.

         {
             return false;
         }
+
+        // Tells whether to skip serialization of a subresource.
+        virtual bool shouldSkipResource(Resource*)
+        {
+            return false;
+        }
     };
 
     // Constructs a serializer that will write output to the given vector of
     // SerializedResources and uses the Delegate for controlling some
     // serialization aspects.  Callers need to ensure that both arguments stay
     // alive until the FrameSerializer gets destroyed.
     FrameSerializer(Vector<SerializedResource>&, Delegate&);
 
     // Initiates the serialization of the frame. All serialized content and
     // retrieved resources are added to the Vector passed to the constructor.
@@ skipping 22 matching lines @@
 
     Vector<SerializedResource>* m_resources;
     HashSet<KURL> m_resourceURLs;
 
     Delegate& m_delegate;
 };
 
 } // namespace blink
 
 #endif // FrameSerializer_h