Reenable CaptivePortalBrowserTest.TwoBrokenTabs

Reenable CaptivePortalTest.TwoBrokenTabs

BUG=141497
TEST=test works again


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150817

[jochen (gone - plz use gerrit), 2012-08-09 13:21:48]

plz review

should work again after https://chromiumcodereview.appspot.com/10826224/ has
landed

[mmenke, 2012-08-09 14:34:50]

On 2012/08/09 13:21:48, jochen wrote:
> plz review
> 
> should work again after https://chromiumcodereview.appspot.com/10826224/ has
> landed

LGTM, thanks so much for figuring out that fixes my test.

[commit-bot: I haz the power, 2012-08-09 15:32:26]

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

[commit-bot: I haz the power, 2012-08-09 16:53:23]

Change committed as 150817

[jianli, 2012-08-09 20:14:23]

CaptivePortalTest.TwoBrokenTabs is still flaky.

http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...


On Thu, Aug 9, 2012 at 9:53 AM, <commit-bot@chromium.org> wrote:

> Change committed as 150817
>
>
https://chromiumcodereview.**appspot.com/10855074/<https://chromiumcodereview...
>

[mmenke, 2012-08-09 20:26:29]

Yup, thanks for pointing it out.  And I see it's failed on another CQ run for a
CL I reviewed.  I'll revert this and re-disable the test.

For future reference, probably better to post on the bug, in most cases.

On 2012/08/09 20:14:23, jianli wrote:
> CaptivePortalTest.TwoBrokenTabs is still flaky.
> 
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...
> 
> 
> On Thu, Aug 9, 2012 at 9:53 AM, <mailto:commit-bot@chromium.org> wrote:
> 
> > Change committed as 150817
> >
> >
>
https://chromiumcodereview.**appspot.com/10855074/%3Chttps://chromiumcoderevi...>
> >

[mmenke, 2012-08-09 20:30:41]

Actually, it's failing everywhere, so rather than revert, I'll disable
everywhere.