ServiceWorker: Registering a malformed script should fail [1/3]

Source/core/workers/WorkerReportingProxy.h

Index: Source/core/workers/WorkerReportingProxy.h
diff --git a/Source/core/workers/WorkerReportingProxy.h b/Source/core/workers/WorkerReportingProxy.h
index b6a6f723548a736ab7d29a202f5d1186ec404982..52d2c5653e3b53364eec57c8d7606afce9a871bb 100644
--- a/Source/core/workers/WorkerReportingProxy.h
+++ b/Source/core/workers/WorkerReportingProxy.h
@@ -49,6 +49,10 @@ public:
     virtual void reportConsoleMessage(PassRefPtrWillBeRawPtr<ConsoleMessage>) = 0;
     virtual void postMessageToPageInspector(const String&) = 0;
 
+    // Invoked when the worker script is evaluated. |result| is true if the
+    // evaluation succeeded.

[falken, 2014/11/04 05:41:26]

One might wonder what it means for an evaluation to "succeed", maybe say
"evaluation completed with no uncaught exception"

[nhiroki, 2014/11/04 09:03:11]

Right, "succeed" is not descriptive. Rephrased.

+    virtual void workerScriptEvaluated(bool result) = 0;

[falken, 2014/11/04 05:41:26]

"result" is not so clear of a name, how about bool success?

[nhiroki, 2014/11/04 09:03:11]

Done.

+
     // Invoked when the new WorkerGlobalScope is started.
     virtual void workerGlobalScopeStarted(WorkerGlobalScope*) = 0;