Fix the crash of issue 2209773002

Fix two crashes caused by crrev.com/413616

BUG=640150, 640494
Committed: https://crrev.com/41928a8e0daf09725e61d36d35905c28e74716d9
Cr-Commit-Position: refs/heads/master@{#414403}

[sunyunjia, 2016-08-23 13:56:18]

Description was changed from

==========
Fix the crash in fuzzer

BUG=640150
==========

to

==========
Fix the crash in fuzzer, see link
https://bugs.chromium.org/p/chromium/issues/detail?id=640150

BUG=640150
==========

[sunyunjia, 2016-08-23 13:56:31]

The CQ bit was checked by sunyunjia@chromium.org

[commit-bot: I haz the power, 2016-08-23 13:56:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-23 13:56:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-23 13:56:49]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[sunyunjia, 2016-08-23 14:03:41]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-23 14:03:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sunyunjia, 2016-08-23 14:06:58]

sunyunjia@chromium.org changed reviewers:
+ tkent@chromium.org

[sunyunjia, 2016-08-23 14:06:59]

Changed a line of code to fix the crash. PTAL, Thanks!

[commit-bot: I haz the power, 2016-08-23 15:59:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-23 15:59:33]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tkent, 2016-08-23 23:02:12]

please add a test

[sunyunjia, 2016-08-24 14:23:16]

Added a test. PTAL, Thanks!

[sunyunjia, 2016-08-24 14:28:15]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-24 14:28:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-24 16:34:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-24 16:34:35]

Dry run: This issue passed the CQ dry run.

[sunyunjia, 2016-08-24 23:00:49]

Description was changed from

==========
Fix the crash in fuzzer, see link
https://bugs.chromium.org/p/chromium/issues/detail?id=640150

BUG=640150
==========

to

==========
Fix the crash of issue 2209773002

Fix the crash in fuzzer, see link
https://bugs.chromium.org/p/chromium/issues/detail?id=640150

Fix the crash in android, see link
https://bugs.chromium.org/p/chromium/issues/detail?id=640494

BUG=640150, 640494
==========

[sunyunjia, 2016-08-24 23:01:23]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-24 23:02:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-08-25 00:43:24]

Description was changed from

==========
Fix the crash of issue 2209773002

Fix the crash in fuzzer, see link
https://bugs.chromium.org/p/chromium/issues/detail?id=640150

Fix the crash in android, see link
https://bugs.chromium.org/p/chromium/issues/detail?id=640494

BUG=640150, 640494
==========

to

==========
Fix two crashes caused by crrev.com/413616

BUG=640150, 640494
==========

[tkent, 2016-08-25 00:43:56]

C++ code change looks ok.

https://codereview.chromium.org/2270833002/diff/60001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html
(right):

https://codereview.chromium.org/2270833002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:32:
if (isMac) {
Why do you exclude Mac?

https://codereview.chromium.org/2270833002/diff/60001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:37:
var w = slider.clientWidth;
Wrong indentation.

[commit-bot: I haz the power, 2016-08-25 01:00:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-25 01:00:00]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sunyunjia, 2016-08-25 02:48:53]

https://codereview.chromium.org/2270833002/diff/60001/third_party/WebKit/Layo...
>
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:32:
> if (isMac) {
> Why do you exclude Mac?

Because Mac doesn't have touch event

https://codereview.chromium.org/2270833002/diff/60001/third_party/WebKit/Layo...
>
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:37:
> var w = slider.clientWidth;
> Wrong indentation.

Done.

[tkent, 2016-08-25 03:29:52]

lgtm

https://codereview.chromium.org/2270833002/diff/80001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html
(right):

https://codereview.chromium.org/2270833002/diff/80001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:35:
var slider = document.getElementById('slider');
Wrong indentation.

[sunyunjia, 2016-08-25 03:38:24]

The CQ bit was checked by sunyunjia@chromium.org

[sunyunjia, 2016-08-25 03:38:25]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2270833002/#ps100001
(title: "indentation")

[commit-bot: I haz the power, 2016-08-25 03:39:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-08-25 03:40:07]

https://codereview.chromium.org/2270833002/diff/100001/third_party/WebKit/Lay...
File
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html
(right):

https://codereview.chromium.org/2270833002/diff/100001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:32:
if (isMac) {
indentation is inconsistent with other functions, which apply 2-space
indentation.

https://codereview.chromium.org/2270833002/diff/100001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:41:
chrome.gpuBenchmarking.smoothDrag(x, y, x + w, y, function() {},
touchSourceType);
Why did you remove .done() call?

[sunyunjia, 2016-08-25 04:29:09]

The CQ bit was unchecked by sunyunjia@chromium.org

[sunyunjia, 2016-08-25 04:37:57]

https://codereview.chromium.org/2270833002/diff/100001/third_party/WebKit/Lay...
> File
https://codereview.chromium.org/2270833002/diff/100001/third_party/WebKit/Lay...
>
third_party/WebKit/LayoutTests/fast/events/touch/touch-action-range-input-crash.html:41:
> chrome.gpuBenchmarking.smoothDrag(x, y, x + w, y, function() {},
> touchSourceType);
> Why did you remove .done() call?

Sorry my bad. That was by mistake.

[sunyunjia, 2016-08-25 04:38:06]

The CQ bit was checked by sunyunjia@chromium.org

[sunyunjia, 2016-08-25 04:38:06]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2270833002/#ps120001
(title: "Indentation and get .done() function back")

[commit-bot: I haz the power, 2016-08-25 04:38:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 04:45:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-25 04:45:39]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[sunyunjia, 2016-08-25 04:48:16]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-25 04:48:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 06:45:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-25 06:45:29]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sunyunjia, 2016-08-25 06:46:27]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-25 06:46:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sunyunjia, 2016-08-25 06:46:51]

The CQ bit was unchecked by sunyunjia@chromium.org

[sunyunjia, 2016-08-25 06:50:14]

The CQ bit was checked by sunyunjia@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-25 06:50:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 08:51:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-25 08:51:53]

Dry run: This issue passed the CQ dry run.

[sunyunjia, 2016-08-25 12:11:03]

The CQ bit was checked by sunyunjia@chromium.org

[sunyunjia, 2016-08-25 12:11:03]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2270833002/#ps160001
(title: "indentation")

[commit-bot: I haz the power, 2016-08-25 12:11:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 12:16:33]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2016-08-25 12:20:00]

Description was changed from

==========
Fix two crashes caused by crrev.com/413616

BUG=640150, 640494
==========

to

==========
Fix two crashes caused by crrev.com/413616

BUG=640150, 640494

Committed: https://crrev.com/41928a8e0daf09725e61d36d35905c28e74716d9
Cr-Commit-Position: refs/heads/master@{#414403}
==========

[commit-bot: I haz the power, 2016-08-25 12:20:02]

Patchset 9 (id:??) landed as
https://crrev.com/41928a8e0daf09725e61d36d35905c28e74716d9
Cr-Commit-Position: refs/heads/master@{#414403}