DOMUI Settings: UTH: Fix the wrapping of the privacy explanation text.

DOMUI Settings: UTH: Fix the wrapping of the privacy explanation text.

BUG=63835
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=69557

[James Hawkins, 2010-12-16 02:34:00]

http://codereview.chromium.org/5930003/diff/2001/chrome/app/chromium_strings.grd
File chrome/app/chromium_strings.grd (right):

http://codereview.chromium.org/5930003/diff/2001/chrome/app/chromium_strings....
chrome/app/chromium_strings.grd:401: <!--TODO(kmadhusu): Remove/delete
"IDS_OPTIONS_DISABLE_SERVICES" after DOMUI Settings is complete. It is used only
in native options dialog box.-->
nit: Just 'Remove' is fine.

http://codereview.chromium.org/5930003/diff/2001/chrome/app/chromium_strings....
chrome/app/chromium_strings.grd:401: <!--TODO(kmadhusu): Remove/delete
"IDS_OPTIONS_DISABLE_SERVICES" after DOMUI Settings is complete. It is used only
in native options dialog box.-->
"...after platform-specific dialogs are removed."

http://codereview.chromium.org/5930003/diff/2001/chrome/app/google_chrome_str...
File chrome/app/google_chrome_strings.grd (right):

http://codereview.chromium.org/5930003/diff/2001/chrome/app/google_chrome_str...
chrome/app/google_chrome_strings.grd:447: <!-- TODO(kmadhusu): Remove/delete
"IDS_OPTIONS_DISABLE_SERVICES" after DOMUI Settings is complete. It is used only
in native options dialog box.-->
Just 'Remove'.

http://codereview.chromium.org/5930003/diff/2001/chrome/browser/resources/opt...
File chrome/browser/resources/options/advanced_options.html (right):

http://codereview.chromium.org/5930003/diff/2001/chrome/browser/resources/opt...
chrome/browser/resources/options/advanced_options.html:13:
class="disable-services-div">
In advanced_options.css, rename '.disable-services-div' to '.disable-services',
and just use that one class for both div and span.

[kmadhusu, 2010-12-16 17:41:37]

http://codereview.chromium.org/5930003/diff/2001/chrome/app/chromium_strings.grd
File chrome/app/chromium_strings.grd (right):

http://codereview.chromium.org/5930003/diff/2001/chrome/app/chromium_strings....
chrome/app/chromium_strings.grd:401: <!--TODO(kmadhusu): Remove/delete
"IDS_OPTIONS_DISABLE_SERVICES" after DOMUI Settings is complete. It is used only
in native options dialog box.-->
On 2010/12/16 02:34:01, James Hawkins wrote:
> nit: Just 'Remove' is fine.

Done.

http://codereview.chromium.org/5930003/diff/2001/chrome/app/chromium_strings....
chrome/app/chromium_strings.grd:401: <!--TODO(kmadhusu): Remove/delete
"IDS_OPTIONS_DISABLE_SERVICES" after DOMUI Settings is complete. It is used only
in native options dialog box.-->
On 2010/12/16 02:34:01, James Hawkins wrote:
> "...after platform-specific dialogs are removed."

Done.

http://codereview.chromium.org/5930003/diff/2001/chrome/app/google_chrome_str...
File chrome/app/google_chrome_strings.grd (right):

http://codereview.chromium.org/5930003/diff/2001/chrome/app/google_chrome_str...
chrome/app/google_chrome_strings.grd:447: <!-- TODO(kmadhusu): Remove/delete
"IDS_OPTIONS_DISABLE_SERVICES" after DOMUI Settings is complete. It is used only
in native options dialog box.-->
On 2010/12/16 02:34:01, James Hawkins wrote:
> Just 'Remove'.

Done.

http://codereview.chromium.org/5930003/diff/2001/chrome/browser/resources/opt...
File chrome/browser/resources/options/advanced_options.html (right):

http://codereview.chromium.org/5930003/diff/2001/chrome/browser/resources/opt...
chrome/browser/resources/options/advanced_options.html:13:
class="disable-services-div">
On 2010/12/16 02:34:01, James Hawkins wrote:
> In advanced_options.css, rename '.disable-services-div' to
'.disable-services',
> and just use that one class for both div and span.

Done.

[James Hawkins, 2010-12-16 21:26:58]

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
File chrome/browser/resources/options/advanced_options.css (right):

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
chrome/browser/resources/options/advanced_options.css:7: .disable-services {
I'm not sure I like this class name.  It doesn't reflect the purpose of the grey
color.  Maybe informational-text is better.

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
File chrome/browser/resources/options/advanced_options.html (right):

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
chrome/browser/resources/options/advanced_options.html:13:
class="disable-services">
This can fit on the previous line in 80 cols.

[kmadhusu, 2010-12-16 22:08:58]

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
File chrome/browser/resources/options/advanced_options.css (right):

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
chrome/browser/resources/options/advanced_options.css:7: .disable-services {
On 2010/12/16 21:26:58, James Hawkins wrote:
> I'm not sure I like this class name.  It doesn't reflect the purpose of the
grey
> color.  Maybe informational-text is better.

Done.

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
File chrome/browser/resources/options/advanced_options.html (right):

http://codereview.chromium.org/5930003/diff/10001/chrome/browser/resources/op...
chrome/browser/resources/options/advanced_options.html:13:
class="disable-services">
On 2010/12/16 21:26:58, James Hawkins wrote:
> This can fit on the previous line in 80 cols.

Done.

[James Hawkins, 2010-12-16 22:21:26]

LGTM