Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1280)

Unified Diff: chrome/browser/google_apis/gdata_wapi_url_util.cc

Issue 11417108: google_apis: Get rid of gdata_wapi_url_util namespace (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/google_apis/gdata_wapi_url_util.cc
diff --git a/chrome/browser/google_apis/gdata_wapi_url_util.cc b/chrome/browser/google_apis/gdata_wapi_url_util.cc
deleted file mode 100644
index 78b639019c268c8ad595e10682aa842e35503484..0000000000000000000000000000000000000000
--- a/chrome/browser/google_apis/gdata_wapi_url_util.cc
+++ /dev/null
@@ -1,167 +0,0 @@
-// Copyright (c) 2012 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 "chrome/browser/google_apis/gdata_wapi_url_util.h"
-
-#include "base/logging.h"
-#include "base/stringprintf.h"
-#include "chrome/common/net/url_util.h"
-#include "googleurl/src/gurl.h"
-#include "net/base/escape.h"
-
-namespace google_apis {
-namespace {
-
-// URL requesting documents list that belong to the authenticated user only
-// (handled with '/-/mine' part).
-const char kGetDocumentListURLForAllDocuments[] =
- "/feeds/default/private/full/-/mine";
-
-// URL requesting documents list in a particular directory specified by "%s"
-// that belong to the authenticated user only (handled with '/-/mine' part).
-const char kGetDocumentListURLForDirectoryFormat[] =
- "/feeds/default/private/full/%s/contents/-/mine";
-
-// URL requesting single document entry whose resource id is specified by "%s".
-const char kGetDocumentEntryURLFormat[] = "/feeds/default/private/full/%s";
-
-// Root document list url.
-const char kDocumentListRootURL[] = "/feeds/default/private/full";
-
-// Metadata feed with things like user quota.
-const char kAccountMetadataURL[] = "/feeds/metadata/default";
-
-#ifndef NDEBUG
-// Use smaller 'page' size while debugging to ensure we hit feed reload
-// almost always. Be careful not to use something too small on account that
-// have many items because server side 503 error might kick in.
-const int kMaxDocumentsPerFeed = 500;
-const int kMaxDocumentsPerSearchFeed = 50;
-#else
-const int kMaxDocumentsPerFeed = 500;
-const int kMaxDocumentsPerSearchFeed = 50;
-#endif
-
-// URL requesting documents list that shared to the authenticated user only
-const char kGetDocumentListURLForSharedWithMe[] =
- "/feeds/default/private/full/-/shared-with-me";
-
-// URL requesting documents list of changes to documents collections.
-const char kGetChangesListURL[] = "/feeds/default/private/changes";
-
-} // namespace
-
-namespace gdata_wapi_url_util {
-
-const char kBaseUrlForProduction[] = "https://docs.google.com/";
-const char kBaseUrlForTesting[] = "http://127.0.0.1/";
-
-GURL AddStandardUrlParams(const GURL& url) {
- GURL result =
- chrome_common_net::AppendOrReplaceQueryParameter(url, "v", "3");
- result =
- chrome_common_net::AppendOrReplaceQueryParameter(result, "alt", "json");
- return result;
-}
-
-GURL AddMetadataUrlParams(const GURL& url) {
- GURL result = AddStandardUrlParams(url);
- result = chrome_common_net::AppendOrReplaceQueryParameter(
- result, "include-installed-apps", "true");
- return result;
-}
-
-GURL AddFeedUrlParams(const GURL& url,
- int num_items_to_fetch,
- int changestamp,
- const std::string& search_string) {
- GURL result = AddStandardUrlParams(url);
- result = chrome_common_net::AppendOrReplaceQueryParameter(
- result,
- "showfolders",
- "true");
- result = chrome_common_net::AppendOrReplaceQueryParameter(
- result,
- "max-results",
- base::StringPrintf("%d", num_items_to_fetch));
- result = chrome_common_net::AppendOrReplaceQueryParameter(
- result, "include-installed-apps", "true");
-
- if (changestamp) {
- result = chrome_common_net::AppendQueryParameter(
- result,
- "start-index",
- base::StringPrintf("%d", changestamp));
- }
-
- if (!search_string.empty()) {
- result = chrome_common_net::AppendOrReplaceQueryParameter(
- result, "q", search_string);
- }
- return result;
-}
-
-} // namespace gdata_wapi_url_util
-
-
-// TODO(satorux): Move the following code to a separate file
-// gdata_wapi_url_generator.cc
-
-GDataWapiUrlGenerator::GDataWapiUrlGenerator(const GURL& base_url)
- : base_url_(GURL(base_url)) {
-}
-
-GDataWapiUrlGenerator::~GDataWapiUrlGenerator() {
-}
-
-GURL GDataWapiUrlGenerator::GenerateDocumentListUrl(
- const GURL& override_url,
- int start_changestamp,
- const std::string& search_string,
- bool shared_with_me,
- const std::string& directory_resource_id) const {
- DCHECK_LE(0, start_changestamp);
-
- int max_docs = search_string.empty() ? kMaxDocumentsPerFeed :
- kMaxDocumentsPerSearchFeed;
- GURL url;
- if (!override_url.is_empty()) {
- url = override_url;
- } else if (shared_with_me) {
- url = base_url_.Resolve(kGetDocumentListURLForSharedWithMe);
- } else if (start_changestamp > 0) {
- // The start changestamp shouldn't be used for a search.
- DCHECK(search_string.empty());
- url = base_url_.Resolve(kGetChangesListURL);
- } else if (!directory_resource_id.empty()) {
- url = base_url_.Resolve(
- base::StringPrintf(kGetDocumentListURLForDirectoryFormat,
- net::EscapePath(
- directory_resource_id).c_str()));
- } else {
- url = base_url_.Resolve(kGetDocumentListURLForAllDocuments);
- }
- return gdata_wapi_url_util::AddFeedUrlParams(
- url, max_docs, start_changestamp, search_string);
-}
-
-GURL GDataWapiUrlGenerator::GenerateDocumentEntryUrl(
- const std::string& resource_id) const {
- GURL result = base_url_.Resolve(
- base::StringPrintf(kGetDocumentEntryURLFormat,
- net::EscapePath(resource_id).c_str()));
- return gdata_wapi_url_util::AddStandardUrlParams(result);
-}
-
-GURL GDataWapiUrlGenerator::GenerateDocumentListRootUrl() const {
- return gdata_wapi_url_util::AddStandardUrlParams(
- base_url_.Resolve(kDocumentListRootURL));
-}
-
-GURL GDataWapiUrlGenerator::GenerateAccountMetadataUrl() const {
- return gdata_wapi_url_util::AddMetadataUrlParams(
- base_url_.Resolve(kAccountMetadataURL));
-}
-
-} // namespace google_apis
« no previous file with comments | « chrome/browser/google_apis/gdata_wapi_url_util.h ('k') | chrome/browser/google_apis/gdata_wapi_url_util_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698