Bug 1443797: Remove SVGTests::IsInChromeDoc. r=dholbert

Summary: It's effectively the same as AsSVGElement()->IsInChromeDocument().

Reviewers: dholbert

Bug #: 1443797

Differential Revision: https://phabricator.services.mozilla.com/D695

MozReview-Commit-ID: 8f8fUsPfdnm
This commit is contained in:
Emilio Cobos Álvarez 2018-03-07 15:18:52 +01:00
parent b360ee3691
commit 73a6a2096e
8 changed files with 9 additions and 32 deletions

View File

@ -353,15 +353,6 @@ SVGAnimationElement::IsNodeOfType(uint32_t aFlags) const
return !(aFlags & ~eANIMATION);
}
//----------------------------------------------------------------------
// SVGTests methods
bool
SVGAnimationElement::IsInChromeDoc() const
{
return nsContentUtils::IsChromeDoc(OwnerDoc());
}
//----------------------------------------------------------------------
// SVG utility methods

View File

@ -88,7 +88,6 @@ public:
void EndElementAt(float offset, ErrorResult& rv);
// SVGTests
virtual bool IsInChromeDoc() const override;
nsSVGElement* AsSVGElement() final { return this; }
protected:

View File

@ -31,11 +31,5 @@ SVGGraphicsElement::~SVGGraphicsElement()
{
}
bool
SVGGraphicsElement::IsInChromeDoc() const
{
return nsContentUtils::IsChromeDoc(OwnerDoc());
}
} // namespace dom
} // namespace mozilla

View File

@ -26,7 +26,6 @@ public:
// interfaces:
NS_DECL_ISUPPORTS_INHERITED
bool IsInChromeDoc() const override;
nsSVGElement* AsSVGElement() final { return this; }
};

View File

@ -42,14 +42,5 @@ SVGSymbolElement::~SVGSymbolElement()
NS_IMPL_ELEMENT_CLONE_WITH_INIT(SVGSymbolElement)
//----------------------------------------------------------------------
// SVGTests methods
bool
SVGSymbolElement::IsInChromeDoc() const
{
return nsContentUtils::IsChromeDoc(OwnerDoc());
}
} // namespace dom
} // namespace mozilla

View File

@ -32,9 +32,6 @@ public:
virtual nsresult Clone(mozilla::dom::NodeInfo *aNodeInfo, nsINode **aResult,
bool aPreallocateChildren) const override;
// SVGTests
bool IsInChromeDoc() const override;
};
} // namespace dom

View File

@ -51,7 +51,8 @@ SVGTests::SystemLanguage()
bool
SVGTests::HasExtension(const nsAString& aExtension)
{
return nsSVGFeatures::HasExtension(aExtension, IsInChromeDoc());
return nsSVGFeatures::HasExtension(aExtension,
AsSVGElement()->IsInChromeDocument());
}
bool
@ -118,7 +119,8 @@ SVGTests::PassesConditionalProcessingTests(const nsString *aAcceptLangs) const
return false;
}
for (uint32_t i = 0; i < mStringListAttributes[EXTENSIONS].Length(); i++) {
if (!nsSVGFeatures::HasExtension(mStringListAttributes[EXTENSIONS][i], IsInChromeDoc())) {
if (!nsSVGFeatures::HasExtension(mStringListAttributes[EXTENSIONS][i],
AsSVGElement()->IsInChromeDocument())) {
return false;
}
}

View File

@ -96,9 +96,13 @@ public:
already_AddRefed<DOMSVGStringList> SystemLanguage();
bool HasExtension(const nsAString& aExtension);
virtual bool IsInChromeDoc() const = 0;
virtual nsSVGElement* AsSVGElement() = 0;
const nsSVGElement* AsSVGElement() const
{
return const_cast<SVGTests*>(this)->AsSVGElement();
}
protected:
virtual ~SVGTests() {}