Adding policy settings for Chrome Frame.

Adding policy settings for Chrome Frame.

TEST=none
BUG=29349

[tommi (sloooow) - chröme, 2010-09-03 20:21:02]

FYI - I added two more files to the change

On Fri, Sep 3, 2010 at 4:15 PM, <tommi@chromium.org> wrote:

> Reviewers: amit,
>
> Description:
> Adding policy settings for Chrome Frame.
>
> TEST=none
> BUG=29349
>
>
> Please review this at http://codereview.chromium.org/3354006/show
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
>
> Affected files:
>  M     chrome/app/policy/policy_templates.grd
>  M     chrome/app/policy/policy_templates.json
>
>
>

[amit, 2010-09-03 20:35:40]

lg

[gfeher, 2010-09-07 16:07:32]

http://codereview.chromium.org/3354006/diff/10001/11002
File chrome/app/policy/policy_templates.json (right):

http://codereview.chromium.org/3354006/diff/10001/11002#newcode246
chrome/app/policy/policy_templates.json:246: },
Note that if you put policies into the same group, then its not possible to set
their values to unspecified independently. (In case of ADM/ADMX.) Each of them
will be specified or unspecified at the same time. It's okay if this is what you
want.

[tommi (sloooow) - chröme, 2010-09-07 16:15:58]

http://codereview.chromium.org/3354006/diff/10001/11002
File chrome/app/policy/policy_templates.json (right):

http://codereview.chromium.org/3354006/diff/10001/11002#newcode246
chrome/app/policy/policy_templates.json:246: },
On 2010/09/07 16:07:32, gfeher wrote:
> Note that if you put policies into the same group, then its not possible to
set
> their values to unspecified independently. (In case of ADM/ADMX.) Each of them
> will be specified or unspecified at the same time. It's okay if this is what
you
> want.

In this case, RendererSettings is essentially a switch between these two lists. 
If RendererSettings is 0, then the RenderInChromeFrame list is used for
exclusion, if it is 1, the RenderInHostList is the exclusion list.  So I think
that's what we want.

[tommi (sloooow) - chröme, 2010-09-07 19:29:34]

This change is being reverted and parts of it have been moved over to here:
http://codereview.chromium.org/3325011/show