Convert HTMLFrameOwnerElement and FocusController to use Frame.

Source/bindings/v8/custom/V8HTMLFrameElementCustom.cpp

 /*
  * Copyright (C) 2007-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 17 matching lines @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "V8HTMLFrameElement.h"
 
 #include "HTMLNames.h"
 #include "bindings/v8/BindingSecurity.h"
 #include "bindings/v8/ExceptionState.h"
 #include "bindings/v8/V8Binding.h"
+#include "core/frame/LocalFrame.h"

[dcheng, 2014/03/17 18:17:44]

Not clear what this new include is for.

[kenrb, 2014/03/18 20:01:19]

Oops, not needed. I had originally added it when looking at the forward
declaration problem mentioned in the other comment.

 #include "core/html/HTMLFrameElement.h"
 #include "core/html/parser/HTMLParserIdioms.h"
 
 namespace WebCore {
 
 using namespace HTMLNames;
 
 void V8HTMLFrameElement::locationAttributeSetterCustom(v8::Local<v8::Value> value, const v8::PropertyCallbackInfo<void>& info)
 {
     HTMLFrameElement* frame = V8HTMLFrameElement::toNative(info.Holder());
     V8TRYCATCH_FOR_V8STRINGRESOURCE_VOID(V8StringResource<WithNullCheck>, locationValue, value);
 
     ExceptionState exceptionState(ExceptionState::SetterContext, "location", "HTMLFrameElement", info.Holder(), info.GetIsolate());
     if (protocolIsJavaScript(stripLeadingAndTrailingHTMLSpaces(locationValue)) && !BindingSecurity::shouldAllowAccessToFrame(info.GetIsolate(), frame->contentFrame(), exceptionState)) {
         exceptionState.throwIfNeeded();
         return;
     }
 
     frame->setLocation(locationValue);
 }
 
 } // namespace WebCore