Oilpan: move MHTML objects to the heap.

Source/web/WebPageSerializer.cpp

 /*
  * Copyright (C) 2009 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 26 matching lines @@
 #include "core/frame/LocalFrame.h"
 #include "core/html/HTMLAllCollection.h"
 #include "core/html/HTMLFrameElementBase.h"
 #include "core/html/HTMLFrameOwnerElement.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/HTMLTableElement.h"
 #include "core/loader/DocumentLoader.h"
 #include "core/page/Page.h"
 #include "core/page/PageSerializer.h"
 #include "platform/SerializedResource.h"
+#include "platform/mhtml/ArchiveResource.h"

[tkent, 2014/07/09 01:04:34]

Why do we need to include it? It seems ArchiveResource-related functions are not
used in this file.

[sof, 2014/07/09 04:42:19]

MHTMLArchive inlines a vector append() so as to require having the full type of
ArchiveResource in scope. We can either "outline" it or revert back to having a
full #include of "ArchiveResource.h" in "MHTMLArchive.h"

[tkent, 2014/07/09 04:49:24]

I see.  So this include looks wrong to me.
Please do either.

[sof, 2014/07/09 05:11:20]

Yes, removed. To make MHTMLArchive.h closed, I moved the definitions of these
helpers to the implementation file.

(Looked over the other #includes of "ArchiveResource.h" added in this CL;
they're needed.)

 #include "platform/mhtml/MHTMLArchive.h"
 #include "platform/weborigin/KURL.h"
 #include "public/platform/WebCString.h"
 #include "public/platform/WebString.h"
 #include "public/platform/WebURL.h"
 #include "public/platform/WebVector.h"
 #include "public/web/WebFrame.h"
 #include "public/web/WebPageSerializerClient.h"
 #include "public/web/WebView.h"
 #include "web/WebLocalFrameImpl.h"
@@ skipping 207 matching lines @@
 
 WebString WebPageSerializer::generateBaseTagDeclaration(const WebString& baseTarget)
 {
     if (baseTarget.isEmpty())
         return String("<base href=\".\">");
     String baseString = "<base href=\".\" target=\"" + static_cast<const String&>(baseTarget) + "\">";
     return baseString;
 }
 
 } // namespace blink