aw: Move clearView implementaiton to native

aw: Move clearView implementaiton to native

Also untangle clearView implementation from picture listener enabled.
Add some comments and removed some error logs.

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248595

[boliu, 2014-02-03 19:37:38]

A clean up for a feature you added in the olden days :)

[Kristian Monsen, 2014-02-03 19:46:43]

lgtm as long as the tests are still passing

[boliu, 2014-02-03 19:52:40]

On 2014/02/03 19:46:43, Kristian Monsen wrote:
> lgtm as long as the tests are still passing

Ah haha, there were no tests added. This actually is one step closer to allowing
me to unit test all of this stuff.

[boliu, 2014-02-03 19:52:49]

The CQ bit was checked by boliu@chromium.org

[Kristian Monsen, 2014-02-03 19:53:32]

On 2014/02/03 19:52:40, boliu wrote:
> On 2014/02/03 19:46:43, Kristian Monsen wrote:
> > lgtm as long as the tests are still passing
> 
> Ah haha, there were no tests added. This actually is one step closer to
allowing
> me to unit test all of this stuff.

I think it is covered by the cts tests.

[commit-bot: I haz the power, 2014-02-03 19:54:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/boliu@chromium.org/136753015/30001

[boliu, 2014-02-03 19:57:06]

On 2014/02/03 19:53:32, Kristian Monsen wrote:
> On 2014/02/03 19:52:40, boliu wrote:
> > On 2014/02/03 19:46:43, Kristian Monsen wrote:
> > > lgtm as long as the tests are still passing
> > 
> > Ah haha, there were no tests added. This actually is one step closer to
> allowing
> > me to unit test all of this stuff.
> 
> I think it is covered by the cts tests.

Ahh clearView...not covered by cts either. At least nothing in cts calls it.

[Kristian Monsen, 2014-02-03 19:59:19]

On 2014/02/03 19:57:06, boliu wrote:
> On 2014/02/03 19:53:32, Kristian Monsen wrote:
> > On 2014/02/03 19:52:40, boliu wrote:
> > > On 2014/02/03 19:46:43, Kristian Monsen wrote:
> > > > lgtm as long as the tests are still passing
> > > 
> > > Ah haha, there were no tests added. This actually is one step closer to
> > allowing
> > > me to unit test all of this stuff.
> > 
> > I think it is covered by the cts tests.
> 
> Ahh clearView...not covered by cts either. At least nothing in cts calls it.

You are right :-(

[commit-bot: I haz the power, 2014-02-03 22:28:34]

Change committed as 248595

[commit-bot: I haz the power, 2014-02-03 22:28:39]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-02-03 22:28:45]

CQ bit was unchecked on CL. Ignoring.