Summary: | Fix the !(ENABLE(SVG_FONTS) || ENABLE(SVG_OTF_CONVERTER)) build after r196322 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> | ||||
Component: | New Bugs | Assignee: | Csaba Osztrogonác <ossy> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, commit-queue, darin, koivisto, mmaxfield, ossy | ||||
Priority: | P2 | ||||||
Version: | Other | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=154035 | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 153414 | ||||||
Attachments: |
|
Description
Csaba Osztrogonác
2016-02-11 03:26:28 PST
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. |