Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(243)

Issue 334037: Update some test failures. (Closed)

Created:
11 years, 1 month ago by tony
Modified:
9 years, 7 months ago
Reviewers:
ojan
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Update some test failures. TBR=ojan Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=30144

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -3 lines) Patch
M webkit/tools/layout_tests/test_expectations.txt View 2 chunks +5 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tony
11 years, 1 month ago (2009-10-27 01:18:28 UTC) #1
ojan
11 years, 1 month ago (2009-10-27 05:14:08 UTC) #2
LGTM

On Mon, Oct 26, 2009 at 6:18 PM, <tony@chromium.org> wrote:

> Reviewers: ojan,
>
> Description:
> Update some test failures.
>
> TBR=ojan
>
> Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=30144
>
> Please review this at http://codereview.chromium.org/334037
>
> Affected files:
>  M webkit/tools/layout_tests/test_expectations.txt
>
>
> Index: webkit/tools/layout_tests/test_expectations.txt
> diff --git a/webkit/tools/layout_tests/test_expectations.txt
> b/webkit/tools/layout_tests/test_expectations.txt
> index
>
07716bba2af7df6e7b9f63f4b311685322d3680a..8aff20cc39c877cd10ac4a233c77714ba9152dfa
> 100644
> --- a/webkit/tools/layout_tests/test_expectations.txt
> +++ b/webkit/tools/layout_tests/test_expectations.txt
> @@ -3206,9 +3206,10 @@ BUG_JOHNNYG LINUX DEBUG :
> LayoutTests/svg/custom/marker-overflow-clip.svg = PASS
>
>  // WebKit roll 49961:49992
>  // Added in http://trac.webkit.org/changeset/49990
> -BUG_TONY :
>
LayoutTests/fast/dom/Document/CaretRangeFromPoint/caretRangeFromPoint-in-zoom-and-scroll.html
> = FAIL
> -// Missing results
> +BUG25884 WIN LINUX :
>
LayoutTests/fast/dom/Document/CaretRangeFromPoint/caretRangeFromPoint-in-zoom-and-scroll.html
> = FAIL
> +// Needs to show image placeholder for http://
>  BUG_TONY : LayoutTests/editing/execCommand/indent-paragraphs.html = FAIL
> +// Missing results.
>  BUG_TONY WIN LINUX SKIP : LayoutTests/fast/dom/defaultView.html = FAIL
>  // Unknown
>  BUG_TONY : LayoutTests/fast/dom/delete-contents.html = FAIL
> @@ -3249,4 +3250,5 @@ BUG_TONY MAC :
> LayoutTests/media/video-error-abort.html = FAIL
>  // jorlow to add expected results.
>  BUG_JORLOW : LayoutTests/storage/domstorage/documentURI.html = FAIL
>  // Test added by Ryosuke: http://trac.webkit.org/changeset/50090
> -BUG_TONY : LayoutTests/editing/style/remove-nested-inline-styles.html =
> FAIL
> +BUG25880 WIN LINUX :
> LayoutTests/editing/style/remove-nested-inline-styles.html = FAIL
> +BUG25889 MAC DEBUG :
> LayoutTests/fast/forms/select-empty-option-height.html = TIMEOUT
>
>
>

Powered by Google App Engine
This is Rietveld 408576698