Remove View::focusable() - Alternate approach.

Remove View::focusable()

This CL removes View::focusable(). The existing usages of View::focusable()
in the Label class are changed to use IsFocusable(). This necessiates adding 
labels to the view hierarchy before using their GetInsets() value in 
StyledLabel::CalculateAndDoLayout(), as well as resetting the label layout,
whenever it is made visible or enabled.

[karandeepb, 2016-04-27 07:27:09]

Description was changed from

==========
----


----


----


git squash commit.

acf4861e1b070aafd23542a306ea856aa6e79f02
--

BUG=
==========

to

==========
Remove View::focusable()

This CL removes View::focusable(). The existing usages of View::focusable()
in the Label class are changed to use IsFocusable(). This necessiates adding 
labels to the view hierarchy before using their GetInsets() value in 
StyledLabel::CalculateAndDoLayout().
==========

[karandeepb, 2016-04-27 07:37:54]

Patchset #1 (id:1) has been deleted

[karandeepb, 2016-04-27 07:38:46]

Patchset #1 (id:20001) has been deleted

[karandeepb, 2016-04-27 07:38:57]

Patchset #1 (id:40001) has been deleted

[karandeepb, 2016-04-27 07:39:12]

Patchset #1 (id:60001) has been deleted

[karandeepb, 2016-04-27 07:59:31]

Description was changed from

==========
Remove View::focusable()

This CL removes View::focusable(). The existing usages of View::focusable()
in the Label class are changed to use IsFocusable(). This necessiates adding 
labels to the view hierarchy before using their GetInsets() value in 
StyledLabel::CalculateAndDoLayout().
==========

to

==========
Remove View::focusable()

This CL removes View::focusable(). The existing usages of View::focusable()
in the Label class are changed to use IsFocusable(). This necessiates adding 
labels to the view hierarchy before using their GetInsets() value in 
StyledLabel::CalculateAndDoLayout(), as well as resetting the label layout,
whenever it is made visible or enabled.
==========

[karandeepb, 2016-04-27 23:53:17]

karandeepb@chromium.org changed reviewers:
+ sky@chromium.org

[karandeepb, 2016-04-27 23:53:17]

PTAL sky@. This approach uses IsFocusable. However, I don't like this approach
much, since the code in styled_label now seems a bit hacky. Maybe we can just
expose View::focus_behavior_?

https://codereview.chromium.org/1915863004/diff/80001/ui/views/controls/label.cc
File ui/views/controls/label.cc (right):

https://codereview.chromium.org/1915863004/diff/80001/ui/views/controls/label...
ui/views/controls/label.cc:338: // will return a different value. Hence reset
layout.
I am not totally sure whether this is needed or not.