Fix crash when calling getMsg with no options.

Fix crash when calling getMsg with no options.

(This is cherry-picking a single bug fix from ChromeVox upstream.)

The repro is this html:
<div tabIndex=0 role="slider" aria-valuenow="20"></div>

The bug happens when cvox.BrailleUtil.getState calls
cvox.ChromeVox.msgs.getMsg to see if the "_brl" translation exists,
and passes it 'aria_value_now_brl' in this particular case,
whose value is just "$1". If you don't pass it an argument, you
get '' back, and that throws an error because a message should never
return the empty string.

BUG=379296
R=plundblad@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274815

[chromium-reviews, 2014-05-30 23:25:44]

LGTM

FWIW, I think this should affect the background page as well, but presumably
not
in a way that matters.

Thanks,
//Peter

-- 

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[dmazzoni, 2014-06-02 19:26:26]

The CQ bit was checked by dmazzoni@chromium.org

[commit-bot: I haz the power, 2014-06-02 19:28:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dmazzoni@chromium.org/302063005/1

[commit-bot: I haz the power, 2014-06-02 19:28:42]

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

[commit-bot: I haz the power, 2014-06-02 19:28:42]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional 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.

[Peter Lundblad, 2014-06-02 19:33:34]

lgtm

[Peter Lundblad, 2014-06-02 19:34:41]

The CQ bit was checked by plundblad@chromium.org

[commit-bot: I haz the power, 2014-06-02 19:36:31]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dmazzoni@chromium.org/302063005/1

[commit-bot: I haz the power, 2014-06-02 20:20:00]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-06-02 20:24:06]

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

[commit-bot: I haz the power, 2014-06-02 20:24:07]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[Peter Lundblad, 2014-06-04 01:13:40]

The CQ bit was checked by plundblad@chromium.org

[commit-bot: I haz the power, 2014-06-04 01:14:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dmazzoni@chromium.org/302063005/1

[commit-bot: I haz the power, 2014-06-04 01:19:17]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-06-04 01:22:52]

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

[commit-bot: I haz the power, 2014-06-04 01:22:53]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[dmazzoni, 2014-06-04 14:08:07]

Committed patchset #1 manually as r274815 (presubmit successful).