views: add PlatformStyle docs

views: add PlatformStyle docs

Add advice documentation for dialog and control authors about how to use
PlatformStyle.

BUG=

Committed: https://crrev.com/dc3cb5d2837dc71bd904f32876e5a743e6b6f1e1
Cr-Commit-Position: refs/heads/master@{#392303}

[Elly Fong-Jones, 2016-05-05 21:32:42]

Description was changed from

==========
views: add PlatformStyle docs

Add advice documentation for dialog and control authors about how to use
PlatformStyle.

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

to

==========
views: add PlatformStyle docs

Add advice documentation for dialog and control authors about how to use
PlatformStyle.

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

[Elly Fong-Jones, 2016-05-05 21:32:43]

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

[Elly Fong-Jones, 2016-05-05 21:32:58]

sky: ptal? :) These are still sort of rough and have a couple of TODOs.

[sky, 2016-05-05 23:27:43]

https://codereview.chromium.org/1960443002/diff/1/docs/ui/views/platform_styl...
File docs/ui/views/platform_style.md (right):

https://codereview.chromium.org/1960443002/diff/1/docs/ui/views/platform_styl...
docs/ui/views/platform_style.md:67: ### Use subclassing to add complex behavior
switches
Do you have an example of this? I could see this for focus/border/painting, but
in general we shouldn't subclass controls per platform. If we did, it makes
subclassing impossible.

[Elly Fong-Jones, 2016-05-06 19:23:29]

sky: ptal? :)

https://codereview.chromium.org/1960443002/diff/1/docs/ui/views/platform_styl...
File docs/ui/views/platform_style.md (right):

https://codereview.chromium.org/1960443002/diff/1/docs/ui/views/platform_styl...
docs/ui/views/platform_style.md:67: ### Use subclassing to add complex behavior
switches
On 2016/05/05 23:27:43, sky wrote:
> Do you have an example of this? I could see this for focus/border/painting,
but
> in general we shouldn't subclass controls per platform. If we did, it makes
> subclassing impossible.

Done.

[sky, 2016-05-06 21:40:03]

LGTM

[Elly Fong-Jones, 2016-05-09 13:23:54]

The CQ bit was checked by ellyjones@chromium.org

[commit-bot: I haz the power, 2016-05-09 13:24:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960443002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960443002/20001

[commit-bot: I haz the power, 2016-05-09 13:56:03]

Description was changed from

==========
views: add PlatformStyle docs

Add advice documentation for dialog and control authors about how to use
PlatformStyle.

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

to

==========
views: add PlatformStyle docs

Add advice documentation for dialog and control authors about how to use
PlatformStyle.

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

[commit-bot: I haz the power, 2016-05-09 13:56:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-09 13:57:18]

Description was changed from

==========
views: add PlatformStyle docs

Add advice documentation for dialog and control authors about how to use
PlatformStyle.

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

to

==========
views: add PlatformStyle docs

Add advice documentation for dialog and control authors about how to use
PlatformStyle.

BUG=

Committed: https://crrev.com/dc3cb5d2837dc71bd904f32876e5a743e6b6f1e1
Cr-Commit-Position: refs/heads/master@{#392303}
==========

[commit-bot: I haz the power, 2016-05-09 13:57:19]

Patchset 2 (id:??) landed as
https://crrev.com/dc3cb5d2837dc71bd904f32876e5a743e6b6f1e1
Cr-Commit-Position: refs/heads/master@{#392303}