Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Issue 3156039: Attempting to debug the IE8 chrome frame random failures by tracing out the l... (Closed)

Created:
10 years, 4 months ago by ananta
Modified:
9 years, 6 months ago
Reviewers:
amit
CC:
chromium-reviews, amit
Visibility:
Public.

Description

Attempting to debug the IE8 chrome frame random failures by tracing out the last error code from the net::TestServer::WaitToFinish helper function. TBR=amit Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56817

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M chrome_frame/test/http_server.cc View 1 chunk +5 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
ananta
10 years, 4 months ago (2010-08-20 05:11:59 UTC) #1
amit
10 years, 4 months ago (2010-08-20 15:58:06 UTC) #2
ok

On Thu, Aug 19, 2010 at 10:11 PM, <ananta@chromium.org> wrote:

> Reviewers: amit,
>
> Description:
> Attempting to debug the IE8 chrome frame random failures by tracing out the
> last
> error
> code from the net::TestServer::WaitToFinish helper function.
>
> TBR=amit
>
>
> Please review this at http://codereview.chromium.org/3156039/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 56679)
> +++ chrome_frame/test/http_server.cc    (working copy)
> @@ -64,7 +64,11 @@
>  }
>
>  bool ChromeFrameHTTPServer::WaitToFinish(int milliseconds) {
> -  return test_server_.WaitToFinish(milliseconds);
> +  bool ret = test_server_.WaitToFinish(milliseconds);
> +  if (!ret) {
> +    LOG(ERROR) << "WaitToFinish failed with error:" << ::GetLastError();
> +  }
> +  return ret;
>  }
>
>  // TODO(phajdan.jr): Change wchar_t* to std::string& and fix callers.
>
>
>

Powered by Google App Engine
This is Rietveld 408576698