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

Unified Diff: webkit/tools/test_shell/test_geolocation_service.cc

Issue 5938002: Remove legacy non-client-based geolocation code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove non-client-based geolocation code. Created 9 years, 12 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: webkit/tools/test_shell/test_geolocation_service.cc
diff --git a/webkit/tools/test_shell/test_geolocation_service.cc b/webkit/tools/test_shell/test_geolocation_service.cc
deleted file mode 100644
index b4128131a67199cd757b4bcd5e1ce60bcfbef615..0000000000000000000000000000000000000000
--- a/webkit/tools/test_shell/test_geolocation_service.cc
+++ /dev/null
@@ -1,67 +0,0 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "webkit/tools/test_shell/test_geolocation_service.h"
-
-#include "third_party/WebKit/WebKit/chromium/public/WebGeolocationServiceBridge.h"
-
-TestGeolocationService::TestGeolocationService()
- : allowed_(false),
- permission_set_(false) {
-}
-
-TestGeolocationService::~TestGeolocationService() {
- for (IDMap<WebKit::WebGeolocationServiceBridge>::iterator it(&bridges_map_);
- !it.IsAtEnd(); it.Advance()) {
- it.GetCurrentValue()->onWebGeolocationServiceDestroyed();
- }
-}
-
-void TestGeolocationService::SetGeolocationPermission(bool allowed) {
- allowed_ = allowed;
- permission_set_ = true;
- TryToSendPermissions();
-}
-
-void TestGeolocationService::requestPermissionForFrame(
- int bridgeId, const WebKit::WebURL& url) {
- DCHECK(bridges_map_.Lookup(bridgeId)) << "Unknown bridge " << bridgeId;
- pending_permissions_.push_back(bridgeId);
- TryToSendPermissions();
-}
-
-int TestGeolocationService::attachBridge(
- WebKit::WebGeolocationServiceBridge* bridge) {
- return bridges_map_.Add(bridge);
-}
-
-void TestGeolocationService::detachBridge(int bridgeId) {
- bridges_map_.Remove(bridgeId);
- std::vector<int>::iterator i = pending_permissions_.begin();
- while (i != pending_permissions_.end()) {
- if (*i == bridgeId)
- pending_permissions_.erase(i);
- else
- ++i;
- }
-}
-
-void TestGeolocationService::TryToSendPermissions() {
- if (permission_set_ && !permission_timer_.IsRunning())
- permission_timer_.Start(base::TimeDelta::FromMilliseconds(0),
- this, &TestGeolocationService::SendPermission);
-}
-
-void TestGeolocationService::SendPermission() {
- DCHECK(permission_set_);
- std::vector<int> pending_permissions;
- pending_permissions.swap(pending_permissions_);
- for (std::vector<int>::const_iterator i = pending_permissions.begin();
- i != pending_permissions.end(); ++i) {
- WebKit::WebGeolocationServiceBridge* bridge =
- bridges_map_.Lookup(*i);
- DCHECK(bridge);
- bridge->setIsAllowed(allowed_);
- }
-}

Powered by Google App Engine
This is Rietveld 408576698