Split UI-specific bits off ExtensionFunction into a separate class.

This is a precursor to another patch I'm working on, which will add
ExtensionFunctions that can run on the IO thread. The webRequest API will use
this.

I've also done a bit of other cleanup:
- moved almost everything off of {Async,Sync}ExtensionFunction, up into the
base class, since none of it was specific to either of those.
- store Extension directly on ExtensionFunction, rather than needing to look it
up, since it is now refcounted.

BUG=no
TEST=no

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

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

1 files changed, 4 insertions, 0 deletions

diff --git a/chrome/browser/extensions/extensions_quota_service_unittest.cc b/chrome/browser/extensions/extensions_quota_service_unittest.cc
index 63eedb1..39cd64e3 100644
--- a/chrome/browser/extensions/extensions_quota_service_unittest.cc
+++ b/chrome/browser/extensions/extensions_quota_service_unittest.cc
@@ -56,6 +56,10 @@ class MockFunction : public ExtensionFunction {
   virtual const std::string GetError() { return std::string(); }
   virtual const std::string GetResult() { return std::string(); }
   virtual void Run() {}
+  virtual void Destruct() const { delete this; }
+  virtual bool RunImpl() { return true; }
+  virtual void SendResponse(bool) { }
+  virtual void HandleBadMessage() { }
 };
 
 class TimedLimitMockFunction : public MockFunction {