Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1595)

Unified Diff: content/public/browser/plugin_service.h

Issue 2873963005: Use PostTaskAndReplyWithResult in plugin_service_impl.cc. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/plugin_service_impl.cc ('k') | content/test/fake_plugin_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/public/browser/plugin_service.h
diff --git a/content/public/browser/plugin_service.h b/content/public/browser/plugin_service.h
index c404ac82a4822fe7a17a9fa5df6ab2f7c1425feb..ac05c13b1df8d8321fd7e8bb387e5e3defd8eebf 100644
--- a/content/public/browser/plugin_service.h
+++ b/content/public/browser/plugin_service.h
@@ -37,8 +37,8 @@ struct WebPluginInfo;
// doing expensive disk operations on the IO/UI threads.
class PluginService {
public:
- typedef base::Callback<void(const std::vector<WebPluginInfo>&)>
- GetPluginsCallback;
+ using GetPluginsCallback =
+ base::OnceCallback<void(const std::vector<WebPluginInfo>&)>;
// Returns the PluginService singleton.
CONTENT_EXPORT static PluginService* GetInstance();
@@ -94,8 +94,8 @@ class PluginService {
const base::FilePath& plugin_path) = 0;
// Asynchronously loads plugins if necessary and then calls back to the
- // provided function on the calling MessageLoop on completion.
- virtual void GetPlugins(const GetPluginsCallback& callback) = 0;
+ // provided function on the calling sequence on completion.
+ virtual void GetPlugins(GetPluginsCallback callback) = 0;
// Returns information about a pepper plugin if it exists, otherwise nullptr.
// The caller does not own the pointer, and it's not guaranteed to live past
« no previous file with comments | « content/browser/plugin_service_impl.cc ('k') | content/test/fake_plugin_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698