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

Unified Diff: content/browser/geolocation/libgps_2_94_wrapper_linux.cc

Issue 8171006: Fix gps.h license issue, by deleting it (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update checklicenses.py Created 9 years, 2 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: content/browser/geolocation/libgps_2_94_wrapper_linux.cc
diff --git a/content/browser/geolocation/libgps_2_94_wrapper_linux.cc b/content/browser/geolocation/libgps_2_94_wrapper_linux.cc
index c9fb1bd43b031c38e89f4e2d174c21031feccec9..9d213b843d255e6b00f89d9485f2d77d24abd3bf 100644
--- a/content/browser/geolocation/libgps_2_94_wrapper_linux.cc
+++ b/content/browser/geolocation/libgps_2_94_wrapper_linux.cc
@@ -32,6 +32,11 @@ class LibGpsV294 : public LibGps {
position->latitude = gps_data.fix.latitude;
position->longitude = gps_data.fix.longitude;
position->accuracy = std::max(gps_data.fix.epx, gps_data.fix.epy);
+ if (position->accuracy != position->accuracy) {
John Knottenbelt 2011/10/06 16:26:56 Did you mean to include this change in this patch
joth 2011/10/06 19:15:58 Done.
+ // TODO(joth): Fixme. This is a workaround for http://crbug.com/99326
+ LOG(WARNING) << "libgps reported accuracy NaN, forcing to zero";
+ position->accuracy = 0;
+ }
position->altitude = gps_data.fix.altitude;
position->altitude_accuracy = gps_data.fix.epv;
position->heading = gps_data.fix.track;
« no previous file with comments | « content/browser/geolocation/libgps_2_38_wrapper_linux.cc ('k') | content/browser/geolocation/libgps_wrapper_linux.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698