Replicate sandbox flags for OOPIF (Blink part 2)

Source/core/html/HTMLFrameOwnerElement.h

 /*
  * Copyright (C) 2006, 2007, 2009 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 59 matching lines @@
 
     class UpdateSuspendScope {
     public:
         UpdateSuspendScope();
         ~UpdateSuspendScope();
 
     private:
         void performDeferredWidgetTreeOperations();
     };
 
+    virtual SandboxFlags sandboxFlags() const override;
+
     virtual void trace(Visitor*) override;
 
 protected:
     HTMLFrameOwnerElement(const QualifiedName& tagName, Document&);
     void setSandboxFlags(SandboxFlags);
 
     bool loadOrRedirectSubframe(const KURL&, const AtomicString& frameName, bool lockBackForwardList, ContentSecurityPolicyDisposition shouldCheckContentSecurityPolicy);
 
 private:
     virtual bool isKeyboardFocusable() const override;
     virtual bool isFrameOwnerElement() const override final { return true; }
 
     // FrameOwner overrides:
     virtual bool isLocal() const { return true; }
-    virtual SandboxFlags sandboxFlags() const override;
     virtual void dispatchLoad() override;

[dcheng, 2015/01/21 23:19:06]

I think we should move this entire block to public:

There's not a lot of point to making only some of these private, since you can
call them by upcasting to FrameOwner anyway.

[alexmos, 2015/01/22 00:37:58]

Done.  Nate, are you ok with this?

[Nate Chapin, 2015/01/22 19:00:27]

Yeah, that's a good argument.

 
     RawPtrWillBeMember<Frame> m_contentFrame;
     RefPtrWillBeMember<Widget> m_widget;
     SandboxFlags m_sandboxFlags;
 };
 
 DEFINE_ELEMENT_TYPE_CASTS(HTMLFrameOwnerElement, isFrameOwnerElement());
 
 class SubframeLoadingDisabler {
     STACK_ALLOCATED();
@@ skipping 24 matching lines @@
     static WillBeHeapHashCountedSet<RawPtrWillBeMember<Node>>& disabledSubtreeRoots();
 
     RawPtrWillBeMember<Node> m_root;
 };
 
 DEFINE_TYPE_CASTS(HTMLFrameOwnerElement, FrameOwner, owner, owner->isLocal(), owner.isLocal());
 
 } // namespace blink
 
 #endif // HTMLFrameOwnerElement_h