Fix SampleApp BBH shortcut collision.

Fix SampleApp bounding box hierarchy shortcut collision.

'b' is currently bound to the AA toggle, AND to the bounding box
hierachy toggle.

The CL switches BBH to 'B', and fixes a deferred title update issue.

R=reed@google.com,robertphillips@google.com

Committed: http://code.google.com/p/skia/source/detail?r=12620

[robertphillips, 2013-12-11 16:23:09]

lgtm + nits

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp
File samplecode/SampleApp.cpp (left):

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp#old...
samplecode/SampleApp.cpp:1833: 
Is there any more commented out code that tries to use this in other files?

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp
File samplecode/SampleApp.cpp (right):

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp#new...
samplecode/SampleApp.cpp:1879: // the queue.
SkNEW_ARGS?
Shouldn't this be post_event_to_sink everywhere?

[f(malita), 2013-12-11 16:45:50]

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp
File samplecode/SampleApp.cpp (left):

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp#old...
samplecode/SampleApp.cpp:1833: 
On 2013/12/11 16:23:10, robertphillips wrote:
> Is there any more commented out code that tries to use this in other files?

Nope, I checked.

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp
File samplecode/SampleApp.cpp (right):

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp#new...
samplecode/SampleApp.cpp:1879: // the queue.
On 2013/12/11 16:23:10, robertphillips wrote:
> SkNEW_ARGS?
Done.

> Shouldn't this be post_event_to_sink everywhere?

I think in general it's OK to call updateTitle() synchronously (when whatever
state change is affecting the title has already occurred - which is luckily the
case for most part).

It only gets tricky when using the event queue to signal views that they need to
update some state, like this particular case.

[robertphillips, 2013-12-11 16:47:30]

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp
File samplecode/SampleApp.cpp (right):

https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp#new...
samplecode/SampleApp.cpp:1879: // the queue.
I meant just change the name since it is a global static.

[f(malita), 2013-12-11 16:52:36]

On 2013/12/11 16:47:30, robertphillips wrote:
> https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp
> File samplecode/SampleApp.cpp (right):
> 
>
https://codereview.chromium.org/112903002/diff/1/samplecode/SampleApp.cpp#new...
> samplecode/SampleApp.cpp:1879: // the queue.
> I meant just change the name since it is a global static.

Doh :) Will do.

[commit-bot: I haz the power, 2013-12-11 16:56:31]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/fmalita@chromium.org/112903002/40001

[commit-bot: I haz the power, 2013-12-11 17:05:47]

Change committed as 12620