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

Unified Diff: chrome/browser/search_engines/search_provider_install_state_impl.h

Issue 2170313002: Remove OwnedInterface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments Created 4 years, 5 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
Index: chrome/browser/search_engines/search_provider_install_state_impl.h
diff --git a/chrome/browser/search_engines/search_provider_install_state_impl.h b/chrome/browser/search_engines/search_provider_install_state_impl.h
index 8cbe51f7e5aa7ac65f210ed7ccbbd125c5567cd8..389855ff54faec9035f5fc64fd5aabfac0b20e58 100644
--- a/chrome/browser/search_engines/search_provider_install_state_impl.h
+++ b/chrome/browser/search_engines/search_provider_install_state_impl.h
@@ -11,7 +11,7 @@
#include "chrome/browser/search_engines/search_provider_install_data.h"
#include "chrome/common/search_provider.mojom.h"
#include "content/public/browser/browser_message_filter.h"
-#include "mojo/public/cpp/bindings/binding_set.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
class GURL;
class Profile;
@@ -28,9 +28,13 @@ class SearchProviderInstallStateImpl
SearchProviderInstallStateImpl(int render_process_id, Profile* profile);
~SearchProviderInstallStateImpl() override;
- void Bind(chrome::mojom::SearchProviderInstallStateRequest request);
+ static void Create(int render_process_id,
+ Profile* profile,
+ chrome::mojom::SearchProviderInstallStateRequest request);
private:
+ void BindOnIOThread(chrome::mojom::SearchProviderInstallStateRequest request);
+
// Figures out the install state for the search provider.
chrome::mojom::InstallState GetSearchProviderInstallState(
const GURL& page_location,
@@ -54,7 +58,7 @@ class SearchProviderInstallStateImpl
// thread.
const bool is_off_the_record_;
- mojo::BindingSet<chrome::mojom::SearchProviderInstallState> binding_set_;
+ mojo::StrongBinding<chrome::mojom::SearchProviderInstallState> binding_;
// Used to schedule invocations of ReplyWithProviderInstallState.
base::WeakPtrFactory<SearchProviderInstallStateImpl> weak_factory_;

Powered by Google App Engine
This is Rietveld 408576698