Remove SVGStyledLocatableElement class and have SVGGraphicsElement inherit SVGStyledElement directly instead. SVGStyledLocatableElement is no longer needed and according to the specification, SVGGraphicsElement should inherit SVGStyledElement (actually SVGElement once we merge SVGStyledElement into SVGElement). Corresponding Blink revision: https://src.chromium.org/viewvc/blink?revision=153347&view=revision
Created attachment 205872 [details] Patch
Comment on attachment 205872 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205872&action=review r=me but please fix the indentation problem on the Xcode file. > Source/WebCore/WebCore.xcodeproj/project.pbxproj:26708 > - B2227AB30D00BF220071B782 /* SVGStyledLocatableElement.cpp in Sources */, > + B2227AB70D00BF220071B783 /* SVGStyledElement.cpp in Sources */, Indentation is wrong. Same for the other lines.
Comment on attachment 205872 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205872&action=review >> Source/WebCore/WebCore.xcodeproj/project.pbxproj:26708 >> + B2227AB70D00BF220071B783 /* SVGStyledElement.cpp in Sources */, > > Indentation is wrong. Same for the other lines. Oops. looks like my editor replaced the tabs with spaces :( Thanks for catching it.
Created attachment 205920 [details] Patch for landing
Comment on attachment 205920 [details] Patch for landing Clearing flags on attachment: 205920 Committed r152299: <http://trac.webkit.org/changeset/152299>
All reviewed patches have been landed. Closing bug.