Fine tune nsCSSRendering::PaintStyleImageLayerWithSC & SetupImageLayerClip

RESOLVED FIXED in Firefox 54

Status

()

defect
RESOLVED FIXED
2 years ago
11 months ago

People

(Reporter: u459114, Assigned: u459114)

Tracking

unspecified
mozilla54
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox54 fixed)

Details

Attachments

(4 attachments)

Reduce the complexity inside this funciton
1. ImageLayerClipState::mHasRoundedCorners can be true only if ImageLayerClipState::mClippedRadii is not empty

2. The last parameter of SetupImageLayerClip is verbose.
Summary: Fine tune nsCSSRendering::PaintStyleImageLayerWithSC → Fine tune nsCSSRendering::PaintStyleImageLayerWithSC & SetupImageLayerClip
Attachment #8840777 - Flags: review?(mstange)
Attachment #8840778 - Flags: review?(mstange)
Attachment #8840779 - Flags: review?(mstange)
Attachment #8840780 - Flags: review?(mstange)
Comment on attachment 8840777 [details]
Bug 1342302 - Part 1. Implement RectCornerRadii::IsEmpty.

https://reviewboard.mozilla.org/r/115204/#review116758
Attachment #8840777 - Flags: review?(mstange) → review+
Comment on attachment 8840778 [details]
Bug 1342302 - Part 2. Implement ImageLayerClipState::IsValid.

https://reviewboard.mozilla.org/r/115206/#review116760
Attachment #8840778 - Flags: review?(mstange) → review+
Comment on attachment 8840779 [details]
Bug 1342302 - Part 3. Rename ImageLayerClipState::mDirtyRectGfx to mDirtyRectInDevPx.

https://reviewboard.mozilla.org/r/115208/#review116764

::: layout/painting/nsCSSRendering.h:582
(Diff revision 3)
> -    nsRect mDirtyRect;
> -    gfxRect mDirtyRectGfx;
> +    nsRect mDirtyRectInAppUnit;
> +    gfxRect mDirtyRectInDevUnit;

I would prefer the names mDirtyRectInAppUnits and mDirtyRectInDevPx.
Attachment #8840779 - Flags: review?(mstange) → review+
Comment on attachment 8840780 [details]
Bug 1342302 - Part 4. Fine tune nsCSSRendering::PaintStyleImageLayerWithSC.

https://reviewboard.mozilla.org/r/115210/#review116864
Attachment #8840780 - Flags: review?(mstange) → review+
Pushed by cku@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/14fd552c865b
Part 1. Implement RectCornerRadii::IsEmpty. r=mstange
https://hg.mozilla.org/integration/autoland/rev/f4ac43e3f67d
Part 2. Implement ImageLayerClipState::IsValid. r=mstange
https://hg.mozilla.org/integration/autoland/rev/a1deec7b451a
Part 3. Rename ImageLayerClipState::mDirtyRectGfx to mDirtyRectInDevPx. r=mstange
https://hg.mozilla.org/integration/autoland/rev/1b8511b2ca4c
Part 4. Fine tune nsCSSRendering::PaintStyleImageLayerWithSC. r=mstange
Blocks: 1373750
No longer blocks: 1373750
Depends on: 1373750
Component: Layout: View Rendering → Layout: Web Painting
You need to log in before you can comment on or make changes to this bug.