GN(android) Fix oscilloscope crash in webaudio on arm64

GN(android) Fix oscilloscope crash in webaudio on arm64

TBR=dpranke
BUG=578209
Committed: https://crrev.com/c6458dafbea2ec536e8f9a9793f9af47600ef743
Cr-Commit-Position: refs/heads/master@{#369941}

[agrieve, 2016-01-16 02:24:48]

agrieve@chromium.org changed reviewers:
+ rtoy@chromium.org

[agrieve, 2016-01-16 02:24:49]

Verified on my device that this fixes the crash.

[agrieve, 2016-01-16 02:25:07]

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

[commit-bot: I haz the power, 2016-01-16 02:25:15]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1595813002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1595813002/1

[Raymond Toy, 2016-01-16 03:12:34]

lgtm, with a nit.

Thanks so much for fixing this!

https://codereview.chromium.org/1595813002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/config.gni (right):

https://codereview.chromium.org/1595813002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/config.gni:31: # enabled. Whether WebAudio is actually
available depends on runtime settings
Can you update these comments to say arm64 and mipsel are also supported?

[agrieve, 2016-01-16 03:18:09]

https://codereview.chromium.org/1595813002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/config.gni (right):

https://codereview.chromium.org/1595813002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/config.gni:31: # enabled. Whether WebAudio is actually
available depends on runtime settings
On 2016/01/16 03:12:34, Raymond Toy wrote:
> Can you update these comments to say arm64 and mipsel are also supported?

Done.

[agrieve, 2016-01-16 03:18:26]

Description was changed from

==========
GN(android) Fix oscilloscope crash in webaudio on arm64

BUG=578209
==========

to

==========
GN(android) Fix oscilloscope crash in webaudio on arm64

TBR=dpranke
BUG=578209
==========

[agrieve, 2016-01-16 03:18:58]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2016-01-16 03:18:58]

The patchset sent to the CQ was uploaded after l-g-t-m from rtoy@chromium.org
Link to the patchset: https://codereview.chromium.org/1595813002/#ps20001
(title: "update comment")

[commit-bot: I haz the power, 2016-01-16 03:19:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1595813002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1595813002/20001

[commit-bot: I haz the power, 2016-01-16 05:10:57]

Description was changed from

==========
GN(android) Fix oscilloscope crash in webaudio on arm64

TBR=dpranke
BUG=578209
==========

to

==========
GN(android) Fix oscilloscope crash in webaudio on arm64

TBR=dpranke
BUG=578209
==========

[commit-bot: I haz the power, 2016-01-16 05:10:58]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-16 05:12:19]

Description was changed from

==========
GN(android) Fix oscilloscope crash in webaudio on arm64

TBR=dpranke
BUG=578209
==========

to

==========
GN(android) Fix oscilloscope crash in webaudio on arm64

TBR=dpranke
BUG=578209

Committed: https://crrev.com/c6458dafbea2ec536e8f9a9793f9af47600ef743
Cr-Commit-Position: refs/heads/master@{#369941}
==========

[commit-bot: I haz the power, 2016-01-16 05:12:19]

Patchset 2 (id:??) landed as
https://crrev.com/c6458dafbea2ec536e8f9a9793f9af47600ef743
Cr-Commit-Position: refs/heads/master@{#369941}

[Dirk Pranke, 2016-01-16 19:40:50]

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

[Dirk Pranke, 2016-01-16 19:40:51]

lgtm