Initial framework setup and skeleton for BackgroundFetchContext

content/browser/background_fetch/fetch_request.cc

+// 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/fetch_request.h"
+
+#include "base/guid.h"
+#include "content/public/browser/download_item.h"
+
+namespace content {
+
+FetchRequest::FetchRequest()
+    : origin_(), url_(), tag_(), download_item_(nullptr) {}

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

nit: {} -> default, everywhere else in this file too

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

nit: members initialize themselves. you only need to list them in the
constructor if you need to assign them values, or call them if they're a parent
class.

As for |download_item_|, you could assign `nullptr` in the header file's
declaration to avoid having to do it here.

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

It seems that our compiler doesn't like = default after the member
inititalizations, but I changed it in the other two places.

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

Done.

+
+FetchRequest::FetchRequest(const url::Origin& origin,
+                           const url::Origin& url,
+                           const std::string& tag,
+                           DownloadItem* download_item)
+    : guid_(base::GenerateGUID()),
+      origin_(origin),
+      url_(url),
+      tag_(tag),
+      download_item_(download_item) {}
+
+FetchRequest::FetchRequest(const FetchRequest& request)
+    : guid_(request.guid_),
+      origin_(request.origin_),
+      url_(request.url_),
+      tag_(request.tag_),
+      download_item_(request.download_item_) {}
+
+FetchRequest::~FetchRequest() {}
+
+}  // namespace content