Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(607)

Side by Side Diff: Source/core/html/HTMLFrameOwnerElement.cpp

Issue 146193003: Remove SVGDocument binding (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2009 Apple Inc. All rights reserved.
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 void HTMLFrameOwnerElement::setSandboxFlags(SandboxFlags flags) 106 void HTMLFrameOwnerElement::setSandboxFlags(SandboxFlags flags)
107 { 107 {
108 m_sandboxFlags = flags; 108 m_sandboxFlags = flags;
109 } 109 }
110 110
111 bool HTMLFrameOwnerElement::isKeyboardFocusable() const 111 bool HTMLFrameOwnerElement::isKeyboardFocusable() const
112 { 112 {
113 return m_contentFrame && HTMLElement::isKeyboardFocusable(); 113 return m_contentFrame && HTMLElement::isKeyboardFocusable();
114 } 114 }
115 115
116 SVGDocument* HTMLFrameOwnerElement::getSVGDocument(ExceptionState& exceptionStat e) const 116 Document* HTMLFrameOwnerElement::getSVGDocument(ExceptionState& exceptionState) const
Inactive 2014/01/24 00:00:38 Why do we need to return a Document. We can keep r
117 { 117 {
118 Document* doc = contentDocument(); 118 Document* doc = contentDocument();
119 if (doc && doc->isSVGDocument()) 119 if (doc && doc->isSVGDocument())
120 return toSVGDocument(doc); 120 return toSVGDocument(doc);
121 return 0; 121 return 0;
122 } 122 }
123 123
124 bool HTMLFrameOwnerElement::loadOrRedirectSubframe(const KURL& url, const Atomic String& frameName, bool lockBackForwardList) 124 bool HTMLFrameOwnerElement::loadOrRedirectSubframe(const KURL& url, const Atomic String& frameName, bool lockBackForwardList)
125 { 125 {
126 RefPtr<Frame> parentFrame = document().frame(); 126 RefPtr<Frame> parentFrame = document().frame();
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
167 // FIXME: In this case the Frame will have finished loading before 167 // FIXME: In this case the Frame will have finished loading before
168 // it's being added to the child list. It would be a good idea to 168 // it's being added to the child list. It would be a good idea to
169 // create the child first, then invoke the loader separately. 169 // create the child first, then invoke the loader separately.
170 if (childFrame->loader().state() == FrameStateComplete && !childFrame->loade r().policyDocumentLoader()) 170 if (childFrame->loader().state() == FrameStateComplete && !childFrame->loade r().policyDocumentLoader())
171 childFrame->loader().checkCompleted(); 171 childFrame->loader().checkCompleted();
172 return true; 172 return true;
173 } 173 }
174 174
175 175
176 } // namespace WebCore 176 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698