| commit | 4e91ab8a35b52410b6a1fe7e61e9c8d2a6e6e4b1 | [log] [tgz] |
|---|---|---|
| author | tkent <[email protected]> | Wed Aug 05 22:36:46 2015 |
| committer | Commit bot <[email protected]> | Wed Aug 05 22:37:21 2015 |
| tree | c0d76f0585dab350ac82896b22ba84090d5fb322 | |
| parent | 02f3a3045fbc74e1b2d30ae27a8c5ac55d5ea52f [diff] |
Revert of Disable ExternalPopupMenuRemoveTest.RemoveOnChange. (patchset #1 id:1 of https://codereview.chromium.org/1271223002/ ) Reason for revert: Reverted Blink r200000. Original issue's description: > Disable ExternalPopupMenuRemoveTest.RemoveOnChange. > > The behavior was intentionally changed by Blink r200000 [1]. > > [1] https://src.chromium.org/viewvc/blink?revision=200000&view=revision > > BUG=516945 > [email protected],[email protected] > NOTRY=true > > Committed: https://crrev.com/372de78e05c538ae724563ee107e47fbd17498d4 > Cr-Commit-Position: refs/heads/master@{#341867} [email protected],[email protected] NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=516945 Review URL: https://codereview.chromium.org/1274813005 Cr-Commit-Position: refs/heads/master@{#341991}
diff --git a/content/renderer/external_popup_menu_browsertest.cc b/content/renderer/external_popup_menu_browsertest.cc index 675d915..885070e 100644 --- a/content/renderer/external_popup_menu_browsertest.cc +++ b/content/renderer/external_popup_menu_browsertest.cc
@@ -138,8 +138,7 @@ // Tests that nothing bad happen when the page removes the select when it // changes. (http://crbug.com/61997) -// TODO(tkent): Change event becomes asynchronous. crbug.com/516945 -TEST_F(ExternalPopupMenuRemoveTest, DISABLED_RemoveOnChange) { +TEST_F(ExternalPopupMenuRemoveTest, RemoveOnChange) { // Click the text field once to show the popup. EXPECT_TRUE(SimulateElementClick(kSelectID));