Remove always true if statement and commented out code in GrBitmapTextContext

Remove always true if statement and commented out code in GrBitmapTextContext

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/06fb35fe073f4bf213eb6a48498a1319bcff6cf2

[qiankun, 2015-01-28 05:01:06]

qiankun.miao@intel.com changed reviewers:
+ reed@google.com

[qiankun, 2015-01-28 05:01:07]

Hi reed, please help to review this CL. What does "SkCLZ(3)" do here? I didn't
understand the comments for debug.

[qiankun1, 2015-01-29 14:57:22]

On 2015/01/28 05:01:07, qiankun wrote:
> Hi reed, please help to review this CL. What does "SkCLZ(3)" do here? I didn't
> understand the comments for debug.

reed@, does this CL make sense?

[reed1, 2015-01-29 15:56:57]

reed@google.com changed reviewers:
+ jvanverth@google.com

[reed1, 2015-01-29 15:56:57]

The "true" and "CLZ" removals seem fine. What happened to the NULL check on
fPlot?

[qiankun1, 2015-01-29 16:20:49]

On 2015/01/29 15:56:57, reed1 wrote:
> The "true" and "CLZ" removals seem fine. What happened to the NULL check on
> fPlot?

I think it should return if quichReject returns true. At least it's current
status in the code.

[jvanverth1, 2015-01-29 16:38:20]

On 2015/01/29 16:20:49, qiankun1 wrote:
> On 2015/01/29 15:56:57, reed1 wrote:
> > The "true" and "CLZ" removals seem fine. What happened to the NULL check on
> > fPlot?
> 
> I think it should return if quichReject returns true. At least it's current
> status in the code.

I believe the SkCLZ(3) was there so you have an instruction to put a breakpoint
on if necessary (it was added before I started working on it). I don't see the
point at the moment. There's also one in GrDistanceFieldTextContext that could
be removed as well. Otherwise, LGTM.

[qiankun1, 2015-01-29 17:06:17]

On 2015/01/29 16:38:20, jvanverth1 wrote:
> On 2015/01/29 16:20:49, qiankun1 wrote:
> > On 2015/01/29 15:56:57, reed1 wrote:
> > > The "true" and "CLZ" removals seem fine. What happened to the NULL check
on
> > > fPlot?
> > 
> > I think it should return if quichReject returns true. At least it's current
> > status in the code.
> 
> I believe the SkCLZ(3) was there so you have an instruction to put a
breakpoint
> on if necessary (it was added before I started working on it). I don't see the
> point at the moment. There's also one in GrDistanceFieldTextContext that could
> be removed as well. Otherwise, LGTM.

Thanks for review. Remove it in another CL:
https://codereview.chromium.org/888533004/.

[qiankun, 2015-01-30 02:28:34]

The CQ bit was checked by qiankun.miao@intel.com

[commit-bot: I haz the power, 2015-01-30 02:29:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/873473008/1

[commit-bot: I haz the power, 2015-01-30 02:36:56]

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