ServiceWorker: Implement attributes of ExtendableMessageEvent

third_party/WebKit/Source/bindings/modules/v8/custom/V8ExtendableMessageEventCustom.cpp

Index: third_party/WebKit/Source/bindings/modules/v8/custom/V8ExtendableMessageEventCustom.cpp
diff --git a/third_party/WebKit/Source/bindings/modules/v8/custom/V8ServiceWorkerMessageEventCustom.cpp b/third_party/WebKit/Source/bindings/modules/v8/custom/V8ExtendableMessageEventCustom.cpp
similarity index 76%
copy from third_party/WebKit/Source/bindings/modules/v8/custom/V8ServiceWorkerMessageEventCustom.cpp
copy to third_party/WebKit/Source/bindings/modules/v8/custom/V8ExtendableMessageEventCustom.cpp
index 7ca9fcd39cf0613df783ff1b26ab008044b0a9f1..ced0bf35a072e90255f1a98ba75e348053cfbeb1 100644
--- a/third_party/WebKit/Source/bindings/modules/v8/custom/V8ServiceWorkerMessageEventCustom.cpp
+++ b/third_party/WebKit/Source/bindings/modules/v8/custom/V8ExtendableMessageEventCustom.cpp
@@ -1,17 +1,17 @@
-// Copyright 2015 The Chromium Authors. All rights reserved.
+// Copyright 2016 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.
 
-#include "bindings/modules/v8/V8ServiceWorkerMessageEvent.h"
+#include "bindings/modules/v8/V8ExtendableMessageEvent.h"
 
 #include "bindings/core/v8/SerializedScriptValue.h"
 #include "bindings/core/v8/V8HiddenValue.h"
 
 namespace blink {
 
-void V8ServiceWorkerMessageEvent::dataAttributeGetterCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
+void V8ExtendableMessageEvent::dataAttributeGetterCustom(const v8::FunctionCallbackInfo<v8::Value>& info)
 {
-    ServiceWorkerMessageEvent* event = V8ServiceWorkerMessageEvent::toImpl(info.Holder());

[bashi, 2016/02/10 00:38:55]

It seems that ServiceWorkerMessageEvent also causes memory leak...

+    ExtendableMessageEvent* event = V8ExtendableMessageEvent::toImpl(info.Holder());

[bashi, 2016/02/10 00:38:55]

Please refer V8CustomEvent::detailAttributeGetterCustom() to see how to
implement a getter for "any" types. We should consider where the |data| comes
from and shouldn't expose it to different worlds.

[nhiroki, 2016/02/10 08:08:57]

Done.

     v8::Isolate* isolate = info.GetIsolate();
     ScriptState* scriptState = ScriptState::current(isolate);
     v8::Local<v8::Value> result = V8HiddenValue::getHiddenValue(scriptState, info.Holder(), V8HiddenValue::data(isolate));