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

Side by Side Diff: Source/core/svg/SVGExternalResourcesRequired.cpp

Issue 62083002: Remove support for the externalResourcesRequired attribute. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: rebaseline test 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
« no previous file with comments | « Source/core/svg/SVGEllipseElement.idl ('k') | Source/core/svg/SVGExternalResourcesRequired.idl » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 /*
2 * Copyright (C) 2004, 2005, 2008 Nikolas Zimmermann <zimmermann@kde.org>
3 * Copyright (C) 2004, 2005, 2007 Rob Buis <buis@kde.org>
4 *
5 * This library is free software; you can redistribute it and/or
6 * modify it under the terms of the GNU Library General Public
7 * License as published by the Free Software Foundation; either
8 * version 2 of the License, or (at your option) any later version.
9 *
10 * This library is distributed in the hope that it will be useful,
11 * but WITHOUT ANY WARRANTY; without even the implied warranty of
12 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
13 * Library General Public License for more details.
14 *
15 * You should have received a copy of the GNU Library General Public License
16 * along with this library; see the file COPYING.LIB. If not, write to
17 * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
18 * Boston, MA 02110-1301, USA.
19 */
20
21 #include "config.h"
22
23 #include "core/svg/SVGExternalResourcesRequired.h"
24
25 #include "SVGNames.h"
26 #include "core/svg/SVGElement.h"
27
28 namespace WebCore {
29
30 bool SVGExternalResourcesRequired::parseAttribute(const QualifiedName& name, con st AtomicString& value)
31 {
32 if (name == SVGNames::externalResourcesRequiredAttr) {
33 setExternalResourcesRequiredBaseValue(value == "true");
34 return true;
35 }
36
37 return false;
38 }
39
40 bool SVGExternalResourcesRequired::isKnownAttribute(const QualifiedName& attrNam e)
41 {
42 return attrName == SVGNames::externalResourcesRequiredAttr;
43 }
44
45 void SVGExternalResourcesRequired::addSupportedAttributes(HashSet<QualifiedName> & supportedAttributes)
46 {
47 supportedAttributes.add(SVGNames::externalResourcesRequiredAttr);
48 }
49
50 bool SVGExternalResourcesRequired::handleAttributeChange(SVGElement* targetEleme nt, const QualifiedName& attrName)
51 {
52 ASSERT(targetElement);
53 if (!isKnownAttribute(attrName))
54 return false;
55 if (!targetElement->inDocument())
56 return true;
57
58 // Handle dynamic updates of the 'externalResourcesRequired' attribute. Only possible case: changing from 'true' to 'false'
59 // causes an immediate dispatch of the SVGLoad event. If the attribute value was 'false' before inserting the script element
60 // in the document, the SVGLoad event has already been dispatched.
61 if (!externalResourcesRequiredBaseValue() && !haveFiredLoadEvent() && !isPar serInserted()) {
62 setHaveFiredLoadEvent(true);
63 ASSERT(targetElement->haveLoadedRequiredResources());
64
65 targetElement->sendSVGLoadEventIfPossible();
66 }
67
68 return true;
69 }
70
71 void SVGExternalResourcesRequired::dispatchLoadEvent(SVGElement* targetElement)
72 {
73 bool externalResourcesRequired = externalResourcesRequiredBaseValue();
74
75 if (isParserInserted())
76 ASSERT(externalResourcesRequired != haveFiredLoadEvent());
77 else if (haveFiredLoadEvent())
78 return;
79
80 // HTML and SVG differ completely in the 'onload' event handling of <script> elements.
81 // HTML fires the 'load' event after it sucessfully loaded a remote resource , otherwise an error event.
82 // SVG fires the SVGLoad event immediately after parsing the <script> elemen t, if externalResourcesRequired
83 // is set to 'false', otherwise it dispatches the 'SVGLoad' event just after loading the remote resource.
84 if (!externalResourcesRequired)
85 return;
86
87 ASSERT(!haveFiredLoadEvent());
88
89 // Dispatch SVGLoad event
90 setHaveFiredLoadEvent(true);
91 ASSERT(targetElement->haveLoadedRequiredResources());
92
93 targetElement->sendSVGLoadEventIfPossible();
94 }
95
96 void SVGExternalResourcesRequired::insertedIntoDocument(SVGElement* targetElemen t)
97 {
98 if (isParserInserted())
99 return;
100
101 // Eventually send SVGLoad event now for the dynamically inserted script ele ment.
102 if (externalResourcesRequiredBaseValue())
103 return;
104 setHaveFiredLoadEvent(true);
105 targetElement->sendSVGLoadEventIfPossibleAsynchronously();
106 }
107
108 void SVGExternalResourcesRequired::finishParsingChildren()
109 {
110 // A SVGLoad event has been fired by SVGElement::finishParsingChildren.
111 if (!externalResourcesRequiredBaseValue())
112 setHaveFiredLoadEvent(true);
113 }
114
115 bool SVGExternalResourcesRequired::haveLoadedRequiredResources() const
116 {
117 return !externalResourcesRequiredBaseValue() || haveFiredLoadEvent();
118 }
119
120 }
OLDNEW
« no previous file with comments | « Source/core/svg/SVGEllipseElement.idl ('k') | Source/core/svg/SVGExternalResourcesRequired.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698