Initial framework setup and skeleton for BackgroundFetchContext

content/browser/background_fetch/background_fetch_data_manager_unittest.cc

Index: content/browser/background_fetch/background_fetch_data_manager_unittest.cc
diff --git a/content/browser/background_fetch/background_fetch_data_manager_unittest.cc b/content/browser/background_fetch/background_fetch_data_manager_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..ab1c0852719336797976cedefc9179904a89e4f5
--- /dev/null
+++ b/content/browser/background_fetch/background_fetch_data_manager_unittest.cc
@@ -0,0 +1,51 @@
+// Copyright 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.
+
+#include "content/browser/background_fetch/background_fetch_data_manager.h"
+
+#include "content/browser/background_fetch/background_fetch_context.h"
+#include "content/browser/background_fetch/fetch_request.h"
+#include "content/browser/service_worker/service_worker_context_wrapper.h"
+#include "content/public/test/test_browser_context.h"
+#include "content/public/test/test_browser_thread_bundle.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+const char kOrigin[] = "https://example.com/";
+const char kResource[] = "https://example.com/resource.html";
+const char kTag[] = "TestRequestTag";
+
+class BackgroundFetchDataManagerTest : public testing::Test {
+ protected:
+  BackgroundFetchDataManagerTest()
+      : browser_context_(new TestBrowserContext),
+        background_fetch_data_manager_(
+            new BackgroundFetchDataManager(new BackgroundFetchContext(
+                browser_context_.get(),
+                new ServiceWorkerContextWrapper(browser_context_.get())))) {}

[Peter Beverloo, 2017/02/13 18:35:06]

I don't think this is OK. See EmbeddedWorkerTestHelper and co.

[harkness, 2017/02/14 13:52:46]

Ah, that's a nifty helper class. Updated.

+
+  BackgroundFetchDataManager* GetDataManager() const {
+    return background_fetch_data_manager_.get();
+  }
+
+ private:
+  TestBrowserThreadBundle browser_thread_bundle_;
+
+  std::unique_ptr<TestBrowserContext> browser_context_;

[Peter Beverloo, 2017/02/13 18:35:06]

nit: no need to use an std::unique_ptr<>, just have a TestBrowserContext as a
member

[harkness, 2017/02/14 13:52:46]

Done.

+  std::unique_ptr<BackgroundFetchDataManager> background_fetch_data_manager_;
+};
+
+TEST_F(BackgroundFetchDataManagerTest, AddRequest) {
+  BackgroundFetchDataManager* data_manager = GetDataManager();
+  FetchRequest request(url::Origin(GURL(kOrigin)), url::Origin(GURL(kResource)),
+                       kTag, nullptr);
+
+  data_manager->CreateRequest(request);
+
+  // TODO(harkness) There's no output to test yet. Once there is, check that the
+  // request was actually added.
+}
+
+}  // namespace content