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

Unified Diff: third_party/WebKit/Source/core/svg/SVGViewSpec.cpp

Issue 2485113002: Remove currentView, useCurrentView properties of SVGSVGElement and SVGViewSpec interface (Closed)
Patch Set: rebased Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/svg/SVGViewSpec.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGViewSpec.cpp b/third_party/WebKit/Source/core/svg/SVGViewSpec.cpp
index 6e3d0d487ed0a9780805909fd67de1f8d7e30ba4..eb541272a44fe237a2833316c43b4acaf2ae9371 100644
--- a/third_party/WebKit/Source/core/svg/SVGViewSpec.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGViewSpec.cpp
@@ -38,7 +38,7 @@ SVGViewSpec::SVGViewSpec(SVGSVGElement* contextElement)
// AnimatedProperty tearoff -(contextElement)-> SVGSVGElement -(RefPtr)->
// SVGViewSpec.
: SVGFitToViewBox(contextElement, PropertyMapPolicySkip),
- m_contextElement(this, contextElement),
+ m_contextElement(contextElement),
m_transform(SVGAnimatedTransformList::create(contextElement,
SVGNames::transformAttr)) {
ASSERT(m_contextElement);
@@ -56,10 +56,6 @@ DEFINE_TRACE(SVGViewSpec) {
SVGFitToViewBox::trace(visitor);
}
-DEFINE_TRACE_WRAPPERS(SVGViewSpec) {
- visitor->traceWrappers(m_contextElement);
-}
-
bool SVGViewSpec::parseViewSpec(const String& spec) {
if (spec.isEmpty() || !m_contextElement)
return false;
@@ -85,51 +81,6 @@ void SVGViewSpec::reset() {
m_viewTargetString = emptyString();
}
-void SVGViewSpec::detachContextElement() {
- m_transform = nullptr;
- clearViewBox();
- clearPreserveAspectRatio();
- m_contextElement = nullptr;
-}
-
-SVGElement* SVGViewSpec::viewTarget() const {
- if (!m_contextElement)
- return nullptr;
- Element* element = m_contextElement->treeScope().getElementById(
- AtomicString(m_viewTargetString));
- if (!element || !element->isSVGElement())
- return nullptr;
- return toSVGElement(element);
-}
-
-String SVGViewSpec::viewBoxString() const {
- if (!viewBox())
- return String();
-
- return viewBox()->currentValue()->valueAsString();
-}
-
-String SVGViewSpec::preserveAspectRatioString() const {
- if (!preserveAspectRatio())
- return String();
-
- return preserveAspectRatio()->baseValue()->valueAsString();
-}
-
-String SVGViewSpec::transformString() const {
- if (!m_transform)
- return String();
-
- return m_transform->baseValue()->valueAsString();
-}
-
-void SVGViewSpec::setZoomAndPan(unsigned short,
- ExceptionState& exceptionState) {
- // SVGViewSpec and all of its content is read-only.
- exceptionState.throwDOMException(NoModificationAllowedError,
- ExceptionMessages::readOnly());
-}
-
namespace {
enum ViewSpecFunctionType {
« no previous file with comments | « third_party/WebKit/Source/core/svg/SVGViewSpec.h ('k') | third_party/WebKit/Source/core/svg/SVGViewSpec.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698