Revert debugging checks from SincResampler; no crashes found.

Revert debugging checks from SincResampler; no crashes found.

None of the crash checks were hit, so this seems to imply that
we're always dealing with non-nan values for the virtual index.

I'm not sure what's gone awry, but have simplified the while
loop here (and retested locally on ARM, x86, performance is
actually improved on modern ARM devices and slightly negative
on pre-64bit ARM devices). Results from media_perftests for
AudioConverterPerfTest.ConvertBenchmark

Z620 (X86): ~35% faster.
N5X (ARM64): ~3.7% faster.
N4 (ARM): ~0.81% slower.

BUG=663814
TEST=none

Committed: https://crrev.com/4147f786f38d0af6f19d62f73dc5dfb67d6c3a5a
Cr-Commit-Position: refs/heads/master@{#436852}

[DaleCurtis, 2016-12-07 00:49:00]

dalecurtis@chromium.org changed reviewers:
+ watk@chromium.org

[watk, 2016-12-07 01:30:23]

lgtm

[DaleCurtis, 2016-12-07 01:34:19]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2016-12-07 01:34:51]

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-12-07 03:41:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481074459456330, "parent_rev":
"d04b7e542b3abb22b7a4d1db1e4d25c60ba33c58", "commit_rev":
"1db823fc7ae771016033c893f7dcfcdcef73ec9f"}

[commit-bot: I haz the power, 2016-12-07 03:42:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 03:44:32]

Description was changed from

==========
Revert debugging checks from SincResampler; no crashes found.

None of the crash checks were hit, so this seems to imply that
we're always dealing with non-nan values for the virtual index.

I'm not sure what's gone awry, but have simplified the while
loop here (and retested locally on ARM, x86, performance is
actually improved on modern ARM devices and slightly negative
on pre-64bit ARM devices). Results from media_perftests for
AudioConverterPerfTest.ConvertBenchmark

Z620 (X86): ~35% faster.
N5X (ARM64): ~3.7% faster.
N4 (ARM): ~0.81% slower.

BUG=663814
TEST=none
==========

to

==========
Revert debugging checks from SincResampler; no crashes found.

None of the crash checks were hit, so this seems to imply that
we're always dealing with non-nan values for the virtual index.

I'm not sure what's gone awry, but have simplified the while
loop here (and retested locally on ARM, x86, performance is
actually improved on modern ARM devices and slightly negative
on pre-64bit ARM devices). Results from media_perftests for
AudioConverterPerfTest.ConvertBenchmark

Z620 (X86): ~35% faster.
N5X (ARM64): ~3.7% faster.
N4 (ARM): ~0.81% slower.

BUG=663814
TEST=none

Committed: https://crrev.com/4147f786f38d0af6f19d62f73dc5dfb67d6c3a5a
Cr-Commit-Position: refs/heads/master@{#436852}
==========

[commit-bot: I haz the power, 2016-12-07 03:44:33]

Patchset 1 (id:??) landed as
https://crrev.com/4147f786f38d0af6f19d62f73dc5dfb67d6c3a5a
Cr-Commit-Position: refs/heads/master@{#436852}