Don't expose DocumentWriter to embedding layer

Source/core/loader/SubstituteData.h

Index: Source/core/loader/SubstituteData.h
diff --git a/Source/core/loader/SubstituteData.h b/Source/core/loader/SubstituteData.h
index cdf7979c90aeecc5490a367e03f76dc6d998da1a..7b16a8d4b231fa5e0aee4a8223496541de66293d 100644
--- a/Source/core/loader/SubstituteData.h
+++ b/Source/core/loader/SubstituteData.h
@@ -32,32 +32,42 @@
 #include "wtf/RefPtr.h"
 
 namespace WebCore {
+enum SubstituteDataLoadPolicy {

[abarth-chromium, 2014/01/06 22:29:31]

We usually have a blank line above these sorts of declarations.

+    LoadNormally,
+    ForceSynchronousLoad
+};
 
-    class SubstituteData {
-    public:
-        SubstituteData() { }
+class SubstituteData {
+public:
+    SubstituteData()
+        : m_substituteDataLoadPolicy(LoadNormally)
+    {
+    }
 
-        SubstituteData(PassRefPtr<SharedBuffer> content, const AtomicString& mimeType, const AtomicString& textEncoding, const KURL& failingURL)
-            : m_content(content)
-            , m_mimeType(mimeType)
-            , m_textEncoding(textEncoding)
-            , m_failingURL(failingURL)
-        {
-        }
+    SubstituteData(PassRefPtr<SharedBuffer> content, const AtomicString& mimeType, const AtomicString& textEncoding, const KURL& failingURL, SubstituteDataLoadPolicy substituteDataLoadPolicy = LoadNormally)
+        : m_content(content)
+        , m_mimeType(mimeType)
+        , m_textEncoding(textEncoding)
+        , m_failingURL(failingURL)
+        , m_substituteDataLoadPolicy(substituteDataLoadPolicy)
+    {
+    }
 
-        bool isValid() const { return m_content != 0; }
+    bool isValid() const { return m_content; }
 
-        SharedBuffer* content() const { return m_content.get(); }
-        const AtomicString& mimeType() const { return m_mimeType; }
-        const AtomicString& textEncoding() const { return m_textEncoding; }
-        const KURL& failingURL() const { return m_failingURL; }
+    SharedBuffer* content() const { return m_content.get(); }
+    const AtomicString& mimeType() const { return m_mimeType; }
+    const AtomicString& textEncoding() const { return m_textEncoding; }
+    const KURL& failingURL() const { return m_failingURL; }
+    bool forceSynchronousLoad() const { return m_substituteDataLoadPolicy == ForceSynchronousLoad; }

[abarth-chromium, 2014/01/06 22:29:31]

This sounds a bit like a verb...  I think it's ok though.

 
-    private:
-        RefPtr<SharedBuffer> m_content;
-        AtomicString m_mimeType;
-        AtomicString m_textEncoding;
-        KURL m_failingURL;
-    };
+private:
+    RefPtr<SharedBuffer> m_content;
+    AtomicString m_mimeType;
+    AtomicString m_textEncoding;
+    KURL m_failingURL;
+    SubstituteDataLoadPolicy m_substituteDataLoadPolicy;
+};
 
 }