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

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

Issue 889563002: Make RenderObject::style() return a const object (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix a crashers (everything is building!) Created 5 years, 10 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2004, 2005, 2006 Nikolas Zimmermann <zimmermann@kde.org> 2 * Copyright (C) 2004, 2005, 2006 Nikolas Zimmermann <zimmermann@kde.org>
3 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010 Rob Buis <buis@kde.org> 3 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010 Rob Buis <buis@kde.org>
4 * Copyright (C) 2007 Apple Inc. All rights reserved. 4 * Copyright (C) 2007 Apple Inc. All rights reserved.
5 * Copyright (C) 2014 Google, Inc. 5 * Copyright (C) 2014 Google, Inc.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 499 matching lines...) Expand 10 before | Expand all | Expand 10 after
510 { 510 {
511 // FIXME: We should respect display: none on the documentElement svg element 511 // FIXME: We should respect display: none on the documentElement svg element
512 // but many things in FrameView and SVGImage depend on the RenderSVGRoot whe n 512 // but many things in FrameView and SVGImage depend on the RenderSVGRoot whe n
513 // they should instead depend on the RenderView. 513 // they should instead depend on the RenderView.
514 // https://bugs.webkit.org/show_bug.cgi?id=103493 514 // https://bugs.webkit.org/show_bug.cgi?id=103493
515 if (document().documentElement() == this) 515 if (document().documentElement() == this)
516 return true; 516 return true;
517 return Element::rendererIsNeeded(style); 517 return Element::rendererIsNeeded(style);
518 } 518 }
519 519
520 RenderObject* SVGSVGElement::createRenderer(RenderStyle*) 520 RenderObject* SVGSVGElement::createRenderer(const RenderStyle*)
521 { 521 {
522 if (isOutermostSVGSVGElement()) 522 if (isOutermostSVGSVGElement())
523 return new RenderSVGRoot(this); 523 return new RenderSVGRoot(this);
524 524
525 return new RenderSVGViewportContainer(this); 525 return new RenderSVGViewportContainer(this);
526 } 526 }
527 527
528 Node::InsertionNotificationRequest SVGSVGElement::insertedInto(ContainerNode* ro otParent) 528 Node::InsertionNotificationRequest SVGSVGElement::insertedInto(ContainerNode* ro otParent)
529 { 529 {
530 if (rootParent->inDocument()) { 530 if (rootParent->inDocument()) {
(...skipping 240 matching lines...) Expand 10 before | Expand all | Expand 10 after
771 visitor->trace(m_width); 771 visitor->trace(m_width);
772 visitor->trace(m_height); 772 visitor->trace(m_height);
773 visitor->trace(m_translation); 773 visitor->trace(m_translation);
774 visitor->trace(m_timeContainer); 774 visitor->trace(m_timeContainer);
775 visitor->trace(m_viewSpec); 775 visitor->trace(m_viewSpec);
776 SVGGraphicsElement::trace(visitor); 776 SVGGraphicsElement::trace(visitor);
777 SVGFitToViewBox::trace(visitor); 777 SVGFitToViewBox::trace(visitor);
778 } 778 }
779 779
780 } // namespace blink 780 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698