Commit 155ce4a7 authored by battre@chromium.org's avatar battre@chromium.org

Revert 98398 - Disabling the ReadyPromptContentTest and the TopLevelPostReissueFromChromeFramePage

tests as these tests rely on resource dlls being present in the locales folder. 
After revision 98103 it looks like chrome uses resource packs internally.

We need to fix ChromeFrame to not use resource dlls and use the packs instead.

BUG=94362
Review URL: http://codereview.chromium.org/7740045

TBR=ananta@chromium.org
Review URL: http://codereview.chromium.org/7745050

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98424 0039d316-1c4b-4281-b951-d872f2087c98
parent 78af0dd9
...@@ -1051,9 +1051,7 @@ TEST_F(FullTabSeleniumTest, Core) { ...@@ -1051,9 +1051,7 @@ TEST_F(FullTabSeleniumTest, Core) {
// This should bring up a confirmation dialog which we hit yes on. This should // This should bring up a confirmation dialog which we hit yes on. This should
// reissue the top level post request in response to which the html content // reissue the top level post request in response to which the html content
// containing the meta tag is sent again. // containing the meta tag is sent again.
// This test has been disabled due to bug TEST_F(FullTabDownloadTest, TopLevelPostReissueFromChromeFramePage) {
// http://code.google.com/p/chromium/issues/detail?id=94362
TEST_F(FullTabDownloadTest, DISABLED_TopLevelPostReissueFromChromeFramePage) {
chrome_frame_test::MockWindowObserver post_reissue_watcher; chrome_frame_test::MockWindowObserver post_reissue_watcher;
post_reissue_watcher.WatchWindow("Confirm Form Resubmission", ""); post_reissue_watcher.WatchWindow("Confirm Form Resubmission", "");
......
...@@ -219,9 +219,7 @@ class ReadyPromptWindowButtonTest : public ReadyPromptWindowTest { ...@@ -219,9 +219,7 @@ class ReadyPromptWindowButtonTest : public ReadyPromptWindowTest {
chrome_frame_test::TimedMsgLoop message_loop_; chrome_frame_test::TimedMsgLoop message_loop_;
}; // class ReadyPromptWindowButtonTest }; // class ReadyPromptWindowButtonTest
// This test has been disabled due to bug TEST_F(ReadyPromptTest, ReadyPromptContentTest) {
// http://code.google.com/p/chromium/issues/detail?id=94362
TEST_F(ReadyPromptTest, DISABLED_ReadyPromptContentTest) {
// owned by ReadyPromptContent // owned by ReadyPromptContent
MockReadyModeState* state = new MockReadyModeState(); MockReadyModeState* state = new MockReadyModeState();
MockUrlLauncher* url_launcher = new MockUrlLauncher(); MockUrlLauncher* url_launcher = new MockUrlLauncher();
......
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