Reland add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs.

Add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs (Patch set 1 Reverts revert of http://codereview.chromium.org/7670041/)

This adds the flag --use-more-webui to allow turning on WebUI replacements for native dialogs. This flag is automatically set when --use-pure-views is specified. Modifies replaced native dialogs to be toggled based on flag value. BookmarkEditor has also been refactored to use static constructors to EditDetails to describe what is being edited to make the code more readable.

BUG=96935
TEST=Tested that webui dialogs can be turned on or left off with --use-more-webui.


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

[flackr, 2011-09-20 02:22:00]

There was an issue on the linux_clang builder with the last commit. The revert
of the revert (patch set 1) is the code that you have already reviewed. Patch
set 2 should fix the clang build issue.

[flackr, 2011-09-20 02:26:30]

The first clang trybot run should have failed with the same issue that occurred
on the main build bot with the previous commit:
http://build.chromium.org/p/chromium/builders/Linux%20Clang%20%28dbg%29/build...
In file included from chrome/browser/ui/webui/certificate_viewer.cc:13:
./chrome/browser/ui/webui/certificate_viewer.h:23:1: error: [chromium-style]
Complex class/struct needs an explicit out-of-line destructor.
class CertificateViewerDialog : private HtmlDialogUIDelegate {
^
./chrome/browser/ui/webui/certificate_viewer.h:64:1: error: [chromium-style]
Complex class/struct needs an explicit out-of-line destructor.
class CertificateViewerDialogHandler : public WebUIMessageHandler {
^
  CXX(target)
out/Debug/obj.target/browser/chrome/browser/ui/webui/chrome_web_ui.o
2 errors generated.
make: ***
[out/Debug/obj.target/browser/chrome/browser/ui/webui/certificate_viewer.o]
Error 1

I filed an issue on crbug about this.

[Rick Byers, 2011-09-20 13:21:33]

Trivial addition of the dtor LGTM.

Sucks that the clang trybot didn't hit the same error as the clang buildbot -
thanks for filing a bug to get this fixed.

[Rick Campbell, 2011-09-20 14:43:11]

bookmarks_helper.cc still LGTM (as Sync owner)

[commit-bot: I haz the power, 2011-09-21 02:03:40]

Change committed as 102071