Add support for HTTP range requests in the AppCacheURLLoaderImpl class.

content/browser/appcache/appcache_job.h

 // Copyright (c) 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_APPCACHE_APPCACHE_JOB_H_
 #define CONTENT_BROWSER_APPCACHE_APPCACHE_JOB_H_
 
 #include <memory>
 
 #include "base/logging.h"
 #include "base/memory/weak_ptr.h"
 #include "base/sequence_checker.h"
 #include "base/strings/string16.h"
 #include "content/common/content_export.h"
+#include "net/http/http_byte_range.h"
 #include "url/gurl.h"
 
 namespace net {
+class HttpRequestHeaders;
+class HttpResponseInfo;
 class NetworkDelegate;
 class URLRequestJob;
 }
 
 namespace content {
 
 class AppCacheEntry;
 class AppCacheHost;
 class AppCacheRequest;
+class AppCacheResponseInfo;
+class AppCacheResponseReader;
 class AppCacheStorage;
 class AppCacheURLLoaderJob;
 class URLRequestJob;
 
 // Interface for an AppCache job. This is used to send data stored in the
 // AppCache to networking consumers.
 // Subclasses implement this interface to wrap custom job objects like
 // URLRequestJob, URLLoaderJob, etc to ensure that these dependencies stay out
 // of the AppCache code.
 class CONTENT_EXPORT AppCacheJob : public base::SupportsWeakPtr<AppCacheJob> {
@@ skipping 72 matching lines @@
   // AppCaches loaded via the URLLoader mechanism.
   virtual net::URLRequestJob* AsURLRequestJob();
 
   // Returns the underlying ApppCacheURLLoaderJob if any. This only applies to
   // AppCaches loaded via the URLRequest mechanism.
   virtual AppCacheURLLoaderJob* AsURLLoaderJob();
 
  protected:
   AppCacheJob();
 
+  bool is_range_request() const { return range_requested_.IsValid(); }
+
+  void InitializeRangeRequestInfo(const net::HttpRequestHeaders& headers);
+  void SetupRangeResponse(AppCacheResponseReader* reader);
+
   SEQUENCE_CHECKER(sequence_checker_);
 
   // Set to true if the AppCache entry is not found.
   bool cache_entry_not_found_;
 
   // The jobs delivery status.
   DeliveryType delivery_type_;
 
+  // Byte range request if any.
+  net::HttpByteRange range_requested_;

[michaeln, 2017/06/14 02:37:24]

nit: keep the two range related data members next to each other

[ananta, 2017/06/14 18:52:18]

Done.

+
+  // The response details.
+  scoped_refptr<AppCacheResponseInfo> info_;
+
+  std::unique_ptr<net::HttpResponseInfo> range_response_info_;
+
   base::WeakPtrFactory<AppCacheJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(AppCacheJob);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_APPCACHE_APPCACHE_JOB_H_