Changeset 61865 in webkit

Timestamp:

Jun 25, 2010 8:45:41 AM (14 years ago)

Author:

commit-queue@webkit.org

Message:

2010-06-25 Lei Zheng <​lzheng@chromium.org>

Reviewed by Dimitri Glazkov.

Add a flag to the ResourceResponse for tracking if a request was fetched when
Alternate-protocol is available.
​https://bugs.webkit.org/show_bug.cgi?id=41001

No new tests. (This is just a setter and getter.)

• platform/network/chromium/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse): (WebCore::ResourceResponse::wasAlternateProtocolAvailable): (WebCore::ResourceResponse::setWasAlternateProtocolAvailable):

2010-06-25 Lei Zheng <​lzheng@chromium.org>

Reviewed by Dimitri Glazkov.

[Chromium] Add a flag to the ResourceResponse for tracking if a request was fetched
when Alternate-Protocol is available.

​https://bugs.webkit.org/show_bug.cgi?id=41001

• public/WebURLResponse.h:
• src/WebURLResponse.cpp: (WebKit::WebURLResponse::wasAlternateProtocolAvailable): (WebKit::WebURLResponse::setWasAlternateProtocolAvailable):

Location:

trunk

Files:

• WebCore/ChangeLog (modified) (1 diff)
• WebCore/platform/network/chromium/ResourceResponse.h (modified) (5 diffs)
• WebKit/chromium/ChangeLog (modified) (1 diff)
• WebKit/chromium/public/WebURLResponse.h (modified) (1 diff)
• WebKit/chromium/src/WebURLResponse.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-06-25  Lei Zheng  <lzheng@chromium.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        Add a flag to the ResourceResponse for tracking if a request was fetched when
+        Alternate-protocol is available.
+        https://bugs.webkit.org/show_bug.cgi?id=41001
+
+        No new tests. (This is just a setter and getter.)
+
+        * platform/network/chromium/ResourceResponse.h:
+        (WebCore::ResourceResponse::ResourceResponse):
+        (WebCore::ResourceResponse::wasAlternateProtocolAvailable):
+        (WebCore::ResourceResponse::setWasAlternateProtocolAvailable):
+
 2010-06-25  Lyon Chen  <liachen@rim.com>
 

trunk/WebCore/platform/network/chromium/ResourceResponse.h

@@ -22 +22 @@
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
@@ -42 +42 @@
             , m_wasFetchedViaSPDY(false)
             , m_wasNpnNegotiated(false)
+            , m_wasAlternateProtocolAvailable(false)
             , m_wasFetchedViaProxy(false)
             , m_responseTime(0)
@@ -54 +55 @@
             , m_wasFetchedViaSPDY(false)
             , m_wasNpnNegotiated(false)
+            , m_wasAlternateProtocolAvailable(false)
             , m_wasFetchedViaProxy(false)
             , m_responseTime(0)
@@ -76 +78 @@
         bool wasNpnNegotiated() const { return m_wasNpnNegotiated; }
         void setWasNpnNegotiated(bool value) { m_wasNpnNegotiated = value; }
+
+        bool wasAlternateProtocolAvailable() const
+        {
+          return m_wasAlternateProtocolAvailable;
+        }
+        void setWasAlternateProtocolAvailable(bool value)
+        {
+          m_wasAlternateProtocolAvailable = value;
+        }
 
         bool wasFetchedViaProxy() const { return m_wasFetchedViaProxy; }
@@ -120 +131 @@
         bool m_wasNpnNegotiated;
 
+        // Was the resource fetched over a channel which specified "Alternate-Protocol"
+        // (e.g.: Alternate-Protocol: 443:npn-spdy/1).
+        bool m_wasAlternateProtocolAvailable;
+
         // Was the resource fetched over an explicit proxy (HTTP, SOCKS, etc).
         bool m_wasFetchedViaProxy;

trunk/WebKit/chromium/ChangeLog

@@ +1 @@
+2010-06-25  Lei Zheng  <lzheng@chromium.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        [Chromium] Add a flag to the ResourceResponse for tracking if a request was fetched
+        when Alternate-Protocol is available.
+
+        https://bugs.webkit.org/show_bug.cgi?id=41001
+
+        * public/WebURLResponse.h:
+        * src/WebURLResponse.cpp:
+        (WebKit::WebURLResponse::wasAlternateProtocolAvailable):
+        (WebKit::WebURLResponse::setWasAlternateProtocolAvailable):
+
 2010-06-25  Michael Nordman  <michaeln@google.com>
 

trunk/WebKit/chromium/public/WebURLResponse.h

@@ -132 +132 @@
     WEBKIT_API void setWasNpnNegotiated(bool);
 
+    // Flag whether this request was made when "Alternate-Protocol: xxx"
+    // is present in server's response.
+    WEBKIT_API bool wasAlternateProtocolAvailable() const;
+    WEBKIT_API void setWasAlternateProtocolAvailable(bool);
+
     // Flag whether this request was loaded via an explicit proxy (HTTP, SOCKS, etc).
     WEBKIT_API bool wasFetchedViaProxy() const;

trunk/WebKit/chromium/src/WebURLResponse.cpp

@@ -288 +288 @@
 }
 
+bool WebURLResponse::wasAlternateProtocolAvailable() const
+{
+    return m_private->m_resourceResponse->wasAlternateProtocolAvailable();
+}
+
+void WebURLResponse::setWasAlternateProtocolAvailable(bool value)
+{
+    m_private->m_resourceResponse->setWasAlternateProtocolAvailable(value);
+}
+
 bool WebURLResponse::wasFetchedViaProxy() const
 {