Bug 104089

Summary: REGRESSION(r136615): Incorrect style sharing in view-source documents.
Product: WebKit Reporter: Takashi Sakamoto <tasak>
Component: Tools / TestsAssignee: Andreas Kling <kling>
Status: RESOLVED FIXED    
Severity: Normal CC: allan.jensen, cmarcelo, kling, koivisto, macpherson, menard, mkwst, ojan, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch koivisto: review+, kling: commit-queue-

Description Takashi Sakamoto 2012-12-04 22:18:30 PST
The following layout test is failing on Linux and Mac,

fast/frames/viewsource-attribute.html
fast/frames/viewsource-on-image-file.html

Probable cause:
This is probably caused by a patch filed on http://webkit.org/b/103925.

Looking at fast/frames/viewsource-attribute.html, the following TDs share the same style:
     TD      0x7fffea86a300 CLASS=webkit-line-number
     TD      0x7fffea86a680 CLASS=webkit-line-content

.webkit-line-number has "counter" for showing line numbers, but .webkit-line-content doesn't have "counter".

+                              RenderTableCell {TD} at (76,0) size 224x12 [color=#808080] [bgcolor=#F0F0F0] [border: none (1px solid #BBBBBB) none] [r=0 c=1 rs=1 cs=1]
+                                RenderInline (generated) at (0,0) size 5x12
+                                  RenderCounter at (138,0) size 5x12
+                                    text run at (138,0) width 5: "2"
+                                RenderInline {SPAN} at (0,0) size 15x12 

One more RenderCounter is generated.
I guess, other differences would come from the same reason, styles, which cannot be shared, are shared.
Comment 1 Andreas Kling 2012-12-05 01:49:28 PST
Created attachment 177699 [details]
Patch
Comment 2 Mike West 2012-12-05 02:47:34 PST
Skipped in r136662: <http://trac.webkit.org/changeset/136662> and r136643 <http://trac.webkit.org/changeset/136643>. Would you mind removing it from the TestExpectations files when you land the change?

Thanks!
Comment 3 Antti Koivisto 2012-12-05 11:25:39 PST
Comment on attachment 177699 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=177699&action=review

> Source/WebCore/ChangeLog:12
> +        Tests: fast/frames/viewsource-attribute.html
> +               fast/frames/viewsource-on-image-file.html

I don't see any tests.
Comment 4 Andreas Kling 2012-12-05 11:32:09 PST
Committed r136722: <http://trac.webkit.org/changeset/136722>