Vibration cannot be canceled during pattern vibration.

Source/modules/vibration/NavigatorVibration.cpp

Index: Source/modules/vibration/NavigatorVibration.cpp
diff --git a/Source/modules/vibration/NavigatorVibration.cpp b/Source/modules/vibration/NavigatorVibration.cpp
index bba89fcadf195bece96040d44363d7013086ccc3..ee9ae29ce65340d14fb1fca6094f5e14ddc74b51 100644
--- a/Source/modules/vibration/NavigatorVibration.cpp
+++ b/Source/modules/vibration/NavigatorVibration.cpp
@@ -85,6 +85,7 @@ bool NavigatorVibration::vibrate(const VibrationPattern& pattern)
     }
 
     m_timerStart.startOneShot(0);
+    m_isVibrating = true;

[tkent, 2013/08/28 03:58:07]

It seems m_isVibrating is equivalent to m_timerStart.isActive() ||
m_timerStop.isActive() (if we apply the following comment).  We can remove
m_isVibrating.

[kihong, 2013/08/28 05:25:25]

It's not exactly same because if there are some codes like below, vibration can
not be canceled becuase of timer issue.
m_timerStart.startOneShort(0); is called by line:1 but timer could not be
activated until line:2 is called. Therefore cancel would be not worked.
Then device would be vibrating when timer is activated.
Therefore IMHO timer is not enough to check vibrating or not.

below ---------------------------
1: navigator.vibrate(1000);
2: navigator.vibrate(0);

     return true;
 }
 
@@ -114,7 +115,8 @@ void NavigatorVibration::timerStopFired(Timer<NavigatorVibration>* timer)
 {
     ASSERT_UNUSED(timer, timer == &m_timerStop);
 
-    m_isVibrating = false;
+    if (m_pattern.isEmpty())
+        m_isVibrating = false;
 
     if (m_pattern.size()) {
         m_timerStart.startOneShot(m_pattern[0] / 1000.0);