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

Unified Diff: blimp/engine/app/blimp_content_browser_client.h

Issue 2028823002: Refactor to make BlimpLocationProvider accessible to content layer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addresses kmarshall's and mvanouwerkerk's comments + code clean up Created 4 years, 6 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: blimp/engine/app/blimp_content_browser_client.h
diff --git a/blimp/engine/app/blimp_content_browser_client.h b/blimp/engine/app/blimp_content_browser_client.h
index 936d1267fe775b43ec797bf0efc7df7077ba0861..24e3896ccadccbf6883206a3b38b877db34f0946 100644
--- a/blimp/engine/app/blimp_content_browser_client.h
+++ b/blimp/engine/app/blimp_content_browser_client.h
@@ -31,16 +31,16 @@ class BlimpContentBrowserClient : public content::ContentBrowserClient {
content::WebPreferences* prefs) override;
void RegisterRenderProcessMojoServices(
content::ServiceRegistry* registry) override;
+ content::LocationProvider* OverrideSystemLocationProvider() override;
+ bool UseNetworkLocationProviders() override;
BlimpBrowserContext* GetBrowserContext();
- content::LocationProvider* SoleLocationProvider() override;
-
private:
// Owned by BrowserMainLoop
BlimpBrowserMainParts* blimp_browser_main_parts_ = nullptr;
- BlimpLocationProvider location_provider_;
+ std::unique_ptr<BlimpLocationProvider> location_provider_;
DISALLOW_COPY_AND_ASSIGN(BlimpContentBrowserClient);
};

Powered by Google App Engine
This is Rietveld 408576698