Bug 58078 – dump-as-markup conversion: editing/pasteboard/paste-text-001.html and paste-text-011.html

• Status: RESOLVED
• Resolution: FIXED
• Component: HTML Editing

[Ryosuke Niwa]

The following tests should be converted to dump-as-markup tests so that their outputs are easily understood:
editing/pasteboard/paste-text-001.html
editing/pasteboard/paste-text-011.html

[Ryosuke Niwa]

Created attachment 88700 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 88700 [details]
Patch

LGTM.

[WebKit Commit Bot]

Comment on attachment 88700 [details]
Patch

Clearing flags on attachment: 88700

Committed r83246: <http://trac.webkit.org/changeset/83246>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Kent Tamura]

paste-text-011 doesn't pass on Chromium-win and Chromium-linux.
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=editing%2Fpasteboard%2Fpaste-text-011.html&showExpectations=true&showLargeExpectations=true&group=%40ToT%20-%20chromium.org


Is it expected?

[Ryosuke Niwa]

(In reply to comment #5)
> paste-text-011 doesn't pass on Chromium-win and Chromium-linux.
> http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=editing%2Fpasteboard%2Fpaste-text-011.html&showExpectations=true&showLargeExpectations=true&group=%40ToT%20-%20chromium.org
> 
> 
> Is it expected?

No.  That seems like a bug in Chromium's DRT or font resolution since all other ports are passing the test.  But I don't really care. That's such an insignificant difference I'd just rebaseline it.

[Kent Tamura]

The DRT standard font of these platforms is 'times new roman' intentionally.  So, the test result should have the standard font name, shouldn't it?

[Ryosuke Niwa]

(In reply to comment #7)
> The DRT standard font of these platforms is 'times new roman' intentionally.  So, the test result should have the standard font name, shouldn't it?

I don't know why we' do that.  That's different from all other platforms.  They all use Times.

[Tony Chang]

(In reply to comment #8)
> (In reply to comment #7)
> > The DRT standard font of these platforms is 'times new roman' intentionally.  So, the test result should have the standard font name, shouldn't it?
> 
> I don't know why we'd do that.  That's different from all other platforms.  They all use Times.

Chromium Win and Linux try to match IE's font rendering as closely as possible.

[Ryosuke Niwa]

(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > The DRT standard font of these platforms is 'times new roman' intentionally.  So, the test result should have the standard font name, shouldn't it?
> > 
> > I don't know why we'd do that.  That's different from all other platforms.  They all use Times.
> 
> Chromium Win and Linux try to match IE's font rendering as closely as possible.

But we don't have to use that on DRT, do we?

[Tony Chang]

(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #8)
> > > (In reply to comment #7)
> > > > The DRT standard font of these platforms is 'times new roman' intentionally.  So, the test result should have the standard font name, shouldn't it?
> > > 
> > > I don't know why we'd do that.  That's different from all other platforms.  They all use Times.
> > 
> > Chromium Win and Linux try to match IE's font rendering as closely as possible.
> 
> But we don't have to use that on DRT, do we?

We don't have to, but it would be nice to test what we ship.  For example, the whitespace behavior is different when you paste on Win/Linux so we have expected results for those tests that reflect the platform behavior.

[Darin Adler]

DumpRenderTree does many things to make tests sharable across platforms.

I don’t think that “test what you ship” should override that for the vast majority of tests.

[Ryosuke Niwa]

(In reply to comment #11)
> We don't have to, but it would be nice to test what we ship.  For example, the whitespace behavior is different when you paste on Win/Linux so we have expected results for those tests that reflect the platform behavior.

Sure, I understand the whitespace behavior but is there some behavioral difference between Times and Times New Roman besides the rendering and the name?

[Tony Chang]

(In reply to comment #12)
> DumpRenderTree does many things to make tests sharable across platforms.
> 
> I don’t think that “test what you ship” should override that for the vast majority of tests.

I agree, but I don't think this difference accounts for a vast majority of tests unless you're including render tree tests.

(In reply to comment #13)
> Sure, I understand the whitespace behavior but is there some behavioral difference between Times and Times New Roman besides the rendering and the name?

The rendering difference is a behavioral difference that shows up in render trees.  I think if we switched the default font to Times in Chromium Win/Linux's DRT, we'd have to rebaseline most of the render tree tests.

It's a behavioral change in the same way that getting the exact right form button sizes as IE is a behavioral change.