Fix misplaced guard

Fix misplaced guard

This fixes an error in https://codereview.chromium.org/2125333002 (Add choke point for modifying non-AA rect draws (e.g., applying clipping)).

In GrDrawContext::drawRect, when we need to draw a filled rect, we always want to call drawFilledRect when useDrawInsteadOfClear is false. With the buggy code I believe we were falling back to drawing a path.

TBR=bsalomon@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002

Committed: https://skia.googlesource.com/skia/+/3ab14ca950d98a5c4d4c4a5f96ea8b0c984e0116

[robertphillips, 2016-07-10 18:20:18]

Description was changed from

==========
Fix misplaced guard
==========

to

==========
Fix misplaced guard
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

[robertphillips, 2016-07-10 18:21:07]

Description was changed from

==========
Fix misplaced guard
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

to

==========
Fix misplaced guard
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

[robertphillips, 2016-07-10 18:21:08]

robertphillips@google.com changed reviewers:
+ bsalomon@google.com

[robertphillips, 2016-07-10 18:21:33]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-10 18:21:35]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[robertphillips, 2016-07-10 18:25:33]

Description was changed from

==========
Fix misplaced guard
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

to

==========
Fix misplaced guard

This fixes an error in https://codereview.chromium.org/2125333002 (Add choke
point for modifying non-AA rect draws (e.g., applying clipping)).

In GrDrawContext::drawRect, when we need to draw a filled rect, we always want
to call drawFilledRect when useDrawInsteadOfClear is false. With the buggy code
I believe we were falling back to drawing a path.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

[robertphillips, 2016-07-10 18:27:12]

Description was changed from

==========
Fix misplaced guard

This fixes an error in https://codereview.chromium.org/2125333002 (Add choke
point for modifying non-AA rect draws (e.g., applying clipping)).

In GrDrawContext::drawRect, when we need to draw a filled rect, we always want
to call drawFilledRect when useDrawInsteadOfClear is false. With the buggy code
I believe we were falling back to drawing a path.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

to

==========
Fix misplaced guard

This fixes an error in https://codereview.chromium.org/2125333002 (Add choke
point for modifying non-AA rect draws (e.g., applying clipping)).

In GrDrawContext::drawRect, when we need to draw a filled rect, we always want
to call drawFilledRect when useDrawInsteadOfClear is false. With the buggy code
I believe we were falling back to drawing a path.

TBR=bsalomon@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

[commit-bot: I haz the power, 2016-07-10 18:40:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-10 18:40:38]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-07-10 18:48:45]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2016-07-10 18:48:49]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-10 18:49:41]

Description was changed from

==========
Fix misplaced guard

This fixes an error in https://codereview.chromium.org/2125333002 (Add choke
point for modifying non-AA rect draws (e.g., applying clipping)).

In GrDrawContext::drawRect, when we need to draw a filled rect, we always want
to call drawFilledRect when useDrawInsteadOfClear is false. With the buggy code
I believe we were falling back to drawing a path.

TBR=bsalomon@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002
==========

to

==========
Fix misplaced guard

This fixes an error in https://codereview.chromium.org/2125333002 (Add choke
point for modifying non-AA rect draws (e.g., applying clipping)).

In GrDrawContext::drawRect, when we need to draw a filled rect, we always want
to call drawFilledRect when useDrawInsteadOfClear is false. With the buggy code
I believe we were falling back to drawing a path.

TBR=bsalomon@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2135883002

Committed:
https://skia.googlesource.com/skia/+/3ab14ca950d98a5c4d4c4a5f96ea8b0c984e0116
==========

[commit-bot: I haz the power, 2016-07-10 18:49:42]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/3ab14ca950d98a5c4d4c4a5f96ea8b0c984e0116