Summary: | Add the ability to force text to render in white, not just black | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Peyton Randolph <prandolph> | ||||||
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, esprehn+autocc, glenn, kondapallykalyan, mmaxfield, prandolph, sam, simon.fraser | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 135257 | ||||||||
Attachments: |
|
Description
Peyton Randolph
2014-08-05 15:32:32 PDT
Created attachment 236057 [details]
Patch
Comment on attachment 236057 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=236057&action=review > Source/WebCore/rendering/InlineTextBox.cpp:631 > + const ShadowData* textShadow = (paintInfo.forceTextColor()) ? 0 : lineStyle.textShadow(); 0 -> nullptr > Source/WebCore/rendering/PaintPhase.h:63 > + PaintBehaviorForceWhiteText = 1 << 6, Please put this under PaintBehaviorForceBlackText and change the numbers. > Source/WebCore/rendering/TextPaintStyle.cpp:142 > + const ShadowData* shadow = paintInfo.forceTextColor() ? 0 : pseudoStyle->textShadow(); 0 -> nullptr (In reply to comment #2) > (From update of attachment 236057 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=236057&action=review > > > Source/WebCore/rendering/InlineTextBox.cpp:631 > > + const ShadowData* textShadow = (paintInfo.forceTextColor()) ? 0 : lineStyle.textShadow(); > > 0 -> nullptr Done. > > > Source/WebCore/rendering/PaintPhase.h:63 > > + PaintBehaviorForceWhiteText = 1 << 6, > > Please put this under PaintBehaviorForceBlackText and change the numbers. Done. > > > Source/WebCore/rendering/TextPaintStyle.cpp:142 > > + const ShadowData* shadow = paintInfo.forceTextColor() ? 0 : pseudoStyle->textShadow(); > > 0 -> nullptr Done. Created attachment 236061 [details]
Patch
Comment on attachment 236061 [details]
Patch
Re-doing Simon's r+ and adding cq+
Instead of having both Black and white hard coded, should we instead just allow specifying a specific color? Comment on attachment 236061 [details] Patch Clearing flags on attachment: 236061 Committed r172110: <http://trac.webkit.org/changeset/172110> All reviewed patches have been landed. Closing bug. |