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

Unified Diff: blimp/common/logging.cc

Issue 1858043003: [Blimp Net] logs input message type. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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
« no previous file with comments | « no previous file | blimp/common/logging_unittest.cc » ('j') | blimp/common/logging_unittest.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/common/logging.cc
diff --git a/blimp/common/logging.cc b/blimp/common/logging.cc
index c115f56db6315557901a882e83904b3b981aca02..07954c6c237ee1a0802855b174a1b75c951c851a 100644
--- a/blimp/common/logging.cc
+++ b/blimp/common/logging.cc
@@ -147,6 +147,41 @@ class InputLogExtractor : public LogExtractor {
void ExtractFields(const BlimpMessage& message,
LogFields* output) const override {
AddField("render_widget_id", message.input().render_widget_id(), output);
+ AddField("timestamp_seconds", message.input().timestamp_seconds(), output);
+ switch (message.input().type()) {
+ case InputMessage::Type_GestureScrollBegin:
+ AddField("subtype", "GestureScrollBegin", output);
+ break;
+ case InputMessage::Type_GestureScrollEnd:
+ AddField("subtype", "GestureScrollEnd", output);
+ break;
+ case InputMessage::Type_GestureScrollUpdate:
+ AddField("subtype", "GestureScrollUpdate", output);
+ break;
+ case InputMessage::Type_GestureFlingStart:
+ AddField("subtype", "GestureFlingStart", output);
+ break;
+ case InputMessage::Type_GestureFlingCancel:
+ AddField("subtype", "GestureFlingCancel", output);
+ AddField("prevent_boosting",
+ message.input().gesture_fling_cancel().prevent_boosting(),
+ output);
+ break;
+ case InputMessage::Type_GestureTap:
+ AddField("subtype", "GestureTap", output);
+ break;
+ case InputMessage::Type_GesturePinchBegin:
+ AddField("subtype", "GesturePinchBegin", output);
+ break;
+ case InputMessage::Type_GesturePinchEnd:
+ AddField("subtype", "GesturePinchEnd", output);
+ break;
+ case InputMessage::Type_GesturePinchUpdate:
+ AddField("subtype", "GesturePinchUpdate", output);
+ break;
+ default: // unknown
+ break;
+ }
}
};
« no previous file with comments | « no previous file | blimp/common/logging_unittest.cc » ('j') | blimp/common/logging_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698