Site Settings: Remove FullScreen as top-level category.

Site Settings: Remove FullScreen as top-level category.

Note: It is still accessible through the Site Details, if you look
it up per site, but it doesn't need to be a top-level category.

This CL also cleans up a few obsolete prefs-specific functions
in the Behavior.

BUG=543635
Committed: https://crrev.com/f3e513453ca688808092ce04405e6ef92f510b71
Cr-Commit-Position: refs/heads/master@{#388179}

[Finnur, 2016-04-18 15:22:32]

Description was changed from

==========
Remove FullScreen as top-level category.

Note: It is still accessible through the Site Details, if you look it up
per site, but it doesn't need to be a top-level category.

This CL also cleans up a few obsolete prefs-specific functions in the Behavior.

BUG=543635
==========

to

==========
Site Settings: Remove FullScreen as top-level category.

Note: It is still accessible through the Site Details, if you look
it up per site, but it doesn't need to be a top-level category.

This CL also cleans up a few obsolete prefs-specific functions
in the Behavior.

BUG=543635
==========

[Finnur, 2016-04-18 15:22:32]

finnur@chromium.org changed reviewers:
+ tommycli@chromium.org

[tommycli, 2016-04-18 16:14:12]

lgtm assuming these changes are driven by modifications to the mocks? / with
discussion with bettes@?

[Finnur, 2016-04-19 10:41:13]

FullScreen is going away as a specific permission, which is an effort driven by
mgiuca@ and discussed among me, him, Elisabeth (PM) and Dan Beam (who noticed an
effort to remove it and asked I be kept in the loop).

It will take a bit of time to fully remove this permission, and in the meantime
it should still show up on Site Details page. But there's no reason to have it
be a top-level category.

[Finnur, 2016-04-19 10:41:31]

The CQ bit was checked by finnur@chromium.org

[commit-bot: I haz the power, 2016-04-19 10:41:44]

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

[commit-bot: I haz the power, 2016-04-19 10:44:53]

Description was changed from

==========
Site Settings: Remove FullScreen as top-level category.

Note: It is still accessible through the Site Details, if you look
it up per site, but it doesn't need to be a top-level category.

This CL also cleans up a few obsolete prefs-specific functions
in the Behavior.

BUG=543635
==========

to

==========
Site Settings: Remove FullScreen as top-level category.

Note: It is still accessible through the Site Details, if you look
it up per site, but it doesn't need to be a top-level category.

This CL also cleans up a few obsolete prefs-specific functions
in the Behavior.

BUG=543635
==========

[commit-bot: I haz the power, 2016-04-19 10:44:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-19 10:46:34]

Description was changed from

==========
Site Settings: Remove FullScreen as top-level category.

Note: It is still accessible through the Site Details, if you look
it up per site, but it doesn't need to be a top-level category.

This CL also cleans up a few obsolete prefs-specific functions
in the Behavior.

BUG=543635
==========

to

==========
Site Settings: Remove FullScreen as top-level category.

Note: It is still accessible through the Site Details, if you look
it up per site, but it doesn't need to be a top-level category.

This CL also cleans up a few obsolete prefs-specific functions
in the Behavior.

BUG=543635

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

[commit-bot: I haz the power, 2016-04-19 10:46:35]

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