diff options
| author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-22 04:34:00 +0000 |
|---|---|---|
| committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-10-22 04:34:00 +0000 |
| commit | fb6ab40ad779b56fbb45df03747540e8948f15ba (patch) | |
| tree | c9d51d64b94822c47575a487bab9ea402632cb7b /webkit/glue/bookmarklet_unittest.cc | |
| parent | 411022556a6950ebc82f88bee6c11e5334116daa (diff) | |
| download | chromium_src-fb6ab40ad779b56fbb45df03747540e8948f15ba.zip chromium_src-fb6ab40ad779b56fbb45df03747540e8948f15ba.tar.gz chromium_src-fb6ab40ad779b56fbb45df03747540e8948f15ba.tar.bz2 | |
Disable this bookmarklet test since it covers functionality that JSC doesn't support.
R=ojan
Review URL: http://codereview.chromium.org/8047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@3722 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/bookmarklet_unittest.cc')
| -rw-r--r-- | webkit/glue/bookmarklet_unittest.cc | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/webkit/glue/bookmarklet_unittest.cc b/webkit/glue/bookmarklet_unittest.cc index 8c5282b..89d1129 100644 --- a/webkit/glue/bookmarklet_unittest.cc +++ b/webkit/glue/bookmarklet_unittest.cc @@ -34,10 +34,19 @@ TEST_F(BookmarkletTest, Redirect) { } TEST_F(BookmarkletTest, NonEmptyResult) { + std::wstring text; + + // TODO(darin): This test fails in a JSC build. WebCore+JSC does not really + // need to support this usage until WebCore supports javascript: URLs that + // generate content (https://bugs.webkit.org/show_bug.cgi?id=14959). It is + // important to note that Safari does not support bookmarklets, and this is + // really an edge case. Our behavior with V8 is consistent with FF and IE. +#if 0 test_shell_->LoadURL(L"javascript:false"); MessageLoop::current()->RunAllPending(); - std::wstring text = test_shell_->GetDocumentText(); + text = test_shell_->GetDocumentText(); EXPECT_EQ(L"false", text); +#endif test_shell_->LoadURL(L"javascript:'hello world'"); MessageLoop::current()->RunAllPending(); |