Land 3122014 for Anton: Initial change that allows to disable bookmarks

Land 3122014 for Anton: Initial change that allows to disable bookmarks

Original CL: http://codereview.chromium.org/3122014

Initial change that allows to disable bookmarks in Chrome for Chrome OS
(BWSI mode). Added disable-bookmarks flag and disabled some UI elements.

BUG=chromium-os:4302
TEST=Run Chrome build for Chrome OS with --disable-bookmarks flag and
verify that the corresponding UI is disabled. Verify that BWSI mode runs
with this flag.

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

[sky, 2010-08-18 16:12:09]

http://codereview.chromium.org/3191007/diff/1/2
File chrome/app/chrome_dll_main.cc (right):

http://codereview.chromium.org/3191007/diff/1/2#newcode576
chrome/app/chrome_dll_main.cc:576:
singleton_command_line->AppendSwitch(switches::kDisableBookmarks);
See comments in other review. I don't think you need to add a new command line
switch for this.

http://codereview.chromium.org/3191007/diff/1/3
File chrome/browser/browser.cc (right):

http://codereview.chromium.org/3191007/diff/1/3#newcode1095
chrome/browser/browser.cc:1095: browser_defaults::bookmarks_enabled &&
show_main_ui);
Much better. Thank you!

[Dmitry Polukhin, 2010-08-18 18:33:40]

http://codereview.chromium.org/3191007/diff/1/2
File chrome/app/chrome_dll_main.cc (right):

http://codereview.chromium.org/3191007/diff/1/2#newcode576
chrome/app/chrome_dll_main.cc:576:
singleton_command_line->AppendSwitch(switches::kDisableBookmarks);
On 2010/08/18 16:12:10, sky wrote:
> See comments in other review. I don't think you need to add a new command line
> switch for this.

Done.

[sky, 2010-08-18 20:39:00]

LGTM