[Offline pages] Empty RequestQueue and SavePageRequest

components/offline_pages/background/request_queue.cc

+// Copyright 2016 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 "components/offline_pages/background/request_queue.h"
+

[dougarnett, 2016/04/29 15:39:14]

I saw Pete do this too. Is there some style pref to have empty lines between
includes (even from same directory)? I've been more accustomed to them being
packed together.

[Pete Williamson, 2016/04/29 16:19:03]

The style guide says that the include file that goes with this .cc file should
be first.

Other than that, it says to put blank lines between different sections, and what
order the sections should be in.  It says to use alphabetical order within a
section.

https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes

[fgorski, 2016/04/30 18:56:14]

As Pete said.

+#include "components/offline_pages/background/save_page_request.h"
+
+namespace offline_pages {
+
+RequestQueue::RequestQueue() {}
+
+RequestQueue::~RequestQueue() {}
+
+void RequestQueue::GetRequests(const GetRequestsResult& callback) {}
+
+void RequestQueue::AddRequest(const SavePageRequest& request,
+                              const AddRequestCallback& callback) {}
+
+void RequestQueue::UpdateRequest(const SavePageRequest& request,
+                                 const UpdateRequestCallback& callback) {}
+
+void RequestQueue::RemoveRequest(int64_t request_id,
+                                 const UpdateRequestCallback& callback) {}
+
+void RequestQueue::PurgeRequests(const PurgeRequestsCallback& callback) {}
+
+}  // namespace offline_pages
+
+#endif  // COMPONENTS_OFFLINE_PAGES_BACKGROUND_REQUEST_QUEUE_H_