Make V8Binding helpers return LocalFrame/LocalDOMWindow as appropriate

third_party/WebKit/Source/bindings/core/v8/V8Binding.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009 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 699 matching lines @@
   if (value.IsEmpty() || !value->IsObject())
     return 0;
 
   v8::Local<v8::Object> windowWrapper = V8Window::findInstanceInPrototypeChain(
       v8::Local<v8::Object>::Cast(value), isolate);
   if (!windowWrapper.IsEmpty())
     return V8Window::toImpl(windowWrapper);
   return 0;
 }
 
-DOMWindow* toDOMWindow(v8::Local<v8::Context> context) {
+LocalDOMWindow* toDOMWindow(v8::Local<v8::Context> context) {
   if (context.IsEmpty())
     return 0;
-  return toDOMWindow(context->GetIsolate(), context->Global());
+  return toLocalDOMWindow(
+      toDOMWindow(context->GetIsolate(), context->Global()));
 }
 
 LocalDOMWindow* enteredDOMWindow(v8::Isolate* isolate) {
-  LocalDOMWindow* window =
-      toLocalDOMWindow(toDOMWindow(isolate->GetEnteredContext()));
+  LocalDOMWindow* window = toDOMWindow(isolate->GetEnteredContext());
   if (!window) {
     // We don't always have an entered DOM window, for example during microtask
     // callbacks from V8 (where the entered context may be the DOM-in-JS
     // context). In that case, we fall back to the current context.
     //
     // TODO(haraken): It's nasty to return a current window from
     // enteredDOMWindow.  All call sites should be updated so that it works even
     // if it doesn't have an entered window.
     window = currentDOMWindow(isolate);
     ASSERT(window);
   }
   return window;
 }
 
 LocalDOMWindow* currentDOMWindow(v8::Isolate* isolate) {
-  return toLocalDOMWindow(toDOMWindow(isolate->GetCurrentContext()));
+  return toDOMWindow(isolate->GetCurrentContext());
 }
 
 ExecutionContext* toExecutionContext(v8::Local<v8::Context> context) {
   if (context.IsEmpty())
     return 0;
   v8::Local<v8::Object> global = context->Global();
   v8::Local<v8::Object> windowWrapper =
       V8Window::findInstanceInPrototypeChain(global, context->GetIsolate());
   if (!windowWrapper.IsEmpty())
     return V8Window::toImpl(windowWrapper)->getExecutionContext();
   v8::Local<v8::Object> workerWrapper =
       V8WorkerGlobalScope::findInstanceInPrototypeChain(global,
                                                         context->GetIsolate());
   if (!workerWrapper.IsEmpty())
     return V8WorkerGlobalScope::toImpl(workerWrapper)->getExecutionContext();
   v8::Local<v8::Object> workletWrapper =
       V8WorkletGlobalScope::findInstanceInPrototypeChain(global,
                                                          context->GetIsolate());
   if (!workletWrapper.IsEmpty())
     return V8WorkletGlobalScope::toImpl(workletWrapper);
   // FIXME: Is this line of code reachable?
   return nullptr;
 }
 
 ExecutionContext* currentExecutionContext(v8::Isolate* isolate) {
   return toExecutionContext(isolate->GetCurrentContext());
 }
 
-Frame* toFrameIfNotDetached(v8::Local<v8::Context> context) {
-  DOMWindow* window = toDOMWindow(context);
+LocalFrame* toFrameIfNotDetached(v8::Local<v8::Context> context) {
+  LocalDOMWindow* window = toDOMWindow(context);
   if (window && window->isCurrentlyDisplayedInFrame())
     return window->frame();
   // We return 0 here because |context| is detached from the Frame. If we
   // did return |frame| we could get in trouble because the frame could be
   // navigated to another security origin.
   return nullptr;
 }
 
 void toFlexibleArrayBufferView(v8::Isolate* isolate,
                                v8::Local<v8::Value> value,
@@ skipping 196 matching lines @@
   if (!v8Call(v8::JSON::Parse(isolate, v8String(isolate, stringifiedJSON)),
               parsed, tryCatch)) {
     if (tryCatch.HasCaught())
       exceptionState.rethrowV8Exception(tryCatch.Exception());
   }
 
   return parsed;
 }
 
 }  // namespace blink