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

Source/core/workers/WorkerReportingProxy.h

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 22 matching lines @@
 
 #include "core/frame/ConsoleTypes.h"
 #include "platform/heap/Handle.h"
 #include "wtf/Forward.h"
 
 namespace blink {
 
 class ConsoleMessage;
 class WorkerGlobalScope;
 
-// APIs used by workers to report console activity.
+// APIs used by workers to report console and worker activity.
 class WorkerReportingProxy {
 public:
     virtual ~WorkerReportingProxy() { }
 
     virtual void reportException(const String& errorMessage, int lineNumber, int columnNumber, const String& sourceURL) = 0;
     virtual void reportConsoleMessage(PassRefPtrWillBeRawPtr<ConsoleMessage>) = 0;
     virtual void postMessageToPageInspector(const String&) = 0;
 
+    // Invoked when the worker script is evaluated. |success| is true if the
+    // evaluation completed with no uncaught exception.
+    virtual void workerScriptEvaluated(bool success) = 0;

[tkent, 2014/11/05 02:10:22]

Please name this as |didEvaluateWorkerScript|.  |will| and |did| are common
prefixes for notification receiver functions in Blink.

We usually avoid bool arguments, however the bool argument of this function is
acceptable in this case because literal |true| or |false| won't appear in the
source code.

[nhiroki, 2014/11/05 03:09:48]

Done.

+
     // Invoked when the new WorkerGlobalScope is started.
     virtual void workerGlobalScopeStarted(WorkerGlobalScope*) = 0;
 
     // Invoked when close() is invoked on the worker context.
     virtual void workerGlobalScopeClosed() = 0;
 
     // Invoked when the thread is stopped and WorkerGlobalScope is being
     // destructed. (This is be the last method that is called on this
     // interface)
     virtual void workerThreadTerminated() = 0;
 
     // Invoked when the thread is about to be stopped and WorkerGlobalScope
     // is to be destructed. (When this is called it is guaranteed that
     // WorkerGlobalScope is still alive)
     virtual void willDestroyWorkerGlobalScope() = 0;
 };
 
 } // namespace blink
 
 #endif // WorkerReportingProxy_h