Remove DialogDelegate::UseNewStyle and unused code.

chrome/browser/ui/views/autofill/autofill_dialog_views.cc

Index: chrome/browser/ui/views/autofill/autofill_dialog_views.cc
diff --git a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
index 65620bfdf844c259b685a99a0886e79d8459bf9d..5931ba80409502c01f2a97e2e8d897c9e4d37a11 100644
--- a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
+++ b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
@@ -1195,12 +1195,8 @@ void AutofillDialogViews::Show() {
           delegate_->GetWebContents());
   WebContentsModalDialogManagerDelegate* modal_delegate =
       web_contents_modal_dialog_manager->delegate();
-  DCHECK(modal_delegate);

[sky, 2013/11/16 03:03:16]

How come you removed this?

[msw, 2013/11/16 04:15:44]

I removed all these DCHECKs because I thought we typically discouraged DCHECKing
against null pointers immediately before dereferencing them. We crash either way
and it'd only help in this case because it's checked on a distinct line from the
call to CreateWindowAsFramelessChild. Anyway, it's not a big deal, so I restored
them all in response to your concern/inquiry.

[sky, 2013/11/18 17:03:10]

I like DCHECKs like this as a way to document modal_delegate should never be
NULL.

-
   window_ = views::Widget::CreateWindowAsFramelessChild(
-      this,
-      delegate_->GetWebContents()->GetView()->GetNativeView(),
-      modal_delegate->GetWebContentsModalDialogHost()->GetHostView());
+      this, modal_delegate->GetWebContentsModalDialogHost()->GetHostView());
   web_contents_modal_dialog_manager->ShowDialog(window_->GetNativeView());
   focus_manager_ = window_->GetFocusManager();
   focus_manager_->AddFocusChangeListener(this);