Closed Bug 1091787 Opened 10 years ago Closed 10 years ago

Port the code that uses gfxContext::RoundedRectangle to Moz2D

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: jwatt, Assigned: jwatt)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

      No description provided.
Depends on: 1091794
Attachment #8514461 - Flags: review?(matt.woodrow)
Attached patch nsCSSRenderingSplinter Review
Attachment #8514463 - Flags: review?(matt.woodrow)
Attachment #8514464 - Flags: review?(matt.woodrow)
Attachment #8514461 - Flags: review?(matt.woodrow) → review+
Comment on attachment 8514463 [details] [diff] [review]
nsCSSRendering

Review of attachment 8514463 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/base/nsCSSRendering.cpp
@@ +1548,5 @@
>                 shadowContext == blurringArea.GetContext());
>  
>      // Set the shadow color; if not specified, use the foreground color
> +    Color shadowColor = Color::FromABGR(shadowItem->mHasColor ?
> +                                        shadowItem->mColor :

Indent this one more so it doesn't look like a second param to FromABGR.
Attachment #8514463 - Flags: review?(matt.woodrow) → review+
Attachment #8514464 - Flags: review?(matt.woodrow) → review+
Comment on attachment 8514461 [details] [diff] [review]
nsCSSRenderingBorders

I had to add some fuzz for this.

https://hg.mozilla.org/integration/mozilla-inbound/rev/45638c4e2cd7
Attachment #8514461 - Flags: checkin+
Depends on: 1094063
Depends on: 1094480
Depends on: 1097119
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: