Improve audible token detection.

Improve audible token detection.
The current code doesn't differentiate between tokens decoded by the DSSS
(inaudible) or DTMF (audible) encoder/decoder. Fix this and send up the
correct medium to the server.

R=kalman@chromium.org, xiyuan@chromium.org
BUG=402334
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288872

[not at google - send to devlin, 2014-08-11 19:52:55]

extensions lgtm

[xiyuan, 2014-08-11 19:58:21]

lgtm

https://codereview.chromium.org/460743004/diff/1/chrome/browser/resources/whi...
File chrome/browser/resources/whispernet_proxy/js/wrapper.js (right):

https://codereview.chromium.org/460743004/diff/1/chrome/browser/resources/whi...
chrome/browser/resources/whispernet_proxy/js/wrapper.js:193: * decoder or not.
nit: 4-space indent (aligned with @param)

[rkc, 2014-08-11 20:12:21]

https://codereview.chromium.org/460743004/diff/1/chrome/browser/resources/whi...
File chrome/browser/resources/whispernet_proxy/js/wrapper.js (right):

https://codereview.chromium.org/460743004/diff/1/chrome/browser/resources/whi...
chrome/browser/resources/whispernet_proxy/js/wrapper.js:193: * decoder or not.
On 2014/08/11 19:58:21, xiyuan wrote:
> nit: 4-space indent (aligned with @param)

Done.

[rkc, 2014-08-11 20:12:24]

The CQ bit was checked by rkc@chromium.org

[commit-bot: I haz the power, 2014-08-11 20:14:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rkc@chromium.org/460743004/20001

[xiyuan, 2014-08-11 20:17:28]

You might need to manually land "whispernet_proxy_pnacl.pexe.png" since it's a
binary file.

[rkc, 2014-08-11 20:34:44]

On 2014/08/11 20:17:28, xiyuan wrote:
> You might need to manually land "whispernet_proxy_pnacl.pexe.png" since it's a
> binary file.

I believe they've fixed that,
http://www.chromium.org/developers/testing/commit-queue#TOC-Binary-files-

These are fails because I forgot to update all the tests :)

[commit-bot: I haz the power, 2014-08-11 21:35:41]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_chromium_gn_compile_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-11 21:44:13]

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

[commit-bot: I haz the power, 2014-08-11 21:44:15]

Try jobs failed on following builders:
  mac_chromium_rel_swarming on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[rkc, 2014-08-11 22:23:32]

The CQ bit was checked by rkc@chromium.org

[commit-bot: I haz the power, 2014-08-11 22:28:35]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rkc@chromium.org/460743004/40001

[rkc, 2014-08-12 01:59:00]

Committed patchset #3 manually as 288872 (presubmit successful).