Extract a DOMWindow interface from LocalDOMWindow and use it in the idl.

Source/bindings/core/v8/custom/V8InspectorFrontendHostCustom.cpp

Index: Source/bindings/core/v8/custom/V8InspectorFrontendHostCustom.cpp
diff --git a/Source/bindings/core/v8/custom/V8InspectorFrontendHostCustom.cpp b/Source/bindings/core/v8/custom/V8InspectorFrontendHostCustom.cpp
index 84c513f0d92c2c3b397ce764fff1d1139cefb0e7..1ab8fc3f409f9de36433126c96b2751a5b1052b1 100644
--- a/Source/bindings/core/v8/custom/V8InspectorFrontendHostCustom.cpp
+++ b/Source/bindings/core/v8/custom/V8InspectorFrontendHostCustom.cpp
@@ -158,8 +158,8 @@ void V8InspectorFrontendHost::showContextMenuAtPointMethodCustom(const v8::Funct
         v8::Handle<v8::Object> windowWrapper = V8Window::findInstanceInPrototypeChain(isolate->GetEnteredContext()->Global(), isolate);
         if (windowWrapper.IsEmpty())
             return;
-        LocalDOMWindow* window = V8Window::toImpl(windowWrapper);
-        document = window ? window->document() : nullptr;
+        DOMWindow* window = V8Window::toImpl(windowWrapper);
+        document = window ? toLocalDOMWindow(window)->document() : nullptr;

[haraken, 2014/10/31 06:07:04]

Shouldn't this be:

  document = window->isLocalDOMWindow() ? toLocalDOMWindow(window)->document() :
nullptr;

?

[dcheng, 2014/10/31 06:31:01]

See my previous comment. This code should only ever run in the "right" renderer,
so we should never have to handle a non-LocalDOMWindow here. That is probably
not the case today, but that's why I prefer an explicit crash in
--site-per-process mode: this helps us keep track of problematic areas.

     }
     if (!document || !document->page())
         return;