Add an abstraction for a job in the AppCacheRequestHandler class.

content/browser/appcache/appcache_url_loader_job.h

Index: content/browser/appcache/appcache_url_loader_job.h
diff --git a/content/browser/appcache/appcache_url_loader_job.h b/content/browser/appcache/appcache_url_loader_job.h
new file mode 100644
index 0000000000000000000000000000000000000000..5c007b69afa3546b786ffc84387db056ec1277c3
--- /dev/null
+++ b/content/browser/appcache/appcache_url_loader_job.h
@@ -0,0 +1,53 @@
+// 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_URL_LOADER_JOB_H_
+#define CONTENT_BROWSER_APPCACHE_APPCACHE_URL_LOADER_JOB_H_
+
+#include "base/logging.h"
+#include "base/strings/string16.h"
+#include "content/browser/appcache/appcache_job.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+class AppCacheHost;
+class AppCacheRequest;
+class AppCacheStorage;
+
+// AppCacheJob wrapper for the AppCacheURLLoaderJob class.

[jam, 2017/05/05 23:15:25]

nit: the comment is a bit confusing. maybe it should be like the net:: one and
say 

// A mojom::URLLoader derivative that knows how to return a response stored
// in the appcache.

[ananta, 2017/05/06 03:01:35]

Done.

+class CONTENT_EXPORT AppCacheURLLoaderJob : public AppCacheJob {
+ public:
+  ~AppCacheURLLoaderJob() override;
+
+  // AppCacheJob overrides.
+  void Kill() override;
+  bool IsStarted() const override;
+  bool IsWaiting() const override;
+  bool IsDeliveringAppCacheResponse() const override;
+  bool IsDeliveringNetworkResponse() const override;
+  bool IsDeliveringErrorResponse() const override;
+  bool IsCacheEntryNotFound() const override;
+  void DeliverAppCachedResponse(const GURL& manifest_url,
+                                int64_t cache_id,
+                                const AppCacheEntry& entry,
+                                bool is_fallback) override;
+  void DeliverNetworkResponse() override;
+  void DeliverErrorResponse() override;
+  const GURL& GetURL() const override;
+
+ protected:
+  // AppCacheJob::Create() creates this instance.

[jam, 2017/05/05 23:15:25]

ditto

[ananta, 2017/05/06 03:01:35]

Leaving this as is based on the response in the other comment.

+  friend class AppCacheJob;
+
+  AppCacheURLLoaderJob();
+
+  GURL url_;
+
+  DISALLOW_COPY_AND_ASSIGN(AppCacheURLLoaderJob);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_APPCACHE_APPCACHE_URL_LOADER_JOB_H_