Rebaselining fast/css/font-face-default for Linux as upstream bug has...

Rebaselining fast/css/font-face-default for Linux as upstream bug has been fixed and it looks arial to me now.

BUG=10277
TEST=none

[kinuko (google), 2009-11-11 14:40:44]

Reviewers: gwilson, jon,

Description:
Rebaselining fast/css/font-face-default for Linux as upstream bug has been  
fixed
and it looks arial to me now.

BUG=10277
TEST=none


Please review this at http://codereview.chromium.org/384043

SVN Base: http://src.chromium.org/svn/trunk/src/

Affected files:
   M      
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/fast/css/font-face-default-font-expected.checksum
   M      
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/fast/css/font-face-default-font-expected.png
   M     webkit/tools/layout_tests/test_expectations.txt


Index: webkit/tools/layout_tests/test_expectations.txt
===================================================================
--- webkit/tools/layout_tests/test_expectations.txt	(revision 31662)
+++ webkit/tools/layout_tests/test_expectations.txt	(working copy)
@@ -867,12 +867,6 @@
  // Function arguments object is copied for each access.
  BUG24194 : LayoutTests/fast/js/kde/function_arguments.html = FAIL

-// It looks like the upstream test is incorrectly using font-family: Times
-// instead of font-family: times new roman.  This broke when we recently  
moved
-// more of our font default settings to 'times new roman' from 'Times'.   
This
-// should be fixed upstream.  https://bugs.webkit.org/show_bug.cgi?id=16348
-BUG10277 LINUX : LayoutTests/fast/css/font-face-default-font.html = FAIL
-
  // We don't support dynamic fonts on Linux.
  // Once we have the sandbox working, we should re-evaluate this.
  BUG10278 LINUX : LayoutTests/fast/css/font-face-implicit-local-font.html =  
FAIL
Index:  
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/fast/css/font-face-default-font-expected.png
===================================================================
Cannot display: file marked as a binary type.
svn:mime-type = image/png
Index:  
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/fast/css/font-face-default-font-expected.checksum
===================================================================
---  
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/fast/css/font-face-default-font-expected.checksum	

(revision 31662)
+++  
webkit/data/layout_tests/platform/chromium-linux/LayoutTests/fast/css/font-face-default-font-expected.checksum	

(working copy)
@@ -1 +1 @@
-bcd314eda0cd8a24aac088b2904d7917
\ No newline at end of file
+6c2b752b4f774fe9bfab27d3cdb51db6
\ No newline at end of file

[kinuko (google), 2009-11-11 15:33:32]

side note: I happened to pick this bug from test_expectations.txt before
noticing that it's marked as WONTFIX.  If we really don't want to
track/rebaseline this let's mark it WONTFIX in test_expectations.txt too.

[kinuko (google), 2009-11-11 15:33:58]

side note: I happened to pick this bug from test_expectations.txt before
noticing that it's marked as WONTFIX.  If we really don't want to
track/rebaseline this let's mark it WONTFIX in test_expectations.txt too.

http://codereview.chromium.org/384043

[Glenn Wilson, 2009-11-11 17:26:16]

On 2009/11/11 15:33:58, Kinuko Yasuda wrote:
> side note: I happened to pick this bug from test_expectations.txt before
> noticing that it's marked as WONTFIX.  If we really don't want to
> track/rebaseline this let's mark it WONTFIX in test_expectations.txt too.
> 
> http://codereview.chromium.org/384043

I'm likely not the best reviewer, so adding Dimitri.  Also, Jon is no longer
working on the project, so no need to include him.

[Glenn Wilson, 2009-11-11 17:26:34]

On 2009/11/11 15:33:58, Kinuko Yasuda wrote:
> side note: I happened to pick this bug from test_expectations.txt before
> noticing that it's marked as WONTFIX.  If we really don't want to
> track/rebaseline this let's mark it WONTFIX in test_expectations.txt too.

> http://codereview.chromium.org/384043

I'm likely not the best reviewer, so adding Dimitri.  Also, Jon is no longer
working on the project, so no need to include him.



http://codereview.chromium.org/384043

[dglazkov, 2009-11-11 17:36:45]

LGTM.

[dglazkov, 2009-11-11 17:37:04]

LGTM.

http://codereview.chromium.org/384043

[dglazkov, 2009-11-11 17:37:42]

Please don't use the status of bugs in the bug tracker as a guideline. Read more
here: http://dev.chromium.org/developers/testing/webkit-layout-tests/task-force

[dglazkov, 2009-11-11 17:37:59]

Please don't use the status of bugs in the bug tracker as a guideline. Read  
more
here:  
http://dev.chromium.org/developers/testing/webkit-layout-tests/task-force

http://codereview.chromium.org/384043

[tkent, 2009-11-12 05:28:32]

Committed as r31766.