[ServiceWorker] Add was_fallback_required flag to ResourceResponseInfo. [1/2 blink]

Source/platform/network/ResourceResponse.h

 /*
  * Copyright (C) 2006, 2008 Apple Inc. All rights reserved.
  * 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 144 matching lines @@
     {
       m_wasAlternateProtocolAvailable = value;
     }
 
     bool wasFetchedViaProxy() const { return m_wasFetchedViaProxy; }
     void setWasFetchedViaProxy(bool value) { m_wasFetchedViaProxy = value; }
 
     bool wasFetchedViaServiceWorker() const { return m_wasFetchedViaServiceWorker; }
     void setWasFetchedViaServiceWorker(bool value) { m_wasFetchedViaServiceWorker = value; }
 
+    bool wasFallbackRequired() const { return m_wasFallbackRequired; }
+    void setWasFallbackRequired(bool value) { m_wasFallbackRequired = value; }
+
     bool isMultipartPayload() const { return m_isMultipartPayload; }
     void setIsMultipartPayload(bool value) { m_isMultipartPayload = value; }
 
     double responseTime() const { return m_responseTime; }
     void setResponseTime(double responseTime) { m_responseTime = responseTime; }
 
     const AtomicString& remoteIPAddress() const { return m_remoteIPAddress; }
     void setRemoteIPAddress(const AtomicString& value) { m_remoteIPAddress = value; }
 
     unsigned short remotePort() const { return m_remotePort; }
@@ skipping 76 matching lines @@
     // 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;
 
     // Was the resource fetched over a ServiceWorker.
     bool m_wasFetchedViaServiceWorker;
 
+    // Was the fallback request with skip service worker flag required.
+    bool m_wasFallbackRequired;

[yhirano, 2014/09/26 01:23:32]

Isn't this name too generic? Suffixing with "ByServiceWorker" looks good.

[horo, 2014/09/26 01:44:16]

Done.

+
     // The time at which the response headers were received.  For cached
     // responses, this time could be "far" in the past.
     double m_responseTime;
 
     // Remote IP address of the socket which fetched this resource.
     AtomicString m_remoteIPAddress;
 
     // Remote port number of the socket which fetched this resource.
     unsigned short m_remotePort;
 
@@ skipping 28 matching lines @@
     CString m_securityInfo;
     ResourceResponse::HTTPVersion m_httpVersion;
     long long m_appCacheID;
     KURL m_appCacheManifestURL;
     bool m_isMultipartPayload;
     bool m_wasFetchedViaSPDY;
     bool m_wasNpnNegotiated;
     bool m_wasAlternateProtocolAvailable;
     bool m_wasFetchedViaProxy;
     bool m_wasFetchedViaServiceWorker;
+    bool m_wasFallbackRequired;

[yhirano, 2014/09/26 01:23:32]

ditto

[horo, 2014/09/26 01:44:16]

Done.

     double m_responseTime;
     String m_remoteIPAddress;
     unsigned short m_remotePort;
     String m_downloadedFilePath;
     RefPtr<BlobDataHandle> m_downloadedFileHandle;
 };
 
 } // namespace blink
 
 #endif // ResourceResponse_h