[drape] Draw a bearing arrow for the pedestrian speed mode.

Signed-off-by: Viktor Govako <viktor.govako@gmail.com>

Author: vng

libs/drape_frontend/my_position_controller.cpp

@@ -16,16 +16,19 @@
 #include <algorithm>
 #include <array>
 #include <chrono>
-#include <string>
-#include <vector>
 
 namespace df
 {
 namespace
 {
 int const kPositionRoutingOffsetY = 104;
-double const kMinSpeedThresholdMps = 2.8;  // 10 km/h
-double const kGpsBearingLifetimeSec = 5.0;
+
+// https://t.me/OrganicMapsRu/88317
+double const kMinSpeedThresholdMps = 0.7;  // for the pedestrian mode 2.5 km/h
+/// @todo Should depend on the _previous_ avg speed (say for the last 5 minutes).
+/// Bigger for cars (up to 30 seconds is ok, IMO) and lower for pedestrians.
+double const kGpsBearingLifetimeSec = 3.0;
+
 double const kMaxTimeInBackgroundSec = 60.0 * 60 * 30;  // 30 hours before starting detecting position again
 double const kMaxNotFollowRoutingTimeSec = 20.0;
 double const kMaxUpdateLocationInvervalSec = 30.0;
@@ -529,16 +532,19 @@ void MyPositionController::LoseLocation()
 
 void MyPositionController::OnCompassUpdate(location::CompassInfo const & info, ScreenBase const & screen)
 {
-  double const oldAzimut = GetDrawableAzimut();
   m_isCompassAvailable = true;
 
-  bool const existsFreshGpsBearing = m_lastGPSBearingTimer.ElapsedSeconds() < kGpsBearingLifetimeSec;
-  if ((IsInRouting() && m_isArrowGluedInRouting) || existsFreshGpsBearing)
+  if (m_isArrowGluedInRouting && IsInRouting())
     return;
 
+  if (m_lastGPSBearingTimer.ElapsedSeconds() < kGpsBearingLifetimeSec)
+    return;
+
+  double const oldAzimut = GetDrawableAzimut();
+
   SetDirection(info.m_bearing);
 
-  if (m_isPositionAssigned && !AlmostCurrentAzimut(oldAzimut) && m_mode == location::FollowAndRotate)
+  if (m_isPositionAssigned && m_mode == location::FollowAndRotate && !AlmostCurrentAzimut(oldAzimut))
   {
     CreateAnim(GetDrawablePosition(), oldAzimut, screen);
     m_isDirtyViewport = true;