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

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: Changes to address kmarshall's comments Created 4 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
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 ab5e89578e5d1e634a3e13afc166cfe3a05ab1d5..8b9f989e8f2b27a651c134b4a101a6c0fb88681c 100644
--- a/blimp/engine/app/blimp_content_browser_client.h
+++ b/blimp/engine/app/blimp_content_browser_client.h
@@ -6,6 +6,7 @@
#define BLIMP_ENGINE_APP_BLIMP_CONTENT_BROWSER_CLIENT_H_
#include "base/macros.h"
+#include "blimp/engine/feature/geolocation/blimp_location_provider.h"
#include "content/public/browser/content_browser_client.h"
namespace blimp {
@@ -33,10 +34,16 @@ class BlimpContentBrowserClient : public content::ContentBrowserClient {
BlimpBrowserContext* GetBrowserContext();
+ content::LocationProvider* OverrideSystemLocationProvider() override;
Kevin M 2016/05/31 23:43:55 Cluster these overrides with the others under "con
CJ 2016/05/31 23:51:24 Done.
+
+ bool UseDefaultLocationProviders() override;
+
private:
// Owned by BrowserMainLoop
BlimpBrowserMainParts* blimp_browser_main_parts_ = nullptr;
+ BlimpLocationProvider location_provider_;
+
DISALLOW_COPY_AND_ASSIGN(BlimpContentBrowserClient);
};

Powered by Google App Engine
This is Rietveld 408576698