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

Side by Side Diff: third_party/WebKit/Source/core/layout/svg/SVGResources.cpp

Issue 2758683002: Replace ASSERT_NOT_REACHED with NOTREACHED in core/layout/ (Closed)
Patch Set: Rebase with latest Created 3 years, 9 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) Research In Motion Limited 2010. All rights reserved. 2 * Copyright (C) Research In Motion Limited 2010. 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 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 return s_tagList; 107 return s_tagList;
108 } 108 }
109 109
110 static inline AtomicString targetReferenceFromResource(SVGElement& element) { 110 static inline AtomicString targetReferenceFromResource(SVGElement& element) {
111 String target; 111 String target;
112 if (isSVGPatternElement(element)) 112 if (isSVGPatternElement(element))
113 target = toSVGPatternElement(element).href()->currentValue()->value(); 113 target = toSVGPatternElement(element).href()->currentValue()->value();
114 else if (isSVGGradientElement(element)) 114 else if (isSVGGradientElement(element))
115 target = toSVGGradientElement(element).href()->currentValue()->value(); 115 target = toSVGGradientElement(element).href()->currentValue()->value();
116 else 116 else
117 ASSERT_NOT_REACHED(); 117 NOTREACHED();
118 118
119 return SVGURIReference::fragmentIdentifierFromIRIString(target, 119 return SVGURIReference::fragmentIdentifierFromIRIString(target,
120 element.treeScope()); 120 element.treeScope());
121 } 121 }
122 122
123 static inline bool svgPaintTypeHasURL(SVGPaintType paintType) { 123 static inline bool svgPaintTypeHasURL(SVGPaintType paintType) {
124 switch (paintType) { 124 switch (paintType) {
125 case SVG_PAINTTYPE_URI_NONE: 125 case SVG_PAINTTYPE_URI_NONE:
126 case SVG_PAINTTYPE_URI_CURRENTCOLOR: 126 case SVG_PAINTTYPE_URI_CURRENTCOLOR:
127 case SVG_PAINTTYPE_URI_RGBCOLOR: 127 case SVG_PAINTTYPE_URI_RGBCOLOR:
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
396 if (m_clipperFilterMaskerData->filter == resource) 396 if (m_clipperFilterMaskerData->filter == resource)
397 m_clipperFilterMaskerData->filter = nullptr; 397 m_clipperFilterMaskerData->filter = nullptr;
398 break; 398 break;
399 case ClipperResourceType: 399 case ClipperResourceType:
400 if (!m_clipperFilterMaskerData) 400 if (!m_clipperFilterMaskerData)
401 break; 401 break;
402 if (m_clipperFilterMaskerData->clipper == resource) 402 if (m_clipperFilterMaskerData->clipper == resource)
403 m_clipperFilterMaskerData->clipper = nullptr; 403 m_clipperFilterMaskerData->clipper = nullptr;
404 break; 404 break;
405 default: 405 default:
406 ASSERT_NOT_REACHED(); 406 NOTREACHED();
407 } 407 }
408 } 408 }
409 409
410 void SVGResources::buildSetOfResources( 410 void SVGResources::buildSetOfResources(
411 HashSet<LayoutSVGResourceContainer*>& set) { 411 HashSet<LayoutSVGResourceContainer*>& set) {
412 if (!hasResourceData()) 412 if (!hasResourceData())
413 return; 413 return;
414 414
415 if (m_linkedResource) { 415 if (m_linkedResource) {
416 ASSERT(!m_clipperFilterMaskerData); 416 ASSERT(!m_clipperFilterMaskerData);
(...skipping 235 matching lines...) Expand 10 before | Expand all | Expand 10 after
652 stroke->element()); 652 stroke->element());
653 } 653 }
654 654
655 if (m_linkedResource) 655 if (m_linkedResource)
656 fprintf(stderr, " |-> xlink:href : %p (node=%p)\n", m_linkedResource, 656 fprintf(stderr, " |-> xlink:href : %p (node=%p)\n", m_linkedResource,
657 m_linkedResource->element()); 657 m_linkedResource->element());
658 } 658 }
659 #endif 659 #endif
660 660
661 } // namespace blink 661 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698