Attach mixed content status to resource requests when sent to devtools

Source/platform/network/ResourceRequest.h

Index: Source/platform/network/ResourceRequest.h
diff --git a/Source/platform/network/ResourceRequest.h b/Source/platform/network/ResourceRequest.h
index 824ab21759c72adaa25c6ab6d627f6f719723fde..65315d6bd76a4ed8f6d7cac4deb1dad1cf1adee3 100644
--- a/Source/platform/network/ResourceRequest.h
+++ b/Source/platform/network/ResourceRequest.h
@@ -64,6 +64,12 @@ public:
         virtual ~ExtraData() { }
     };
 
+    enum MixedContentStatus {

[Mike West, 2015/08/17 15:36:45]

1. Why not use `MixedContentChecker::ContentType` instead of creating a new
enum?

2. I'd prefer to align with the spec's "blockable" and "optionally blockable"
language.

[estark, 2015/08/17 17:20:12]

Ah, I didn't realize that blockable/optionally blockable was spec language;
thought it was an internal MixedContentChecker thingy. Ok, so I moved
ContextType to be ResourceRequest::ContextType instead of
MixedContentChecker::ContextType, so that ResourceRequest can use it -- does
that sound okay?

There is a small additional wrinkle that ContextType doesn't allow a way to say
that a given request is NOT mixed content. So I added
ResourceRequest::isMixedContent() to distinguish, but that feels a little clunky
to me. (For example, not sure what to initialize ResourceRequest::contextType()
to if isMixedContent() is false.) Another alternative would be to add a
ContextTypeNotMixedContent value to the ContextType enum. What do you think?

+        NotMixedContent,
+        PassiveMixedContent,
+        ActiveMixedContent
+    };
+
     ResourceRequest()
     {
         initialize(KURL());
@@ -233,6 +239,9 @@ public:
     void setFollowedRedirect(bool followed) { m_followedRedirect = followed; }
     bool followedRedirect() const { return m_followedRedirect; }
 
+    void setMixedContentStatus(MixedContentStatus status) { m_mixedContentStatus = status; }
+    MixedContentStatus mixedContentStatus() const { return m_mixedContentStatus; }
+
 private:
     void initialize(const KURL&);
 
@@ -277,6 +286,8 @@ private:
     static double s_defaultTimeoutInterval;
 
     bool m_followedRedirect;
+
+    MixedContentStatus m_mixedContentStatus;
 };
 
 bool equalIgnoringHeaderFields(const ResourceRequest&, const ResourceRequest&);
@@ -322,6 +333,7 @@ public:
     bool m_originatesFromReservedIPRange;
     InputToLoadPerfMetricReportPolicy m_inputPerfMetricReportPolicy;
     bool m_followedRedirect;
+    ResourceRequest::MixedContentStatus m_mixedContentStatus;
 };
 
 } // namespace blink