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

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

Issue 11316097: google_apis: Rename and simplify GenerateGetDocumentsURL() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address comments 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_unittest.cc
diff --git a/chrome/browser/google_apis/gdata_wapi_url_util_unittest.cc b/chrome/browser/google_apis/gdata_wapi_url_util_unittest.cc
index 5863d91838c3757edc5a1e634124c2b7e523479d..9544d4cccc77b2625b0c029214ee560341812730 100644
--- a/chrome/browser/google_apis/gdata_wapi_url_util_unittest.cc
+++ b/chrome/browser/google_apis/gdata_wapi_url_util_unittest.cc
@@ -50,13 +50,13 @@ TEST(GDataWapiUrlUtilTest, AddFeedUrlParams) {
).spec());
}
-TEST(GDataWapiUrlUtilTest, GenerateGetDocumentsURL) {
+TEST(GDataWapiUrlUtilTest, GenerateDocumentListUrl) {
// This is the very basic URL for the GetDocuments operation.
EXPECT_EQ(
"https://docs.google.com/feeds/default/private/full/-/mine"
"?v=3&alt=json&showfolders=true&max-results=500"
"&include-installed-apps=true",
- GenerateGetDocumentsURL(GURL(), // override_url,
+ GenerateDocumentListUrl(GURL(), // override_url,
0, // start_changestamp,
"", // search_string,
false, // shared_with_me,
@@ -69,7 +69,7 @@ TEST(GDataWapiUrlUtilTest, GenerateGetDocumentsURL) {
"http://localhost/"
"?v=3&alt=json&showfolders=true&max-results=500"
"&include-installed-apps=true",
- GenerateGetDocumentsURL(GURL("http://localhost/"), // override_url,
+ GenerateDocumentListUrl(GURL("http://localhost/"), // override_url,
0, // start_changestamp,
"", // search_string,
false, // shared_with_me,
@@ -83,7 +83,7 @@ TEST(GDataWapiUrlUtilTest, GenerateGetDocumentsURL) {
"?v=3&alt=json&showfolders=true&max-results=500"
"&include-installed-apps=true"
"&start-index=100",
- GenerateGetDocumentsURL(GURL(), // override_url,
+ GenerateDocumentListUrl(GURL(), // override_url,
100, // start_changestamp,
"", // search_string,
false, // shared_with_me,
@@ -96,7 +96,7 @@ TEST(GDataWapiUrlUtilTest, GenerateGetDocumentsURL) {
"https://docs.google.com/feeds/default/private/full/-/mine"
"?v=3&alt=json&showfolders=true&max-results=50"
"&include-installed-apps=true&q=foo",
- GenerateGetDocumentsURL(GURL(), // override_url,
+ GenerateDocumentListUrl(GURL(), // override_url,
0, // start_changestamp,
"foo", // search_string,
false, // shared_with_me,
@@ -109,7 +109,7 @@ TEST(GDataWapiUrlUtilTest, GenerateGetDocumentsURL) {
"https://docs.google.com/feeds/default/private/full/-/shared-with-me"
"?v=3&alt=json&showfolders=true&max-results=500"
"&include-installed-apps=true",
- GenerateGetDocumentsURL(GURL(), // override_url,
+ GenerateDocumentListUrl(GURL(), // override_url,
0, // start_changestamp,
"", // search_string,
true, // shared_with_me,
@@ -122,7 +122,7 @@ TEST(GDataWapiUrlUtilTest, GenerateGetDocumentsURL) {
"https://docs.google.com/feeds/default/private/full/XXX/contents/-/mine"
"?v=3&alt=json&showfolders=true&max-results=500"
"&include-installed-apps=true",
- GenerateGetDocumentsURL(GURL(), // override_url,
+ GenerateDocumentListUrl(GURL(), // override_url,
0, // start_changestamp,
"", // search_string,
false, // shared_with_me,

Powered by Google App Engine
This is Rietveld 408576698