Navigation transitions (web to native app): Get names and rects of transition elements (Step 2)

Source/core/testing/Internals.idl

Index: Source/core/testing/Internals.idl
diff --git a/Source/core/testing/Internals.idl b/Source/core/testing/Internals.idl
index 4b6ba69ca5672a0e8319fdb05986e705b9b6697d..d490df43024b7793fe78714c7ab5bd876c0b4920 100644
--- a/Source/core/testing/Internals.idl
+++ b/Source/core/testing/Internals.idl
@@ -279,7 +279,10 @@
     // This function is for testing HitRegions on Canvas2D.
     unsigned long countHitRegions(CanvasRenderingContext2D context);
 
+    [TypeChecking=Interface] ClientRect boundsInRootViewSpace(Element element);

[esprehn, 2014/10/21 20:50:48]

What's with the TypeChecking attr? I don't think you need that.

[Zhen Wang, 2014/10/21 21:36:48]

ditto

[sof, 2014/10/22 05:22:46]

Please keep using [TypeChecking=Inteface].

boundsInRootViewSpace() will now crash if you pass it {} as argument (or some
other non-Element object.)

[Zhen Wang, 2014/10/22 13:55:55]

I see. Thanks for pointing this out.

     DOMString serializeNavigationMarkup();
+    DOMString[] getTransitionElementNames();
+    [RaisesException, TypeChecking=Interface] ClientRectList getTransitionElementRects();

[esprehn, 2014/10/21 20:50:48]

Remove RaisesException, it doesn't use that argument. Also why you do have this
TypeChecking attr?

[Zhen Wang, 2014/10/21 21:36:48]

I see other places using ClientRectList have RaisesException and TypeChecking
and thought it is required.

They are removed now.

     void hideAllTransitionElements();
 
     [RaisesException, TypeChecking=Interface] void forcePluginPlaceholder(HTMLElement plugin, DocumentFragment fragment);