Add an Error class for ServiceWorkers

Source/modules/serviceworkers/ServiceWorkerError.h

 /*
  * Copyright (C) 2013 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 10 matching lines @@
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef Task_h
-#define Task_h
+#ifndef ServiceWorkerError_h
+#define ServiceWorkerError_h
 
-#include "public/platform/WebThread.h"
-#include "wtf/Functional.h"
+#include "core/dom/DOMError.h"
+#include "public/platform/WebServiceWorkerError.h"
 
 namespace WebCore {
 
-class Task : public WebKit::WebThread::Task {
+class ServiceWorkerError : public DOMError {
 public:
-    explicit Task(const Closure& closure)
-        : m_closure(closure)
+    static PassRefPtr<ServiceWorkerError> create(const String& name, const String& message)
     {
+        return adoptRef(new ServiceWorkerError(name, message));
     }
 
-    virtual void run() OVERRIDE
+    // For CallbackPromiseAdapter
+    typedef WebKit::WebServiceWorkerError WebType;
+    static PassRefPtr<DOMError> from(WebType* worker)

[michaeln, 2013/10/15 01:28:48]

looks like this should be returning a PassRefPtr<ServiceWorkerError>

|worker| as the param name is misleading 

     {
-        m_closure();
+        return create(worker->name(), worker->message());

[kinuko, 2013/10/15 01:43:09]

I'm a bit confused; does this imply WebServiceWorkerError's memory has to be
managed by the caller?

I.e. do you assume a usage like:

WebServiceWorkerErrorImpl error(name, message);
callbacks->onError(&error);

     }
 
+    ~ServiceWorkerError() { }
+
 private:
-    Closure m_closure;
+    ServiceWorkerError(const String& name, const String& message) : DOMError(name, message) { }
 };
 
 } // namespace WebCore
 
-#endif // Task_h
+#endif // ServiceWorkerError_h