Remove DialogDelegate::Accept() variant

Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary. Overriding Close() to no-op would accomplish the same behavior. However I also discovered a bug (linked below) which I fixed. So this patch creates a behavioral change: pressing escape or otherwise closing the window does the same thing as pressing "Wait", which I believe is desirable.

BUG=591186, 585312
Committed: https://crrev.com/1203e605ab26d27ee100afaec794d3c77beaa2a4
Cr-Commit-Position: refs/heads/master@{#378659}

[Evan Stade, 2016-03-01 22:23:31]

Description was changed from

==========
Remove DialogDelegate::Accept() variant

it's only used in one place and isn't necessary (overriding Close()
accomplishes the same purpose).

BUG=none
==========

to

==========
Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary.
Overriding Close() to no-op would accomplish the same behavior. However I also
discovered a bug (linked below) which I fixed. So this patch creates a
behavioral change: pressing escape or otherwise closing the window does the same
thing as pressing "Wait", which I believe is desirable.

BUG=591186
==========

[Evan Stade, 2016-03-01 22:23:31]

estade@chromium.org changed reviewers:
+ sky@chromium.org

[Evan Stade, 2016-03-01 22:23:49]

The CQ bit was checked by estade@chromium.org to run a CQ dry run

[Evan Stade, 2016-03-01 22:24:12]

Description was changed from

==========
Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary.
Overriding Close() to no-op would accomplish the same behavior. However I also
discovered a bug (linked below) which I fixed. So this patch creates a
behavioral change: pressing escape or otherwise closing the window does the same
thing as pressing "Wait", which I believe is desirable.

BUG=591186
==========

to

==========
Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary.
Overriding Close() to no-op would accomplish the same behavior. However I also
discovered a bug (linked below) which I fixed. So this patch creates a
behavioral change: pressing escape or otherwise closing the window does the same
thing as pressing "Wait", which I believe is desirable.

BUG=591186,585312
==========

[commit-bot: I haz the power, 2016-03-01 22:24:50]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1752623004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1752623004/1

[sky, 2016-03-02 00:25:23]

LGTM

[Evan Stade, 2016-03-02 01:00:38]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2016-03-02 01:02:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1752623004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1752623004/20001

[commit-bot: I haz the power, 2016-03-02 02:21:25]

Description was changed from

==========
Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary.
Overriding Close() to no-op would accomplish the same behavior. However I also
discovered a bug (linked below) which I fixed. So this patch creates a
behavioral change: pressing escape or otherwise closing the window does the same
thing as pressing "Wait", which I believe is desirable.

BUG=591186,585312
==========

to

==========
Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary.
Overriding Close() to no-op would accomplish the same behavior. However I also
discovered a bug (linked below) which I fixed. So this patch creates a
behavioral change: pressing escape or otherwise closing the window does the same
thing as pressing "Wait", which I believe is desirable.

BUG=591186,585312
==========

[commit-bot: I haz the power, 2016-03-02 02:21:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-02 02:22:26]

Description was changed from

==========
Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary.
Overriding Close() to no-op would accomplish the same behavior. However I also
discovered a bug (linked below) which I fixed. So this patch creates a
behavioral change: pressing escape or otherwise closing the window does the same
thing as pressing "Wait", which I believe is desirable.

BUG=591186,585312
==========

to

==========
Remove DialogDelegate::Accept() variant

The one and only place it was used (HungRendererDialogView) was unnecessary.
Overriding Close() to no-op would accomplish the same behavior. However I also
discovered a bug (linked below) which I fixed. So this patch creates a
behavioral change: pressing escape or otherwise closing the window does the same
thing as pressing "Wait", which I believe is desirable.

BUG=591186,585312

Committed: https://crrev.com/1203e605ab26d27ee100afaec794d3c77beaa2a4
Cr-Commit-Position: refs/heads/master@{#378659}
==========

[commit-bot: I haz the power, 2016-03-02 02:22:27]

Patchset 2 (id:??) landed as
https://crrev.com/1203e605ab26d27ee100afaec794d3c77beaa2a4
Cr-Commit-Position: refs/heads/master@{#378659}