diff options
| author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-30 17:58:11 +0000 |
|---|---|---|
| committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-30 17:58:11 +0000 |
| commit | 8ac68283bbf58ad85ab929148a33aad8c46230fb (patch) | |
| tree | 08cb146213876cd5ab1a2978a35749c2d0c9f411 /webkit | |
| parent | 910b68dd1de97e43358284a751c1fdc7abd58171 (diff) | |
| download | chromium_src-8ac68283bbf58ad85ab929148a33aad8c46230fb.zip chromium_src-8ac68283bbf58ad85ab929148a33aad8c46230fb.tar.gz chromium_src-8ac68283bbf58ad85ab929148a33aad8c46230fb.tar.bz2 | |
Baselining 2 tests:
onload-navigation-iframe.html
onload-navigation-iframe-timeout.html
The tests submit a form to about:blank.
They fail because there is no ? at the end of about:blank when we land to the new page.
This is because the GURL implementation treats about:blank as a path URL and sets no query for it.
Note: the test diff files are in my home dir under the layoutest_onload dir.
Review URL: http://codereview.chromium.org/19492
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8967 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit')
4 files changed, 31 insertions, 5 deletions
diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/README b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/README index ab07d6f..ebdd532 100644 --- a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/README +++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/README @@ -1,3 +1,11 @@ error404-goback.html ------------------------------------- Only error messages are different. + + +onload-navigation-iframe.html +onload-navigation-iframe-timeout.html +------------------------------------- +We are missing the ? at the end of the about:blank URL, +which is expected per our GURL implementation. +(we ignore query params for path URL and about:blank is one).
\ No newline at end of file diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/onload-navigation-iframe-expected.txt b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/onload-navigation-iframe-expected.txt new file mode 100644 index 0000000..8504282 --- /dev/null +++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/onload-navigation-iframe-expected.txt @@ -0,0 +1,7 @@ +In this case, Safari and Firefox don't add a history item. + + +============== Back Forward List ============== +curr-> http://127.0.0.1:8000/navigation/onload-navigation-iframe.html **nav target** + about:blank (in frame "myFrame") +=============================================== diff --git a/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/onload-navigation-iframe-timeout-expected.txt b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/onload-navigation-iframe-timeout-expected.txt new file mode 100644 index 0000000..6c16143 --- /dev/null +++ b/webkit/data/layout_tests/platform/chromium-win/LayoutTests/http/tests/navigation/onload-navigation-iframe-timeout-expected.txt @@ -0,0 +1,9 @@ +In this case, Safari and Firefox adds a history item but IE doesn't. + + +============== Back Forward List ============== + http://127.0.0.1:8000/navigation/onload-navigation-iframe-timeout.html **nav target** + about:blank (in frame "myFrame") +curr-> http://127.0.0.1:8000/navigation/onload-navigation-iframe-timeout.html + about:blank (in frame "myFrame") **nav target** +=============================================== diff --git a/webkit/tools/layout_tests/test_lists/tests_fixable.txt b/webkit/tools/layout_tests/test_lists/tests_fixable.txt index 45cbb44..3591f88 100644 --- a/webkit/tools/layout_tests/test_lists/tests_fixable.txt +++ b/webkit/tools/layout_tests/test_lists/tests_fixable.txt @@ -103,10 +103,12 @@ DEFER RELEASE : LayoutTests/fast/events/standalone-image-drag-to-editable.html = // and it should replace the first "?" substring. This makes absolutely no sense. DEFER : LayoutTests/fast/events/stopPropagation-submit.html = FAIL -// Expected results has a terminal "?", since "set query" on about:blank is allowed. +// Expected results has a terminal "?", since "set query" on about:blank is allowed, +// but about:blank is considered a path URL by the GURL implementation and no query is +// set for path URLs. // This is strange since query should have no meaning in non-standard urls -DEFER LINUX WIN : LayoutTests/http/tests/navigation/onload-navigation-iframe-timeout.html = FAIL -DEFER LINUX WIN : LayoutTests/http/tests/navigation/onload-navigation-iframe.html = FAIL +DEFER LINUX : LayoutTests/http/tests/navigation/onload-navigation-iframe-timeout.html = FAIL +DEFER LINUX : LayoutTests/http/tests/navigation/onload-navigation-iframe.html = FAIL // ----------------------------------------------------------------- // PENDING TESTS (forked to pending/, need to be sent upstream) @@ -1580,7 +1582,7 @@ LINUX : LayoutTests/fast/css/beforeSelectorOnCodeElement.html = CRASH PASS MAC : LayoutTests/animations/keyframes-rule.html = FAIL // The V8 regular expression engine does not match JSCRE on this one. -// See V8 bug 182. +// See V8 bug 182. LayoutTests/fast/js/regexp-no-extensions.html = FAIL // This test appears to pass on Linux, despite it referencing feed:// URLs which @@ -1663,7 +1665,7 @@ LINUX : LayoutTests/svg/custom/stroke-fallback.svg = FAIL // Merge 39830:39880 - new tests LayoutTests/plugins/npruntime.html = FAIL TIMEOUT -// Merge 39830:39880 - regressions +// Merge 39830:39880 - regressions WIN : LayoutTests/editing/selection/5057506.html = FAIL LINUX WIN : LayoutTests/fast/events/init-events.html = FAIL |