MD Settings: move AdvancedOptionsUtilities to a settings-accessible place

MD Settings: move AdvancedOptionsUtilities to a settings-accessible place

Also, move from class + static methods to simply namespaced methods.
Doesn't make much sense to have a static only class with more code and
longer identifiers, but I could be convinced if there's a reason.

R=dpapad@chromium.org
BUG=546844
Committed: https://crrev.com/71382ce7fc4861fcddc6e261438a2cf6e70d919d
Cr-Commit-Position: refs/heads/master@{#380765}

[Dan Beam, 2016-03-11 20:28:57]

https://codereview.chromium.org/1789543003/diff/1/chrome/browser/ui/webui/set...
File chrome/browser/ui/webui/settings_utils.h (right):

https://codereview.chromium.org/1789543003/diff/1/chrome/browser/ui/webui/set...
chrome/browser/ui/webui/settings_utils.h:20: void
ShowManageSSLCertificates(content::WebContents* web_contents);
this is why i sent to you

[dpapad, 2016-03-11 21:13:06]

LGTM

https://codereview.chromium.org/1789543003/diff/1/chrome/browser/ui/webui/set...
File chrome/browser/ui/webui/settings_utils.h (right):

https://codereview.chromium.org/1789543003/diff/1/chrome/browser/ui/webui/set...
chrome/browser/ui/webui/settings_utils.h:20: void
ShowManageSSLCertificates(content::WebContents* web_contents);
On 2016/03/11 at 20:28:56, Dan Beam wrote:
> this is why i sent to you

I see. For now the md-settings page always opens the WebUI certificate manager.
I guess eventually WIN/MAC need to differentiate.

[Dan Beam, 2016-03-11 21:55:04]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-03-11 21:55:31]

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

[commit-bot: I haz the power, 2016-03-11 22:42:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-11 22:43:56]

Description was changed from

==========
MD Settings: move AdvancedOptionsUtilities to a settings-accessible place

Also, move from class + static methods to simply namespaced methods.
Doesn't make much sense to have a static only class with more code and
longer identifiers, but I could be convinced if there's a reason.

R=dpapad@chromium.org
BUG=546844
==========

to

==========
MD Settings: move AdvancedOptionsUtilities to a settings-accessible place

Also, move from class + static methods to simply namespaced methods.
Doesn't make much sense to have a static only class with more code and
longer identifiers, but I could be convinced if there's a reason.

R=dpapad@chromium.org
BUG=546844

Committed: https://crrev.com/71382ce7fc4861fcddc6e261438a2cf6e70d919d
Cr-Commit-Position: refs/heads/master@{#380765}
==========

[commit-bot: I haz the power, 2016-03-11 22:43:57]

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