Remove chrome::NOTIFICATION_WEB_DIALOG_SHOWN.

Remove chrome::NOTIFICATION_WEB_DIALOG_SHOWN.

This is a preliminary work for moving web dialog stuffs to ui/web_dialogs.

BUG=125841
TEST=rowser_tests --gtest_filter="CertificateViewerUITest*.*" passes


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

[mazda, 2012-05-17 23:56:42]

Could you review this change?

This is a preliminary work for moving web dialog code to ui/web_dialogs.

[Ben Goodger (Google), 2012-05-18 15:52:13]

Cool! LGTM. Thanks for doing this.

[flackr, 2012-05-18 16:37:22]

Cool, LGTM

[flackr, 2012-05-18 16:44:19]

http://codereview.chromium.org/10332231/diff/7001/chrome/browser/ui/webui/web...
File chrome/browser/ui/webui/web_dialog_delegate.h (right):

http://codereview.chromium.org/10332231/diff/7001/chrome/browser/ui/webui/web...
chrome/browser/ui/webui/web_dialog_delegate.h:72: content::RenderViewHost*
render_view_host) {}
It might make sense to have the notifications delivered in this (or another)
base class so that we don't have to repeat the notification code for any new
WebUI dialogs we want to test.

[mazda, 2012-05-18 17:23:20]

On 2012/05/18 16:44:19, flackr wrote:
>
http://codereview.chromium.org/10332231/diff/7001/chrome/browser/ui/webui/web...
> File chrome/browser/ui/webui/web_dialog_delegate.h (right):
> 
>
http://codereview.chromium.org/10332231/diff/7001/chrome/browser/ui/webui/web...
> chrome/browser/ui/webui/web_dialog_delegate.h:72: content::RenderViewHost*
> render_view_host) {}
> It might make sense to have the notifications delivered in this (or another)
> base class so that we don't have to repeat the notification code for any new
> WebUI dialogs we want to test.

Thanks. That's a good idea. Let me do it in the next CL.

[commit-bot: I haz the power, 2012-05-18 17:38:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mazda@chromium.org/10332231/13002

[commit-bot: I haz the power, 2012-05-18 18:08:13]

Try job failure for 10332231-13002 on win for step "update".
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

Step "update" is always a major failure.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-05-18 20:28:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mazda@chromium.org/10332231/13002

[commit-bot: I haz the power, 2012-05-19 00:04:07]

Change committed as 137979