Closed Bug 1103621 Opened 9 years ago Closed 9 years ago

Get rid of OPERATOR_CLEAR

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: jwatt, Assigned: jwatt)

Details

Attachments

(1 file)

      No description provided.
Attached patch patchSplinter Review
Attachment #8527330 - Flags: review?(matt.woodrow)
Comment on attachment 8527330 [details] [diff] [review]
patch

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

::: gfx/thebes/gfxContext.cpp
@@ -507,5 @@
>  void
>  gfxContext::SetOperator(GraphicsOperator op)
>  {
> -  if (op == OPERATOR_CLEAR) {
> -    CurrentState().opIsClear = true;

Can't we get rid of the opIsClear variable too?
Attachment #8527330 - Flags: review?(matt.woodrow) → review+
Comment on attachment 8527330 [details] [diff] [review]
patch

https://hg.mozilla.org/integration/mozilla-inbound/rev/e43fd991c15c

(In reply to Matt Woodrow (:mattwoodrow) from comment #2)
> Can't we get rid of the opIsClear variable too?

Thanks, did so.
Attachment #8527330 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/e43fd991c15c
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.