Fix a dumb error introduced earlier. We need to call Stop only when the wait ...

Fix a dumb error introduced earlier. We need to call Stop only when the wait on the python server
returns an error

TBR=amit


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

[amit, 2010-08-20 20:01:49]

ok

On Fri, Aug 20, 2010 at 12:25 PM, <ananta@chromium.org> wrote:

> Reviewers: amit,
>
> Description:
> Fix a dumb error introduced earlier. We need to call Stop only when the
> wait on
> the python server
> returns an error
>
> TBR=amit
>
>
> Please review this at http://codereview.chromium.org/3162028/show
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
>
> Affected files:
>  M     chrome_frame/test/http_server.cc
>
>
> Index: chrome_frame/test/http_server.cc
> ===================================================================
> --- chrome_frame/test/http_server.cc    (revision 56883)
> +++ chrome_frame/test/http_server.cc    (working copy)
> @@ -67,7 +67,6 @@
>   bool ret = test_server_.WaitToFinish(milliseconds);
>   if (!ret) {
>     LOG(ERROR) << "WaitToFinish failed with error:" << ::GetLastError();
> -  } else {
>     ret = test_server_.Stop();
>   }
>   return ret;
>
>
>