Unescape resource id and search query contained in feed url in FakeDriveService

chrome/browser/google_apis/fake_drive_service.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/fake_drive_service.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 328 matching lines @@
   std::string directory_resource_id;
   int start_offset = 0;
   int max_results = default_max_results_;
   std::vector<std::pair<std::string, std::string> > parameters;
   if (base::SplitStringIntoKeyValuePairs(
           override_url.query(), '=', '&', &parameters)) {
     for (size_t i = 0; i < parameters.size(); ++i) {
       if (parameters[i].first == "changestamp") {
         base::StringToInt64(parameters[i].second, &start_changestamp);
       } else if (parameters[i].first == "q") {
         search_query = parameters[i].second;

[kinaba, 2013/06/04 06:46:52]

Do we also need unescaping here as well?
If so, could you please add it?

[nhiroki, 2013/06/04 07:05:52]

That's right. Done.

       } else if (parameters[i].first == "parent") {
-        directory_resource_id = parameters[i].second;
+        directory_resource_id =
+            net::UnescapeURLComponent(parameters[i].second,
+                                      net::UnescapeRule::URL_SPECIAL_CHARS);
       } else if (parameters[i].first == "start-offset") {
         base::StringToInt(parameters[i].second, &start_offset);
       } else if (parameters[i].first == "max-results") {
         base::StringToInt(parameters[i].second, &max_results);
       }
     }
   }
 
   GetResourceListInternal(
       start_changestamp, search_query, directory_resource_id,
@@ skipping 1068 matching lines @@
   if (load_counter)
     *load_counter += 1;
   base::MessageLoop::current()->PostTask(
       FROM_HERE,
       base::Bind(callback,
                  HTTP_SUCCESS,
                  base::Passed(&resource_list)));
 }
 
 }  // namespace google_apis