Roll back Rietveld issue 8778 (r5183, appending stderr to layout-test output)...

webkit/tools/test_shell/test_webview_delegate.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file contains the implementation of TestWebViewDelegate, which serves
 // as the WebViewDelegate for the TestShellWebHost.  The host is expected to
 // have initialized a MessageLoop before these methods are called.
 
 #include "webkit/tools/test_shell/test_webview_delegate.h"
 
@@ skipping 64 matching lines @@
   DCHECK_NE(disposition, CURRENT_TAB);  // No code for this
   if (disposition == SUPPRESS_OPEN)
     return;
   TestShell* shell = NULL;
   if (TestShell::CreateNewWindow(UTF8ToWide(url.spec()), &shell))
     shell->Show(shell->webView(), disposition);
 }
 
 void TestWebViewDelegate::DidStartLoading(WebView* webview) {
   if (page_is_loading_) {
-    // When we started including stderr in layout-test output, a number of
-    // number of tests began failing. The vast majority of the new failures
-    // were due to either this error message or the one in DidStopLoading.
-    // This is being disabled temporarily so we can keep running the tests.
-    // See http://code.google.com/p/chromium/issues/detail?id=3937
-    // TODO(pamg): Remove this when the underlying bug is fixed.
-    //LOG(ERROR) << "DidStartLoading called while loading";
+    LOG(ERROR) << "DidStartLoading called while loading";
     return;
   }
   page_is_loading_ = true;
 }
 
 void TestWebViewDelegate::DidStopLoading(WebView* webview) {
   if (!page_is_loading_) {
-    // When we started including stderr in layout-test output, a number of
-    // number of tests began failing. The vast majority of the new failures
-    // were due to either this error message or the one in DidStartLoading.
-    // This is being disabled temporarily so we can keep running the tests.
-    // See http://code.google.com/p/chromium/issues/detail?id=3937
-    // TODO(pamg): Remove this when the underlying bug is fixed.
-    //LOG(ERROR) << "DidStopLoading called while not loading";
+    LOG(ERROR) << "DidStopLoading called while not loading";
     return;
   }
   page_is_loading_ = false;
 }
 
 void TestWebViewDelegate::WindowObjectCleared(WebFrame* webframe) {
   shell_->BindJSObjectsToWindow(webframe);
 }
 
 WindowOpenDisposition TestWebViewDelegate::DispositionForNavigationAction(
@@ skipping 686 matching lines @@
     else
       return L"main frame";
   } else {
     if (name.length())
       return L"frame \"" + name + L"\"";
     else
       return L"frame (anonymous)";
   }
 }