Remove DrawFace enum from GrPipelineBuilder

Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

Committed: https://skia.googlesource.com/skia/+/da152ebb032c61bb2561bbb3e9a78037ad174d34
Committed: https://skia.googlesource.com/skia/+/5fa7f30331585f78cf0f81ac185abbff1fee1152

[robertphillips, 2016-07-21 14:01:48]

Description was changed from

==========
Remove DrawFace enum from GrPipelineBuilder
==========

to

==========
Remove DrawFace enum from GrPipelineBuilder
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2165283002
==========

[robertphillips, 2016-07-21 14:04:56]

Description was changed from

==========
Remove DrawFace enum from GrPipelineBuilder
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2165283002
==========

to

==========
Remove DrawFace enum from GrPipelineBuilder


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

[robertphillips, 2016-07-21 14:04:56]

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

[robertphillips, 2016-07-21 14:08:30]

Description was changed from

==========
Remove DrawFace enum from GrPipelineBuilder


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

to

==========
Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

[robertphillips, 2016-07-21 14:17:19]

On 2016/07/21 14:16:10, bsalomon wrote:

??

[bsalomon, 2016-07-21 14:17:52]

oooops, lgtm

[robertphillips, 2016-07-21 14:22:15]

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

[commit-bot: I haz the power, 2016-07-21 14:22:20]

Dry run: 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-21 14:49:57]

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

[commit-bot: I haz the power, 2016-07-21 14:49:58]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-07-21 14:57:32]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2016-07-21 14:57:44]

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-21 14:58:43]

Description was changed from

==========
Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

to

==========
Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

Committed:
https://skia.googlesource.com/skia/+/da152ebb032c61bb2561bbb3e9a78037ad174d34
==========

[commit-bot: I haz the power, 2016-07-21 14:58:44]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/da152ebb032c61bb2561bbb3e9a78037ad174d34

[robertphillips, 2016-07-21 15:17:54]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2167193002/ by robertphillips@google.com.

The reason for reverting is: No Vulkan bot in CQ :(.

[robertphillips, 2016-07-21 15:18:34]

Description was changed from

==========
Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

Committed:
https://skia.googlesource.com/skia/+/da152ebb032c61bb2561bbb3e9a78037ad174d34
==========

to

==========
Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

Committed:
https://skia.googlesource.com/skia/+/da152ebb032c61bb2561bbb3e9a78037ad174d34
==========

[robertphillips, 2016-07-21 15:53:58]

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

[commit-bot: I haz the power, 2016-07-21 15:54:05]

Dry run: 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-21 16:13:57]

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

[commit-bot: I haz the power, 2016-07-21 16:13:58]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-07-21 16:19:59]

The CQ bit was checked by robertphillips@google.com

[robertphillips, 2016-07-21 16:19:59]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/2165283002/#ps60001
(title: "try again")

[commit-bot: I haz the power, 2016-07-21 16:20:09]

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-21 16:21:08]

Description was changed from

==========
Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

Committed:
https://skia.googlesource.com/skia/+/da152ebb032c61bb2561bbb3e9a78037ad174d34
==========

to

==========
Remove DrawFace enum from GrPipelineBuilder

This gets GrPipelineBuilder.h out of the headers

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

Committed:
https://skia.googlesource.com/skia/+/da152ebb032c61bb2561bbb3e9a78037ad174d34
Committed:
https://skia.googlesource.com/skia/+/5fa7f30331585f78cf0f81ac185abbff1fee1152
==========

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

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/5fa7f30331585f78cf0f81ac185abbff1fee1152