Add ChromeVox braille_util_test to chromevox_tests.

Add ChromeVox braille_util_test to chromevox_tests.

Based on work by dmazzoni@chromium.org in
https://codereview.chromium.org/339923002/
with the addition of support for including the English text messages in
tests and other minor fixes.

BUG=371692
R=dmazzoni@chromium.org

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

[dmazzoni, 2014-06-19 23:28:27]

lgtm

Thanks!

[commit-bot: I haz the power, 2014-06-19 23:30:49]

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

[commit-bot: I haz the power, 2014-06-20 07:57:01]

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

[commit-bot: I haz the power, 2014-06-20 13:16:48]

Change committed as 278694

[dmazzoni, 2014-06-23 07:34:21]

This seems to be failing when I check out a fresh build and run these
tests. Could you take a look?


On Fri, Jun 20, 2014 at 6:16 AM, <commit-bot@chromium.org> wrote:

> Change committed as 278694
>
> https://codereview.chromium.org/342153003/
>

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

[Peter Lundblad, 2014-06-23 23:25:37]

Hi,

Sent out a fix for this. https://codereview.chromium.org/350653006

I don't know why my local run was with messages from the previous
upstream.  I think getting this new test target on the bots will be
helpful :-)

Thanks,
//Peter

Dominic Mazzoni writes:
> This seems to be failing when I check out a fresh build and run these tests.
> Could you take a look?
> 
> 
> On Fri, Jun 20, 2014 at 6:16 AM, <commit-bot@chromium.org> wrote:
> 
>     Change committed as 278694
> 
>     https://codereview.chromium.org/342153003/
> 
> 

-- 

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

[dmazzoni, 2014-06-24 05:54:31]

Thanks. I'll send out the patch to add this test target to the bots now. :)


On Mon, Jun 23, 2014 at 4:25 PM, <plundblad@chromium.org> wrote:

> Hi,
>
> Sent out a fix for this. https://codereview.chromium.org/350653006
>
> I don't know why my local run was with messages from the previous
> upstream.  I think getting this new test target on the bots will be
> helpful :-)
>
> Thanks,
> //Peter
>
> Dominic Mazzoni writes:
> > This seems to be failing when I check out a fresh build and run these
> tests.
> > Could you take a look?
> >
> >
> > On Fri, Jun 20, 2014 at 6:16 AM, <commit-bot@chromium.org> wrote:
> >
> >     Change committed as 278694
> >
> >     https://codereview.chromium.org/342153003/
> >
> >
>
> --
>

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