Fix bug in city-list autoscroll

Fix bug in city-list autoscroll

TBR=esprehn

Committed: https://chromium.googlesource.com/external/mojo/+/0815d480090be7d94964cabaaa99587bb232b9ce

[rafaelw, 2014-11-25 02:27:04]

rafaelw@chromium.org changed reviewers:
+ esprehn@chromium.org

[rafaelw, 2014-11-25 02:28:05]

Committed patchset #1 (id:1) manually as
0815d480090be7d94964cabaaa99587bb232b9ce (presubmit successful).

[abarth-chromium, 2014-11-25 02:34:55]

abarth@chromium.org changed reviewers:
+ abarth@chromium.org

[abarth-chromium, 2014-11-25 02:34:56]

lgtm

[esprehn, 2014-11-25 07:50:21]

https://codereview.chromium.org/759483003/diff/1/sky/examples/city-list/index...
File sky/examples/city-list/index.sky (right):

https://codereview.chromium.org/759483003/diff/1/sky/examples/city-list/index...
sky/examples/city-list/index.sky:34: var scrollBy =
Number(location.search.match(/auto=([0-9]+)/)[1]);
match returns null if it doesn't match anything, this is going to throw an
exception when you index into null.

[rafaelw, 2014-11-28 04:22:49]

https://codereview.chromium.org/759483003/diff/1/sky/examples/city-list/index...
File sky/examples/city-list/index.sky (right):

https://codereview.chromium.org/759483003/diff/1/sky/examples/city-list/index...
sky/examples/city-list/index.sky:34: var scrollBy =
Number(location.search.match(/auto=([0-9]+)/)[1]);
On 2014/11/25 07:50:21, esprehn wrote:
> match returns null if it doesn't match anything, this is going to throw an
> exception when you index into null.

Woops. Good catch. Fixed: https://codereview.chromium.org/760403002/