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

third_party/WebKit/Source/core/frame/FrameSerializer.cpp

Index: third_party/WebKit/Source/core/frame/FrameSerializer.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameSerializer.cpp b/third_party/WebKit/Source/core/frame/FrameSerializer.cpp
index 6427f7e04fab9595002ef102e06221ba685c23a4..bce326dc17d740384798931e2c82066a94fcb00e 100644
--- a/third_party/WebKit/Source/core/frame/FrameSerializer.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameSerializer.cpp
@@ -386,11 +386,16 @@ void FrameSerializer::serializeCSSRule(CSSRule* rule)
 bool FrameSerializer::shouldAddURL(const KURL& url)
 {
     return url.isValid() && !m_resourceURLs.contains(url) && !url.protocolIsData()
-        && !m_delegate.shouldSkipResource(url);
+        && !m_delegate.shouldSkipResourceWithURL(url);
 }
 
 void FrameSerializer::addToResources(Resource* resource, PassRefPtr<SharedBuffer> data, const KURL& url)
 {
+    if (m_delegate.shouldSkipResource(resource))
+        return;
+
+    DLOG(ERROR) << "Adding " << url.getString() << " to resources!";

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

I assume this is temporary (i.e. for printf-style debugging) and can be removed
now?

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

Done.

+
     if (!data) {
         DLOG(ERROR) << "No data for resource " << url.getString();
         return;