Move mediastream module to oilpan transition types

Source/bindings/scripts/code_generator_v8.pm

Index: Source/bindings/scripts/code_generator_v8.pm
diff --git a/Source/bindings/scripts/code_generator_v8.pm b/Source/bindings/scripts/code_generator_v8.pm
index 8156add23434ce619e4a4b83634d9a5e23632014..0c0556f8bf6e46087848c6ea31c1c8b9be4afc01 100644
--- a/Source/bindings/scripts/code_generator_v8.pm
+++ b/Source/bindings/scripts/code_generator_v8.pm
@@ -5429,6 +5429,7 @@ sub GetNativeType
     my $type = shift;
     my $extendedAttributes = shift;
     my $isParameter = shift;
+    my $isMember = shift;
 
     my $svgNativeType = GetSVGTypeNeedingTearOff($type);
     if ($svgNativeType) {
@@ -5479,7 +5480,11 @@ sub GetNativeType
         if ($isParameter) {
             return "$implClassName*";
         } elsif (IsWillBeGarbageCollectedType($interface->name)) {
-            return "RefPtrWillBeRawPtr<$implClassName>";
+            if ($isMember) {
+                return "RefPtrWillBeMember<$implClassName>";

[Nils Barth (inactive), 2014/02/26 07:17:54]

Keishi, could you please add a test case?

Since this is "[WillBeGarbageCollected] on another interface"
(that's being used as an interface type: i.e., not on the current interface),
(correct?)
the relevant file is:
bindings/tests/idls/TestObjectPython.idl
...and you can put it after the other [WillBeGarbageCollected] test case.

This will ensure that we get updated test results, and
run-bindings-tests (r-b-t) catches changes.

You'll also need to update the Python CG (r-b-t enforces this),
or you can wait until we switch to Python (today or tomorrow)
and just work there.

Thanks!

[keishi, 2014/02/27 07:33:34]

I'm not sure if it is right but I've added a test.

[Nils Barth (inactive), 2014/02/27 11:20:53]

Thanks, looks generally ok;
I'll take a closer look later.

[Nils Barth (inactive), 2014/03/03 02:47:53]

Current test case looks ok, but you need one more for callback interfaces.
To check test coverage, first look at changes to output,
and see that all your new code appears.

Currently this is:
https://codereview.chromium.org/173363002/diff/340001/Source/bindings/tests/r...
...and it does include 'WillBeHeapVector' and 'toRefPtrWillBeMemberNativeArray'.

(Also, this gives full test coverage of v8_types.py;
to get this output the code needs to cover all your new code,
and you're just adding a test, without changing existing output,
so this gives full test coverage.)

However, since you're also changing v8_callback_interface.py,
you need to add a test case to TestCallbackInterface.idl.

Could you put the following at the end of TestCallbackInterface.idl ?
(Same test case as you put in TestObjectPython.idl, but different
behavior for callback interfaces.)

void
voidMethodSequenceTestInterfaceWillBeGarbageCollectedArg(sequence<TestInterfaceWillBeGarbageCollected>
sequenceTestInterfaceWillBeGarbageCollectedArg); // [WillBeGarbageCollected]

Thanks!

+            } else {
+                return "RefPtrWillBeRawPtr<$implClassName>";
+            }
         } else {
             return "RefPtr<$implClassName>";
         }
@@ -5489,9 +5494,15 @@ sub GetNativeType
     my $arrayOrSequenceType = GetArrayOrSequenceType($type);
 
     if ($arrayOrSequenceType) {
-        my $nativeType = GetNativeType($arrayOrSequenceType);
-        $nativeType .= " " if ($nativeType =~ />$/);
-        return "Vector<${nativeType}>";
+        if (IsWillBeGarbageCollectedType($arrayOrSequenceType)) {
+            my $nativeType = GetNativeType($arrayOrSequenceType, {}, "", "member");
+            $nativeType .= " " if ($nativeType =~ />$/);
+            return "WillBeHeapVector<${nativeType}>";
+        } else {
+            my $nativeType = GetNativeType($arrayOrSequenceType);
+            $nativeType .= " " if ($nativeType =~ />$/);
+            return "Vector<${nativeType}>";
+        }
     }
 
     # Default, assume native type is a pointer with same type name as idl type
@@ -5508,6 +5519,7 @@ sub GetNativeTypeForCallbacks
     # Callbacks use raw pointers, so pass isParameter = 1
     my $nativeType = GetNativeType($type, {}, "parameter");
     return "const $nativeType&" if $nativeType =~ /^Vector/;
+    return "const $nativeType&" if $nativeType =~ /^WillBeHeapVector/;
     return $nativeType;
 }
 
@@ -5616,6 +5628,10 @@ sub JSValueToNative
     my $arrayOrSequenceType = GetArrayOrSequenceType($type);
 
     if ($arrayOrSequenceType) {
+        if (IsWillBeGarbageCollectedType($arrayOrSequenceType)) {
+            AddToImplIncludes("V8${arrayOrSequenceType}.h");
+            return "(toRefPtrWillBeMemberNativeArray<${arrayOrSequenceType}, V8${arrayOrSequenceType}>($value, $argIndexOrZero, $getIsolate))";

[haraken, 2014/02/26 13:58:37]

Still I don't fully understand why we need toRefPtrWillBeMemberNativeArray() and
toRefPtrNativeArray(). What we need would be toMemberNativeArray() and
toRefPtrNativeArray(), wouldn't it? I think it's enough to have the Member
version and the RefPtr version in order to make both the oilpan build and the
non-oilpan build compilable.

[keishi, 2014/02/27 07:33:34]

I misunderstood. Done.

+        }
         if (IsRefPtrType($arrayOrSequenceType)) {
             AddToImplIncludes("V8${arrayOrSequenceType}.h");
             return "(toRefPtrNativeArray<${arrayOrSequenceType}, V8${arrayOrSequenceType}>($value, $argIndexOrZero, $getIsolate))";