Status: Started
Owner: [email protected]
Labels: Type-Bug Pri-2 OS-All Area-DevTools

New issue 7838 by [email protected]: gcl's no-tests warning is too eager
http://code.google.com/p/chromium/issues/detail?id=7838

This patch incorrectly gave a no-tests warning:

----
Index: webkit/port/bindings/v8/WorkerScriptController.cpp
===================================================================
--- webkit/port/bindings/v8/WorkerScriptController.cpp  (revision 9930)
+++ webkit/port/bindings/v8/WorkerScriptController.cpp  (working copy)
@@ -38,7 +38,7 @@
  #include "DOMTimer.h"
  #include "WorkerContext.h"
  #include "WorkerContextExecutionProxy.h"
-#include "WorkerMessagingProxy.h"
+#include "WorkerObjectProxy.h"
  #include "WorkerThread.h"

  namespace WebCore {
@@ -67,7 +67,7 @@
                           sourceCode.url().string(),
                           sourceCode.startLine() - 1);

-    m_workerContext->thread()->messagingProxy()->
+    m_workerContext->thread()->workerObjectProxy()->
         reportPendingActivity(m_workerContext->hasPendingActivity());

     return ScriptValue();
-----

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to