Refactor and re-implement Stopwatch API. Write tests for it.
Created attachment 797424 [details] [review] https://github.com/mozilla-b2g/gaia/pull/11846 New Stopwatch API start() starts the stopwatch getElapsedTime() gets a Date object storing the duration since starting the stopwatch pause() pauses the stopwatch lap() returns a Date object storing the duration since the last lap, or if no last lap, since starting the stopwatch reset() sets the stopwatch's elapsed time back to 0 Tests tested the API functions in their own suites, leaving out start() since its encompassed by the test for getElapsedTime() Questions not sure if the view needs the model to return a list of all lap times, can the view handle that list itself?
Comment on attachment 797424 [details] [review] https://github.com/mozilla-b2g/gaia/pull/11846 This is a good first pass, Evelyn! I've left comments on the pull request on GitHub. Please re-set the "review" flag when you'd like me to take another look.
Wooh, thanks, captain! Will do!
Comment on attachment 797424 [details] [review] https://github.com/mozilla-b2g/gaia/pull/11846 Looking good, Evelyn! I'll merge this post-haste
master: https://github.com/mozilla-b2g/gaia/commit/6879d212d8f0030d64e30441dd4cdfc16415478e Nice work :)
Comment on attachment 797424 [details] [review] https://github.com/mozilla-b2g/gaia/pull/11846 Looks like I forgot to set the review flag on the patch (which has since landed in master). Setting it now for bookkeeping purposes.