Ensure that modal dialogs from subframes can be cleaned up correctly.

Ensure that modal dialogs from subframes can be cleaned up correctly.

BUG=366510
TEST=See bug comment 7 for repro steps.
TBR=sky@chromium.org

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

[Charlie Reis, 2014-04-28 22:43:30]

Avi, can you take a look at the diff from patch set 1 (which matches
https://codereview.chromium.org/255543009/)?  Looks like we need to dispose of
the IPC reply if it's not going anywhere.

@sky for rubber stamp on chrome/ again.

[Avi (use Gerrit), 2014-04-28 23:04:21]

LGTM

[Charlie Reis, 2014-04-28 23:26:04]

Thanks.  I suppose there's no point in waiting for the linux_valgrind or
linux_asan try bots, since they don't appear to run the browser_tests in the
first place.  I'll TBR @sky since the chrome/ part is unchanged.

[Charlie Reis, 2014-04-28 23:26:14]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2014-04-28 23:27:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/creis@chromium.org/256233002/20001

[commit-bot: I haz the power, 2014-04-28 23:46:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-28 23:46:28]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_rel

[Charlie Reis, 2014-04-28 23:47:16]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2014-04-28 23:48:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/creis@chromium.org/256233002/20001

[commit-bot: I haz the power, 2014-04-29 09:38:30]

Change committed as 266806