Small clean up of GDataWapiUrlGenerator.

chrome/browser/google_apis/gdata_wapi_url_generator.cc

 // 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_generator.h"
 
 #include "base/logging.h"
 #include "base/stringprintf.h"
+#include "base/strings/string_number_conversions.h"
 #include "googleurl/src/gurl.h"
 #include "net/base/escape.h"
 #include "net/base/url_util.h"
 
 namespace google_apis {
 namespace {
 
 // Content URL for modification or resource list retrieval in a particular
 // directory specified by "%s" which will be replaced with its resource id.
 const char kContentURLFormat[] = "/feeds/default/private/full/%s/contents";
@@ skipping 54 matching lines @@
 // static
 GURL GDataWapiUrlGenerator::AddFeedUrlParams(
     const GURL& url,
     int num_items_to_fetch,
     int changestamp,
     const std::string& search_string) {
   GURL result = AddStandardUrlParams(url);
   result = net::AppendOrReplaceQueryParameter(result, "showfolders", "true");
   result = net::AppendOrReplaceQueryParameter(result, "include-shared", "true");
   result = net::AppendOrReplaceQueryParameter(
-      result,
-      "max-results",
-      base::StringPrintf("%d", num_items_to_fetch));
-  result = net::AppendOrReplaceQueryParameter(
-      result, "include-installed-apps", "true");
+      result, "max-results", base::IntToString(num_items_to_fetch));
 
   if (changestamp) {
-    result = net::AppendQueryParameter(result,
-                                       "start-index",
-                                       base::StringPrintf("%d", changestamp));
+    result = net::AppendQueryParameter(
+        result, "start-index", base::IntToString(changestamp));

[kinaba, 2013/04/18 09:47:45]

This is not directly related to this patch, but I'm just wondering,
why is |changestamp| int here? Almost everywhere else in the
code we are using int64 for changestamps.

   }
 
   if (!search_string.empty()) {
     result = net::AppendOrReplaceQueryParameter(result, "q", search_string);
   }
   return result;
 }
 
 GDataWapiUrlGenerator::GDataWapiUrlGenerator(const GURL& base_url)
     : base_url_(GURL(base_url)) {
@@ skipping 95 matching lines @@
     bool include_installed_apps) const {
   GURL result = AddStandardUrlParams(base_url_.Resolve(kAccountMetadataURL));
   if (include_installed_apps) {
     result = net::AppendOrReplaceQueryParameter(
         result, "include-installed-apps", "true");
   }
   return result;
 }
 
 }  // namespace google_apis