http://trac.webkit.org/changeset/196322 broke the !(ENABLE(SVG_FONTS) || ENABLE(SVG_OTF_CONVERTER)) build, because CSSFontFaceSource::CSSFontFaceSource:m_svgFontFaceElement is declared inside ifdef guards, but initialized uncionditionally in the init list of the constructor.
Created attachment 271055 [details] Patch
Or should we declare m_svgFontFaceElement unconditionally? ( similar to bug154035 )
(In reply to comment #2) > Or should we declare m_svgFontFaceElement unconditionally? > ( similar to bug154035 ) I think that would be cleaner.
(In reply to comment #3) > (In reply to comment #2) > > Or should we declare m_svgFontFaceElement unconditionally? > > ( similar to bug154035 ) > I think that would be cleaner. I agree.
EWS was all green on http://trac.webkit.org/changeset/196322 :(
(In reply to comment #4) > (In reply to comment #3) > > (In reply to comment #2) > > > Or should we declare m_svgFontFaceElement unconditionally? > > > ( similar to bug154035 ) > > I think that would be cleaner. > > I agree. I checked it in details, it's impossible. m_svgFontFaceElement has RefPtr<SVGFontFaceElement> type and SVGFontFaceElement class is defined inside ENABLE(SVG_FONTS) guard. To be able to use RefPtr<SVGFontFaceElement> type, we have to remove ENABLE(SVG_FONTS) guard from SVGFontFaceElement.h and I think transitionally all ENABLE(SVG_FONTS) guards everywhere. The question is if we want to support !ENABLE(SVG_FONTS) build or not. If yes, we should land my previously proposed patch. If not, we should remove all ENABLE(SVG_FONTS) guards.
Comment on attachment 271055 [details] Patch r? again, maybe we would like to keep !ENABLE(SVG_FONTS) build configuration work.
(In reply to comment #6) > (In reply to comment #4) > > (In reply to comment #3) > > > (In reply to comment #2) > > > > Or should we declare m_svgFontFaceElement unconditionally? > > > > ( similar to bug154035 ) > > > I think that would be cleaner. > > > > I agree. > > I checked it in details, it's impossible. > > m_svgFontFaceElement has RefPtr<SVGFontFaceElement> type and > SVGFontFaceElement class is defined inside ENABLE(SVG_FONTS) guard. > > To be able to use RefPtr<SVGFontFaceElement> type, we have to > remove ENABLE(SVG_FONTS) guard from SVGFontFaceElement.h and > I think transitionally all ENABLE(SVG_FONTS) guards everywhere. > > The question is if we want to support !ENABLE(SVG_FONTS) build or not. > If yes, we should land my previously proposed patch. If not, we should > remove all ENABLE(SVG_FONTS) guards. FWIW, I am very close to enabling the SVG font converter on Windows, and I hope the remaining ports will be quick to follow. I hope to remove all the SVG font code (in favor of the converter) in a few weeks.
Comment on attachment 271055 [details] Patch Clearing flags on attachment: 271055 Committed r196576: <http://trac.webkit.org/changeset/196576>
All reviewed patches have been landed. Closing bug.