Introduce NativeWebContentsModalDialogManagerDelegate interface

Introduce NativeWebContentsModalDialogManagerDelegate interface

This CL is part 1 of 6 in the work to remove the WebContentsModalDialog
interface.  It introduces an interface from the
NativeWebContentsModalDialogManager back to the WebContentsModalDialogManager,
to decouple these two classes and facilitate independent testing.

BUG=157161
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182942

[Mike Wittman, 2013-02-16 00:43:21]

Hi Ben, my next CLs build on top the of opaque type work in
https://codereview.chromium.org/12221055 that's currently going through the CQ.

I've got a number of changes pending and will be sending them out in a series of
smaller reviews in an attempt to make them easier to follow.

[Ben Goodger (Google), 2013-02-16 00:53:16]

lgtm

https://codereview.chromium.org/12281012/diff/1/chrome/browser/ui/native_web_...
File chrome/browser/ui/native_web_contents_modal_dialog_manager.h (right):

https://codereview.chromium.org/12281012/diff/1/chrome/browser/ui/native_web_...
chrome/browser/ui/native_web_contents_modal_dialog_manager.h:17:
~NativeWebContentsModalDialogManagerDelegate() {}
virtual dtor

[Mike Wittman, 2013-02-16 01:41:15]

https://codereview.chromium.org/12281012/diff/1/chrome/browser/ui/native_web_...
File chrome/browser/ui/native_web_contents_modal_dialog_manager.h (right):

https://codereview.chromium.org/12281012/diff/1/chrome/browser/ui/native_web_...
chrome/browser/ui/native_web_contents_modal_dialog_manager.h:17:
~NativeWebContentsModalDialogManagerDelegate() {}
On 2013/02/16 00:53:16, Ben Goodger (Google) wrote:
> virtual dtor

Done.

[commit-bot: I haz the power, 2013-02-16 01:42:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wittman@chromium.org/12281012/6031

[commit-bot: I haz the power, 2013-02-16 01:50:46]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac&number...

[commit-bot: I haz the power, 2013-02-16 03:58:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/wittman@chromium.org/12281012/6031

[commit-bot: I haz the power, 2013-02-16 06:33:56]

Change committed as 182942