Push bookmarks.remove/removeAll polymorphism into c++. fix bookmarks id schema issues

chrome/browser/extensions/extension_function.h

 // Copyright (c) 2009 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_H_
 
 #include <string>
 
 #include "base/values.h"
@@ skipping 11 matching lines @@
     } \
   } while (0)
 
 // Abstract base class for extension functions the ExtensionFunctionDispatcher
 // knows how to dispatch to.
 //
 // TODO(aa): This will have to become reference counted when we introduce
 // APIs that live beyond a single stack frame.
 class ExtensionFunction : public base::RefCounted<ExtensionFunction> {
  public:
-  ExtensionFunction() : request_id_(-1), has_callback_(false) {}
+  ExtensionFunction() : request_id_(-1), name_(""), has_callback_(false) {}
   virtual ~ExtensionFunction() {}
 
   // Specifies the name of the function.
-  virtual void SetName(const std::string& name) { }
+  void set_name(const std::string& name) { name_ = name; }
+  const std::string name() { return name_; }
 
   // Specifies the raw arguments to the function, as a JSON-encoded string.
   virtual void SetArgs(const std::string& args) = 0;
 
   // Retrieves the results of the function as a JSON-encoded string (may
   // be empty).
   virtual const std::string GetResult() = 0;
 
   // Retrieves any error string from the function.
   virtual const std::string GetError() = 0;
@@ skipping 17 matching lines @@
   // function.
   virtual void Run() = 0;
 
  protected:
   // The peer to the dispatcher that will service this extension function call.
   scoped_refptr<ExtensionFunctionDispatcher::Peer> peer_;
 
   // Id of this request, used to map the response back to the caller.
   int request_id_;
 
+  // The name of this function.
+  std::string name_;
+
   // True if the js caller provides a callback function to receive the response
   // of this call.
   bool has_callback_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ExtensionFunction);
 };
 
 // Base class for an extension function that runs asynchronously *relative to
 // the browser's UI thread*.
@@ skipping 62 matching lines @@
 
   virtual void Run() {
     SendResponse(RunImpl());
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SyncExtensionFunction);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_FUNCTION_H_