Chromium Code Reviews Chromium Code Reviews
Issue 15724023:
Vibration API: use runtime flag, change from client to platform. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2012 Samsung Electronics | 2 * Copyright (C) 2012 Samsung Electronics |
| 3 * | 3 * |
| 4 * This library is free software; you can redistribute it and/or | 4 * This library is free software; you can redistribute it and/or |
| 5 * modify it under the terms of the GNU Library General Public | 5 * modify it under the terms of the GNU Library General Public |
| 6 * License as published by the Free Software Foundation; either | 6 * License as published by the Free Software Foundation; either |
| 7 * version 2 of the License, or (at your option) any later version. | 7 * version 2 of the License, or (at your option) any later version. |
| 8 * | 8 * |
| 9 * This library is distributed in the hope that it will be useful, | 9 * This library is distributed in the hope that it will be useful, |
| 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of | 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of |
| 11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | 11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU |
| 12 * Library General Public License for more details. | 12 * Library General Public License for more details. |
| 13 * | 13 * |
| 14 * You should have received a copy of the GNU Library General Public License | 14 * You should have received a copy of the GNU Library General Public License |
| 15 * along with this library; see the file COPYING.LIB. If not, write to | 15 * along with this library; see the file COPYING.LIB. If not, write to |
| 16 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, | 16 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, |
| 17 * Boston, MA 02110-1301, USA. | 17 * Boston, MA 02110-1301, USA. |
| 18 */ | 18 */ |
| 19 | 19 |
| 20 #ifndef NavigatorVibration_h | 20 #ifndef NavigatorVibration_h |
| 21 #define NavigatorVibration_h | 21 #define NavigatorVibration_h |
| 22 | 22 |
| 23 #if ENABLE(VIBRATION) | |
| 24 | |
| 25 #include "core/dom/ExceptionCode.h" | |
| 26 #include "wtf/Vector.h" | 23 #include "wtf/Vector.h" |
| 27 | 24 |
| 28 namespace WebCore { | 25 namespace WebCore { |
| 29 | 26 |
| 30 class Navigator; | 27 class Navigator; |
| 31 class Uint32Array; | 28 class Uint32Array; |
|
Peter Beverloo
2013/06/11 13:50:12
nit: this forward declaration isn't necessary, it'
Michael van Ouwerkerk
2013/06/11 14:16:43
Done.
| |
| 32 | 29 |
| 33 class NavigatorVibration { | 30 class NavigatorVibration { |
| 34 public: | 31 public: |
| 35 typedef Vector<unsigned> VibrationPattern; | 32 typedef Vector<unsigned> VibrationPattern; |
| 36 | 33 |
| 37 static void vibrate(Navigator*, unsigned time, ExceptionCode&); | 34 static void vibrate(Navigator*, unsigned time); |
| 38 static void vibrate(Navigator*, const VibrationPattern&, ExceptionCode&); | 35 static void vibrate(Navigator*, const VibrationPattern&); |
| 39 | 36 |
| 40 private: | 37 private: |
| 41 NavigatorVibration(); | 38 NavigatorVibration(); |
|
Peter Beverloo
2013/06/11 13:50:12
Do we need the constructor and destructor? They'r
Michael van Ouwerkerk
2013/06/11 14:16:43
Let's just leave a private constructor then.
| |
| 42 ~NavigatorVibration(); | 39 ~NavigatorVibration(); |
| 43 }; | 40 }; |
| 44 | 41 |
| 45 } // namespace WebCore | 42 } // namespace WebCore |
| 46 | 43 |
| 47 #endif // ENABLE(VIBRATION) | |
| 48 | |
| 49 #endif // NavigatorVibration_h | 44 #endif // NavigatorVibration_h |
| 50 | |
| OLD | NEW |
