Commit e1c395fe authored by wittman@chromium.org's avatar wittman@chromium.org

Add WebContentsModalDialogManager unit tests

Implement additional tests covering most of the functionality of
WebContentsModalDialogManager. Refactor common setup into the test
fixture.

BUG=157161
R=sky@chromium.org

Review URL: https://codereview.chromium.org/23475022

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221483 0039d316-1c4b-4281-b951-d872f2087c98
parent 79b2a386
...@@ -38,7 +38,9 @@ class WebContentsModalDialogManager ...@@ -38,7 +38,9 @@ class WebContentsModalDialogManager
// WillClose() when it is being destroyed. // WillClose() when it is being destroyed.
void ShowDialog(NativeWebContentsModalDialog dialog); void ShowDialog(NativeWebContentsModalDialog dialog);
// Returns true if a dialog is currently being shown. // Returns true if any dialogs are active and not closed.
// TODO(wittman): Give this a better name, since it can return true if no
// dialogs are displayed.
bool IsShowingDialog() const; bool IsShowingDialog() const;
// Focus the topmost modal dialog. IsShowingDialog() must be true when // Focus the topmost modal dialog. IsShowingDialog() must be true when
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment