Vibration API: use runtime flag, change from client to platform.

public/platform/Platform.h

Index: public/platform/Platform.h
diff --git a/public/platform/Platform.h b/public/platform/Platform.h
index acff9862dc1663b4f3cc6a0b73606bc332a52457..2918a6d5fcea2a4cd36ae14bf12c88cce72a84e3 100644
--- a/public/platform/Platform.h
+++ b/public/platform/Platform.h
@@ -382,6 +382,9 @@ public:
     // Checks the partition/volume where fileName resides.
     virtual long long availableDiskSpaceInBytes(const WebString& fileName) { return 0; }
 
+    // Vibration -----------------------------------------------------------

[Peter Beverloo, 2013/06/11 10:19:10]

nit: blank line after this one.

[Michael van Ouwerkerk, 2013/06/11 13:26:51]

Done.

+    virtual void cancelVibration() { }
+    virtual void vibrate(unsigned time) { }

[Peter Beverloo, 2013/06/11 10:19:10]

I'd list "vibrate" before "cancelVibration", because cancelling is something you
can only do with an existing vibration (chronologically speaking).

Could you add basic one-line comments as to what these APIs do, despite the
clear naming?  The unit "time" has to be specified in would be useful
information, what happens with existing vibrations would be interesting,
etcetera.

[Michael van Ouwerkerk, 2013/06/11 13:26:51]

Done.

 
     // Testing -------------------------------------------------------------