Bug 240401

Summary: [Repaint] Border ignores currentColor change when hovering
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: zalan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, cdumez, changseok, dino, esprehn+autocc, ews-watchlist, fmalita, glenn, gyuyoung.kim, hi, koivisto, kondapallykalyan, mifenton, mmaxfield, ntim, pdr, sabouhallawa, schenney, sergio, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description zalan 2022-05-13 15:27:52 PDT
ssia
Comment 1 zalan 2022-05-13 16:35:57 PDT
Created attachment 459328 [details]
Patch
Comment 2 zalan 2022-05-13 18:40:41 PDT
Created attachment 459337 [details]
Patch
Comment 3 Tim Nguyen (:ntim) 2022-05-14 00:04:01 PDT
Is bug 223654 a duplicate?
Comment 4 Tim Nguyen (:ntim) 2022-05-14 00:05:18 PDT
Comment on attachment 459337 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=459337&action=review

> LayoutTests/ChangeLog:12
> +        * fast/repaint/repaint-current-color-border-on-hover-expected.txt: Added.
> +        * fast/repaint/repaint-current-color-border-on-hover.html: Added.
> +        * fast/repaint/repaint-pseudo-border-on-hover-expected.txt: Added.
> +        * fast/repaint/repaint-pseudo-border-on-hover.html: Added.
> +

These files are not in the patch
Comment 5 zalan 2022-05-14 05:34:04 PDT
(In reply to Tim Nguyen (:ntim) from comment #4)
> Comment on attachment 459337 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=459337&action=review
> 
> > LayoutTests/ChangeLog:12
> > +        * fast/repaint/repaint-current-color-border-on-hover-expected.txt: Added.
> > +        * fast/repaint/repaint-current-color-border-on-hover.html: Added.
> > +        * fast/repaint/repaint-pseudo-border-on-hover-expected.txt: Added.
> > +        * fast/repaint/repaint-pseudo-border-on-hover.html: Added.
> > +
> 
> These files are not in the patch
oops
Comment 6 Antti Koivisto 2022-05-14 05:39:53 PDT
Comment on attachment 459337 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=459337&action=review

> Source/WebCore/rendering/style/BorderData.h:114
> +        return !currentColorDiffers || (m_top.color().isValid() && m_right.color().isValid() && m_bottom.color().isValid() && m_left.color().isValid());

Might be more stylish to use RenderStyle::isCurrentColor(m_top.color()) etc.
Comment 7 zalan 2022-05-14 06:53:00 PDT
Created attachment 459355 [details]
Patch
Comment 8 zalan 2022-05-14 06:58:35 PDT
(In reply to zalan from comment #5)
> (In reply to Tim Nguyen (:ntim) from comment #4)
> > Comment on attachment 459337 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=459337&action=review
> > 
> > > LayoutTests/ChangeLog:12
> > > +        * fast/repaint/repaint-current-color-border-on-hover-expected.txt: Added.
> > > +        * fast/repaint/repaint-current-color-border-on-hover.html: Added.
> > > +        * fast/repaint/repaint-pseudo-border-on-hover-expected.txt: Added.
> > > +        * fast/repaint/repaint-pseudo-border-on-hover.html: Added.
> > > +
> > 
> > These files are not in the patch
> oops
webkit-patch land is messing with my local state.
Comment 9 EWS 2022-05-14 08:25:41 PDT
Committed r294195 (250561@main): <https://commits.webkit.org/250561@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 459355 [details].
Comment 10 Radar WebKit Bug Importer 2022-05-14 08:26:14 PDT
<rdar://problem/93291192>
Comment 11 Tim Nguyen (:ntim) 2022-05-14 22:40:46 PDT
*** Bug 223654 has been marked as a duplicate of this bug. ***