diff options
author | kalman@chromium.org <kalman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-03 03:41:43 +0000 |
---|---|---|
committer | kalman@chromium.org <kalman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-05-03 03:41:43 +0000 |
commit | a0c91a9ffb1db127cf1f06e285b208a544ed4567 (patch) | |
tree | fe0e8f09a1c04cab35592cd46f9c01c045f95681 /chrome/browser/extensions/extension_function_test_utils.cc | |
parent | 68d3f6bbacbde53e968ae36ae3e4231ea1a8be4b (diff) | |
download | chromium_src-a0c91a9ffb1db127cf1f06e285b208a544ed4567.zip chromium_src-a0c91a9ffb1db127cf1f06e285b208a544ed4567.tar.gz chromium_src-a0c91a9ffb1db127cf1f06e285b208a544ed4567.tar.bz2 |
Drive extension functions from ExtensionFunction::Run. The
SyncExtensionFunction and AsyncExtensionFunction derivates now expose RunSync
and RunAsync respectively. New extension function implementations should just
implement Run directly.
BUG=365732
R=rockot@chromium.org
TBR=dmazzoni@chromium.org
Review URL: https://codereview.chromium.org/257333002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@268033 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/extension_function_test_utils.cc')
-rw-r--r-- | chrome/browser/extensions/extension_function_test_utils.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/extensions/extension_function_test_utils.cc b/chrome/browser/extensions/extension_function_test_utils.cc index e7de247..94951ca 100644 --- a/chrome/browser/extensions/extension_function_test_utils.cc +++ b/chrome/browser/extensions/extension_function_test_utils.cc @@ -257,9 +257,9 @@ bool RunFunction(UIThreadExtensionFunction* function, function->set_browser_context(browser->profile()); function->set_include_incognito(flags & INCLUDE_INCOGNITO); - function->Run(); + function->Run()->Execute(); - // If the RunImpl of |function| didn't already call SendResponse, run the + // If the RunAsync of |function| didn't already call SendResponse, run the // message loop until they do. if (!response_delegate.HasResponse()) { response_delegate.set_should_post_quit(true); |