Remove duplicate calls to SendResponse(). BookmarkFunction already does this.

BUG=106251


Review URL: http://codereview.chromium.org/9295004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119381 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 2 deletions

diff --git a/chrome/browser/bookmarks/bookmark_manager_extension_api.cc b/chrome/browser/bookmarks/bookmark_manager_extension_api.cc
index e879231..c4040d2 100644
--- a/chrome/browser/bookmarks/bookmark_manager_extension_api.cc
+++ b/chrome/browser/bookmarks/bookmark_manager_extension_api.cc
@@ -278,7 +278,6 @@ bool CanPasteBookmarkManagerFunction::RunImpl() {
   }
   bool can_paste = bookmark_utils::CanPasteFromClipboard(parent_node);
   result_.reset(Value::CreateBooleanValue(can_paste));
-  SendResponse(true);
   return true;
 }
 
@@ -368,7 +367,11 @@ bool BookmarkManagerGetStringsFunction::RunImpl() {
   ChromeURLDataManager::DataSource::SetFontAndTextDirection(localized_strings);
 
   result_.reset(localized_strings);
+
+  // This is needed because unlike the rest of these functions, this class
+  // inherits from AsyncFunction directly, rather than BookmarkFunction.
   SendResponse(true);
+
   return true;
 }
 
@@ -444,7 +447,6 @@ bool DropBookmarkManagerFunction::RunImpl() {
                                         drop_parent, drop_index);
 
     router->ClearBookmarkNodeData();
-    SendResponse(true);
     return true;
   } else {
     NOTREACHED();