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

Unified Diff: blimp/common/logging.cc

Issue 2091023006: Adds EngineGeolocationFeature for Blimp Geolocation project. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/common/logging.cc
diff --git a/blimp/common/logging.cc b/blimp/common/logging.cc
index 24d6060adb37028eca7f1d7933fa481ef8e7c26b..120163e9568918688eded61c2f023a177331aa17 100644
--- a/blimp/common/logging.cc
+++ b/blimp/common/logging.cc
@@ -300,6 +300,45 @@ void ExtractBlobChannelMessageFields(const BlimpMessage& message,
}
}
+// Logs fields from GEOLOCATION messages.
+void ExtractGeolocationMessageFields(const BlimpMessage& message,
+ LogFields* output) {
+ switch (message.geolocation().type_case()) {
+ case GeolocationMessage::TypeCase::kUpdateListenState:
+ AddField("subtype", "UPDATE_LISTEN_STATE_MESSAGE", output);
Kevin M 2016/06/27 16:57:40 Remove _MESSAGE suffix
CJ 2016/06/27 22:04:26 Done.
+ AddField("listen_state",
+ message.geolocation().update_listen_state().listen_state(),
+ output);
+ break;
+ case GeolocationMessage::TypeCase::kRequestRefresh:
+ AddField("subtype", "REQUEST_REFRESH", output);
+ break;
+ case GeolocationMessage::TypeCase::kLocation:
+ AddField("subtype", "LOCATION", output);
+ AddField("latitude", message.geolocation().location().latitude(), output);
+ AddField("longitude", message.geolocation().location().longitude(),
+ output);
+ AddField("altitude", message.geolocation().location().altitude(), output);
+ AddField("accuracy", message.geolocation().location().accuracy(), output);
+ AddField("altitude_accuracy",
+ message.geolocation().location().altitude_accuracy(), output);
+ AddField("heading", message.geolocation().location().heading(), output);
+ AddField("speed", message.geolocation().location().speed(), output);
+ AddField("timestamp_millis",
+ message.geolocation().location().timestamp_millis(), output);
+ break;
+ case GeolocationMessage::TypeCase::kError:
+ AddField("subtype", "ERROR", output);
+ AddField("error_code", message.geolocation().error().error_code(),
+ output);
+ AddField("error_message", message.geolocation().error().error_message(),
+ output);
+ break;
+ case GeolocationMessage::TypeCase::TYPE_NOT_SET:
+ break;
+ }
+}
+
} // namespace
std::ostream& operator<<(std::ostream& out, const BlimpMessage& message) {
@@ -342,6 +381,10 @@ std::ostream& operator<<(std::ostream& out, const BlimpMessage& message) {
AddField("type", "IME", &fields);
ExtractImeMessageFields(message, &fields);
break;
+ case BlimpMessage::kGeolocation:
+ AddField("type", "GEOLOCATION", &fields);
+ ExtractGeolocationMessageFields(message, &fields);
+ break;
case BlimpMessage::FEATURE_NOT_SET:
AddField("type", "<UNKNOWN>", &fields);
break;

Powered by Google App Engine
This is Rietveld 408576698