<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>28350</bug_id>
          
          <creation_ts>2009-08-15 20:31:58 -0700</creation_ts>
          <short_desc>REGRESSION (r47255): MediaWiki&apos;s (including Wikipedia) navigation pane appears below the main content</short_desc>
          <delta_ts>2024-05-22 14:30:24 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Evangelism</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=195597</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, Regression</keywords>
          <priority>P1</priority>
          <bug_severity>Critical</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ryosuke Niwa">rniwa</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ayg</cc>
    
    <cc>darin</cc>
    
    <cc>dglazkov</cc>
    
    <cc>ggaren</cc>
    
    <cc>hyatt</cc>
    
    <cc>jon</cc>
    
    <cc>mitz</cc>
    
    <cc>muellerp.bugzilla</cc>
    
    <cc>pkasting</cc>
    
    <cc>simon.bugzilla</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>thakis</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>140520</commentid>
    <comment_count>0</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-15 20:31:58 -0700</bug_when>
    <thetext>Use the nightly build (27291) and go visit http://en.wikipedia.org/wiki/Main_Page
navigation, search, and all other contents on the left-pane shows up below the main content.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140521</commentid>
    <comment_count>1</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2009-08-15 20:44:25 -0700</bug_when>
    <thetext>Maybe caused by http://trac.webkit.org/changeset/47255 ?
We don&apos;t have a regression window yet.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140522</commentid>
    <comment_count>2</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2009-08-15 20:47:44 -0700</bug_when>
    <thetext>Yeah, reverting 47255 fixes it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140523</commentid>
    <comment_count>3</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-15 20:59:13 -0700</bug_when>
    <thetext>Changing the page’s DOCTYPE to &lt;!DOCTYPE html&gt; causes the layout to break even without r47255.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140526</commentid>
    <comment_count>4</comment_count>
      <attachid>34916</attachid>
    <who name="">mitz</who>
    <bug_when>2009-08-15 21:31:58 -0700</bug_when>
    <thetext>Created attachment 34916
Reduction</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140530</commentid>
    <comment_count>5</comment_count>
      <attachid>34918</attachid>
    <who name="">mitz</who>
    <bug_when>2009-08-15 21:46:34 -0700</bug_when>
    <thetext>Created attachment 34918
Extended test case

Firefox behavior is rather strange: in cases A and B, the overflow section (blue) clears the float (yellow), but in case C it does not. Note that the only difference between case B and case C is that the container has a top border.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140532</commentid>
    <comment_count>6</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-15 22:24:14 -0700</bug_when>
    <thetext>The layout difference between WebKit and Gecko is caused by &lt;http://en.wikipedia.org/skins-1.5/monobook/KHTMLFixes.css&gt;, which is loaded dynamically based on UA sniffing via JavaScript. The only rule in that style sheet is:

/* work around the horizontal scrollbars */
#column-content { margin-left: 0; }

which overrides the margin specific in main.css:

margin: 0 0 .6em -12.2em;

Using the Web Inspector to disable the &apos;margin-left: 0;&apos; rule fixes the layout and does not cause a horizontal scroll bar to appear.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140534</commentid>
    <comment_count>7</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-15 22:28:40 -0700</bug_when>
    <thetext>&lt;rdar://problem/7145895&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140535</commentid>
    <comment_count>8</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-15 22:30:03 -0700</bug_when>
    <thetext>(In reply to comment #6)
&gt; The layout difference between WebKit and Gecko is caused by
&gt; &lt;http://en.wikipedia.org/skins-1.5/monobook/KHTMLFixes.css&gt;, which is loaded
&gt; dynamically based on UA sniffing via JavaScript. The only rule in that style
&gt; sheet is:
&gt; 
&gt; /* work around the horizontal scrollbars */
&gt; #column-content { margin-left: 0; }
&gt; 
&gt; which overrides the margin specific in main.css:
&gt; 
&gt; margin: 0 0 .6em -12.2em;
&gt; 
&gt; Using the Web Inspector to disable the &apos;margin-left: 0;&apos; rule fixes the layout
&gt; and does not cause a horizontal scroll bar to appear.

Is that work around still required for the latest WebKit?  i.e. will horizontal
scrollbars appear without it?  If not, we can ask people in wikipedia to
disable that rule for the latest webkit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140536</commentid>
    <comment_count>9</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-15 22:31:15 -0700</bug_when>
    <thetext>&gt; &gt; Using the Web Inspector to disable the &apos;margin-left: 0;&apos; rule fixes the layout
&gt; &gt; and does not cause a horizontal scroll bar to appear.
&gt; 
&gt; Is that work around still required for the latest WebKit?  i.e. will horizontal
&gt; scrollbars appear without it?

No. Like I said, disabling the rule does not cause a horizontal scroll bar to appear.

&gt;  If not, we can ask people in wikipedia to
&gt; disable that rule for the latest webkit.

I have moved the bug to the Evangelism component.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140538</commentid>
    <comment_count>10</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-15 22:35:17 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; No. Like I said, disabling the rule does not cause a horizontal scroll bar to
&gt; appear.

I see.  It seems like we should contact wikipedia then.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140540</commentid>
    <comment_count>11</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-15 22:49:04 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; (In reply to comment #9)
&gt; &gt; No. Like I said, disabling the rule does not cause a horizontal scroll bar to
&gt; &gt; appear.
&gt; 
&gt; I see.  It seems like we should contact wikipedia then.

Maybe we should bring it up at http://en.wikipedia.org/wiki/Wikipedia:Help_desk</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140544</commentid>
    <comment_count>12</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-16 00:53:53 -0700</bug_when>
    <thetext>Ugh... I just noticed that this bug reproduces on other MediaWiki as well.  We can&apos;t really force every site using MediaWiki to change their script.  e.g. http://www.mediawiki.org/wiki/MediaWiki

I guess we need to preserve the old behavior or add work around for MediaWiki.  But meanwhile, we should tell MediaWiki folks to modify the script so that it won&apos;t apply that style rule for the new versions of WebKit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140545</commentid>
    <comment_count>13</comment_count>
      <attachid>34922</attachid>
    <who name="">mitz</who>
    <bug_when>2009-08-16 02:02:12 -0700</bug_when>
    <thetext>Created attachment 34922
Proposed site-specific hack

The scope of this hack can be narrowed further by adding path components before /KHTMLFixes.css or widened by not requiring an exact string match. An alternative hack would be to bypass the UA detection, but I think would be more risky.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140565</commentid>
    <comment_count>14</comment_count>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2009-08-16 09:55:28 -0700</bug_when>
    <thetext>It&apos;s curious that Firefox does something completely different for the first Reduction: blue just overlaps yellow and stays side-by-side. What the heck is the &quot;right&quot; way here?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140566</commentid>
    <comment_count>15</comment_count>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2009-08-16 09:59:16 -0700</bug_when>
    <thetext>Extended test case appears to look the same pre-r47255 and post.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140567</commentid>
    <comment_count>16</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-16 10:00:15 -0700</bug_when>
    <thetext>(In reply to comment #15)
&gt; Extended test case appears to look the same pre-r47255 and post.

Because it has &lt;!DOCTYPE html&gt;. Without it, it looks different in WebKit. Anyway, both test cases are pretty much irrelevant to this bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140569</commentid>
    <comment_count>17</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2009-08-16 10:10:47 -0700</bug_when>
    <thetext>I&apos;m confused... Wikipedia isn&apos;t in strict mode is it?  It looks like it&apos;s in &quot;almost strict&quot; mode, so isn&apos;t the strict mode issue relevant to this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140571</commentid>
    <comment_count>18</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-16 10:14:19 -0700</bug_when>
    <thetext>(In reply to comment #17)
&gt; I&apos;m confused... Wikipedia isn&apos;t in strict mode is it?  It looks like it&apos;s in
&gt; &quot;almost strict&quot; mode, so isn&apos;t the strict mode issue relevant to this?

&lt;http://trac.webkit.org/changeset/47255&gt; dropped unified float clearing behavior across modes since neither Firefox nor IE8 have the quirk WebKit used to have.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140574</commentid>
    <comment_count>19</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-16 10:33:53 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; Created an attachment (id=34918) [details]
&gt; Extended test case
&gt; 
&gt; Firefox behavior is rather strange: in cases A and B, the overflow section
&gt; (blue) clears the float (yellow), but in case C it does not. Note that the only
&gt; difference between case B and case C is that the container has a top border.

The rendering of this on WebKit and MSIE8 are identical.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140576</commentid>
    <comment_count>20</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-16 10:36:07 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; Created an attachment (id=34918) [details]
&gt; Extended test case
&gt; 
&gt; Firefox behavior is rather strange: in cases A and B, the overflow section
&gt; (blue) clears the float (yellow), but in case C it does not. Note that the only
&gt; difference between case B and case C is that the container has a top border.

On MSIE8, width of the outer box is ignored.  Yellow box appears on the right of screen, and blue box appears on the left with the same y.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140578</commentid>
    <comment_count>21</comment_count>
      <attachid>34922</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2009-08-16 10:45:42 -0700</bug_when>
    <thetext>Comment on attachment 34922
Proposed site-specific hack

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140579</commentid>
    <comment_count>22</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2009-08-16 10:50:14 -0700</bug_when>
    <thetext>I marked this as reviewed, because I think it&apos;s a great thing to do. We don&apos;t need a KHTML-specific style sheet that&apos;s working around a bug we no longer have.

Hyatt and other experts may wish to continue to debate other aspects of this, but it seems clear cut that we don&apos;t need this style rule.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140583</commentid>
    <comment_count>23</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2009-08-16 12:04:21 -0700</bug_when>
    <thetext>I&apos;m not a fan of this hack though.  What incentive does Wikipedia have to fix the issue if we work around it like this?  I am worried this hack will just stay in the code forever.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140584</commentid>
    <comment_count>24</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-16 12:11:34 -0700</bug_when>
    <thetext>(In reply to comment #23)
&gt; I&apos;m not a fan of this hack though.  What incentive does Wikipedia have to fix
&gt; the issue if we work around it like this?  I am worried this hack will just
&gt; stay in the code forever.

Yeah, that’s why I originally made this an evangelism bug. However, the problem here is that the buggy code is replicated in many websites that use the MediaWiki system, so this is more like the kind of workarounds we have for 3rd-party apps than a site-specific hack. Maybe the right thing to do is wait for Wikipedia to respond and if they do, re-evaluate the necessity of the hack.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140585</commentid>
    <comment_count>25</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2009-08-16 12:12:11 -0700</bug_when>
    <thetext>I guess I&apos;m just questioning what our site-specific-hack policy is.  Do we have to immediately put the site-specific hack into the code, or can we give evangelism some time to work first?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140586</commentid>
    <comment_count>26</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2009-08-16 12:13:49 -0700</bug_when>
    <thetext>I guess I don&apos;t mind this landing as long as we have a bug open and continue to evangelize, but you have a bigger stick to bargain with when you can actually say &quot;This is broken now. Please fix.&quot;  

If you approach Wikipedia with &quot;Well this would be broken, but we worked around it, so you can work across old and new versions of WebKit without doing anything,&quot; then they have no incentive to change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140593</commentid>
    <comment_count>27</comment_count>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2009-08-16 13:09:14 -0700</bug_when>
    <thetext>I can get WebKit evangelism folks at Google chip in with the effort as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140594</commentid>
    <comment_count>28</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-16 13:24:46 -0700</bug_when>
    <thetext>(In reply to comment #26)
&gt; I guess I don&apos;t mind this landing as long as we have a bug open and continue to
&gt; evangelize, but you have a bigger stick to bargain with when you can actually
&gt; say &quot;This is broken now. Please fix.&quot;
&gt; 
&gt; If you approach Wikipedia with &quot;Well this would be broken, but we worked around
&gt; it, so you can work across old and new versions of WebKit without doing
&gt; anything,&quot; then they have no incentive to change.

I agree.  It&apos;s probably better for us to contact MediaWiki / Wikipedia before we add this work around.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140605</commentid>
    <comment_count>29</comment_count>
    <who name="Peter Kasting">pkasting</who>
    <bug_when>2009-08-16 15:14:21 -0700</bug_when>
    <thetext>(In reply to comment #25)
&gt; I guess I&apos;m just questioning what our site-specific-hack policy is.  Do we have
&gt; to immediately put the site-specific hack into the code, or can we give
&gt; evangelism some time to work first?

When this kind of issue came up for Google Chrome, I proposed that we have a policy of not doing site-specific hacks until we had contacted the site author(s) and obtained an agreement from them to fix their site.  At that point we could check in a temporary hack if fixing things was going to take time.  I&apos;m not sure we made this our official policy but it got general agreement from those to whom I proposed it.  We used this pattern when dealing with a Hotmail problem (see http://googlechromereleases.blogspot.com/2009/01/stable-beta-update-yahoo-mail-and.html ).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140619</commentid>
    <comment_count>30</comment_count>
    <who name="Derk-Jan Hartman">hartman.wiki</who>
    <bug_when>2009-08-16 16:01:00 -0700</bug_when>
    <thetext>This KHTML specific line for mediawiki is already removed upstream, but it will take a LONG while before all the mediawiki installations are going to be updated. As noted, not even Wikipedia itself has this deployed yet at this time.

See: &quot;* Remove five-year-old KHTMLFixes.css, which is unlikely to be relevant anymore  and was causing problems.&quot;

in http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/RELEASE-NOTES?revision=55124&amp;view=markup</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140621</commentid>
    <comment_count>31</comment_count>
    <who name="Derk-Jan Hartman">hartman.wiki</who>
    <bug_when>2009-08-16 16:04:57 -0700</bug_when>
    <thetext>specific revision:
http://svn.wikimedia.org/viewvc/mediawiki?view=rev&amp;revision=53141</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140622</commentid>
    <comment_count>32</comment_count>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2009-08-16 16:14:23 -0700</bug_when>
    <thetext>(In reply to comment #30)
&gt; This KHTML specific line for mediawiki is already removed upstream, but it will
&gt; take a LONG while before all the mediawiki installations are going to be
&gt; updated. As noted, not even Wikipedia itself has this deployed yet at this
&gt; time.
&gt; 
&gt; See: &quot;* Remove five-year-old KHTMLFixes.css, which is unlikely to be relevant
&gt; anymore  and was causing problems.&quot;
&gt; 
&gt; in
&gt; http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/RELEASE-NOTES?revision=55124&amp;view=markup

This is precisely the gist of the issue. We&apos;re not dealing with one site, but a widely used software. This hack has also been duplicated verbatim on many derivative projects, such as GForge and SpringWiki, which makes evangelism effort a bit daunting.

The site-specific hack patch looks pretty good to me atm.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140623</commentid>
    <comment_count>33</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2009-08-16 16:24:42 -0700</bug_when>
    <thetext>This was due to ludicrously broken UA-sniffing on our part; sorry about that.  For those who didn&apos;t catch it, in May 2004 someone committed a workaround to MediaWiki for a KHTML bug.  Unfortunately, rather than using version-based sniffing, he just assumed all KHTML versions would have the bug forever -- KHTML was rather minor back then and it probably didn&apos;t seem important.  So we ended up serving the fix to WebKit well over five years later.

It came to light when I was trying out a switch to HTML 5, and the strict doctype caused incorrect display in WebKit.  When I figured out the problem I removed the check in r53141, which Derk-Jan linked to.  The fix will be in MediaWiki 1.16 and should go live on Wikipedia within a few weeks, but you&apos;re going to have to handle it for other MediaWiki installs for years, sadly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140718</commentid>
    <comment_count>34</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2009-08-17 08:33:36 -0700</bug_when>
    <thetext>I got hold of the appropriate people just now, and Wikipedia is no longer serving the broken CSS:

http://en.wikipedia.org/skins-1.5/monobook/KHTMLFixes.css

The fix should also be in MediaWiki 1.16.  That should theoretically be released in a month, but might take longer.

Needless to say, though, there are a *lot* of MediaWiki installs out there that don&apos;t keep up with updates.  If you guys aren&apos;t willing to keep a site-specific hack in WebKit for a while, we could *maybe* backport the fix to old versions -- normally we only backport security fixes -- but we can&apos;t make people upgrade.  The breakage is pretty ugly, with all content shoved down a large amount.

For future reference, the best place to reach us is wikitech-l: &lt;https://lists.wikimedia.org/mailman/listinfo/wikitech-l&gt;.  Anything posted there will be read by most Wikimedia sysadmins and MediaWiki developers.  For real-time response, you can also go to #mediawiki (for MediaWiki) or #wikimedia-tech (for Wikimedia site administration) on freenode, but of course you&apos;ll reach fewer people that way.

Sorry about the problems we&apos;ve caused.  Please let us know about any future difficulties.  I&apos;ve posted on wikitech-l to remind people to be careful about browser sniffing in the future:

http://lists.wikimedia.org/pipermail/wikitech-l/2009-August/044767.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140733</commentid>
    <comment_count>35</comment_count>
    <who name="Peter Kasting">pkasting</who>
    <bug_when>2009-08-17 09:48:23 -0700</bug_when>
    <thetext>(In reply to comment #34)
&gt; Needless to say, though, there are a *lot* of MediaWiki installs out there that
&gt; don&apos;t keep up with updates.

My question would be whether these sites are high-profile enough that we should check in a site-specific hack for them, and whether not checking in would make it more likely that the site authors would update sooner.

To put it differently, if we do land this hack, what are the criteria we&apos;ll use to decide when to revert it, and how will we remember to check on them?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140744</commentid>
    <comment_count>36</comment_count>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2009-08-17 10:21:19 -0700</bug_when>
    <thetext>We chatted some more on #webkit and I believe the general consensus is that we don&apos;t to land this patch. Wikipedia is now serving the hack-free theme, which should serve as the right motivator for others to catch up.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140745</commentid>
    <comment_count>37</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2009-08-17 10:22:53 -0700</bug_when>
    <thetext>The largest single user of MediaWiki other than Wikimedia is wikia.com, most likely.  Some other large wikis are listed here:

http://s23.org/wikistats/largest_html.php

The problem is more lots of little sites than a few big ones, I&apos;d think.  MediaWiki is probably the most popular wiki software out there even if you don&apos;t count Wikipedia.  For instance, my mother is an art historian and has sometimes used a wiki meant to track university job openings in 19th century art history, which runs MediaWiki.  She was using Chrome for a while and would undoubtedly summon me and ask for an explanation if it randomly broke.  There are zillions of little sites like that.

I don&apos;t what your/WebKit&apos;s attitude is on maintaining compatibility vs. cleaning out cruft, so I can&apos;t say what you should do.  Not having a fix in place for, say, the next couple of years would certainly break a lot of sites, that&apos;s all I can say.  We don&apos;t have a very good upgrade rate; there are still a remarkable number of sites running three-year-old copies of MediaWiki.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140804</commentid>
    <comment_count>38</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2009-08-17 13:11:07 -0700</bug_when>
    <thetext>(In reply to comment #36)
&gt; We chatted some more on #webkit and I believe the general consensus is that we
&gt; don&apos;t to land this patch. Wikipedia is now serving the hack-free theme, which
&gt; should serve as the right motivator for others to catch up.

We should publicize that version 1.16 or later is required.  Or otherwise people think it&apos;s WebKit/Safari/Chrome&apos;s fault and we obviously don&apos;t want that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>140809</commentid>
    <comment_count>39</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-08-17 13:21:32 -0700</bug_when>
    <thetext>Fixed in &lt;http://trac.webkit.org/projects/webkit/changeset/47383&gt;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>141677</commentid>
    <comment_count>40</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2009-08-19 18:04:39 -0700</bug_when>
    <thetext>*** Bug 28372 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>147000</commentid>
    <comment_count>41</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-09-13 15:23:47 -0700</bug_when>
    <thetext>The workaround doesn’t apply to yet older versions of MediaWiki (see &lt;http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/skins/monobook/KHTMLFixes.css?view=log&amp;pathrev=53140&gt;) because:
1) The stylesheet has only one instead of two trailing newlines and
2) The stylesheet is loaded from a &lt;link&gt; element rather than an @import rule</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>147053</commentid>
    <comment_count>42</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2009-09-14 07:46:45 -0700</bug_when>
    <thetext>Are there some sites that still have that older version of MediaWiki running? Clearly it will be straightforward to broaden the workaround to cover the other cases if there is a significant benefit to doing so and at least one site to test with.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>147065</commentid>
    <comment_count>43</comment_count>
    <who name="Aryeh Gregor">ayg</who>
    <bug_when>2009-09-14 09:03:24 -0700</bug_when>
    <thetext>Yes, it should be basically any MediaWiki site other than Wikipedia, due to the &lt;link&gt; issue.  The change from @import to &lt;link&gt; only occurred in &lt;http://www.mediawiki.org/wiki/Special:Code/MediaWiki/52886&gt;, which is after the last stable release (1.15).  So look at almost any non-Wikimedia, non-Wikia wiki that uses the default MonoBook skin, e.g., http://wikileaks.org/.  Wikileaks makes some effort to hide their version, but it looks like it&apos;s based off 1.10, so they should exhibit both problems (extra newline for &lt;= 1.11, and &lt;link&gt; for &lt;= 1.15).  I can confirm the problem is visible there.

For that matter, you could also check almost any wiki from this list that&apos;s not a Wikimedia or Wikia project:

http://s23.org/wikistats/largest_html.php?th=999&amp;lines=999</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>148031</commentid>
    <comment_count>44</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-09-18 00:55:21 -0700</bug_when>
    <thetext>(In reply to comment #42)
&gt; Are there some sites that still have that older version of MediaWiki running?
&gt; Clearly it will be straightforward to broaden the workaround to cover the other
&gt; cases if there is a significant benefit to doing so and at least one site to
&gt; test with.

&lt;http://wikileaks.org/&gt; is one such site.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>150444</commentid>
    <comment_count>45</comment_count>
    <who name="">mitz</who>
    <bug_when>2009-09-27 19:05:44 -0700</bug_when>
    <thetext>Filed bug 29792.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>615128</commentid>
    <comment_count>46</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2012-05-03 03:09:06 -0700</bug_when>
    <thetext>Unfortunately, 47 out of 200 top WikiMedia users are still using the version 1.5 or older according to http://s23.org/wikistats/largest_html.php.

It appears that we&apos;ll need to wait for another couple of years before we can remove this work around.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1504250</commentid>
    <comment_count>47</comment_count>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2019-02-08 12:41:58 -0800</bug_when>
    <thetext>Maybe try again now?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1515203</commentid>
    <comment_count>48</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2019-03-11 19:32:51 -0700</bug_when>
    <thetext>(In reply to Geoffrey Garen from comment #47)
&gt; Maybe try again now?

It looks like Chromium has removed this code in 2013:
https://github.com/chromium/chromium/commit/ecf84fc9c1a51c8ede7adfd0b0cba446d9a8caa0

Only one year after I had commented above.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1515207</commentid>
    <comment_count>49</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2019-03-11 19:38:01 -0700</bug_when>
    <thetext>Tracking the removal in https://bugs.webkit.org/show_bug.cgi?id=195597.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2037143</commentid>
    <comment_count>50</comment_count>
    <who name="Frances Cornwall">frances_c</who>
    <bug_when>2024-05-22 14:30:24 -0700</bug_when>
    <thetext>*** Bug 21434 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="0"
              isprivate="0"
          >
            <attachid>34916</attachid>
            <date>2009-08-15 21:31:58 -0700</date>
            <delta_ts>2009-08-15 22:14:52 -0700</delta_ts>
            <desc>Reduction</desc>
            <filename>28350 reduction.html</filename>
            <type>text/html</type>
            <size>311</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">PGJvZHkgc3R5bGU9IndpZHRoOiAyMDBweDsiPgogICAgPGRpdiBzdHlsZT0iCiAgICAgICAgZmxv
YXQ6IHJpZ2h0OwogICAgICAgIHdpZHRoOiAxNTBweDsKICAgICAgICBoZWlnaHQ6IDIwMHB4Owog
ICAgICAgIGJhY2tncm91bmQtY29sb3I6IHllbGxvdzsKICAgICI+CiAgICA8L2Rpdj4KICAgIDxk
aXYgc3R5bGU9IgogICAgICAgIG92ZXJmbG93OiBoaWRkZW47CiAgICAgICAgaGVpZ2h0OiAxMDBw
eDsKICAgICAgICB3aWR0aDogMTAwcHg7CiAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogYmx1ZTsK
ICAgICI+CiAgICA8L2Rpdj4KPC9ib2R5Pgo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="0"
              isprivate="0"
          >
            <attachid>34918</attachid>
            <date>2009-08-15 21:46:34 -0700</date>
            <delta_ts>2009-08-15 22:14:59 -0700</delta_ts>
            <desc>Extended test case</desc>
            <filename>28350 reduction.html</filename>
            <type>text/html</type>
            <size>773</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIEhUTUw+CjxzdHlsZT4KICAgIC5jb250YWluZXIgewogICAgICAgIHdpZHRoOiAx
NTBweDsKICAgICAgICBoZWlnaHQ6IDIyNXB4OwogICAgICAgIG1hcmdpbi1ib3R0b206IDI1cHg7
CiAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogYWxpY2VibHVlOwogICAgfQoKICAgIC5mbG9hdCB7
CiAgICAgICAgZmxvYXQ6IHJpZ2h0OwogICAgICAgIHdpZHRoOiAxMDBweDsKICAgICAgICBoZWln
aHQ6IDEwMHB4OwogICAgICAgIGJhY2tncm91bmQtY29sb3I6IHllbGxvdzsKICAgIH0KCiAgICAu
b3ZlcmZsb3cgewogICAgICAgIG92ZXJmbG93OiBoaWRkZW47CiAgICAgICAgaGVpZ2h0OiAxMDBw
eDsKICAgICAgICB3aWR0aDogMTAwcHg7CiAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogYmx1ZTsK
ICAgIH0KPC9zdHlsZT4KCkE6CjxkaXYgY2xhc3M9ImNvbnRhaW5lciI+CiAgICA8ZGl2IGNsYXNz
PSJmbG9hdCI+CiAgICA8L2Rpdj4KICAgIGZvbwogICAgPGRpdiBjbGFzcz0ib3ZlcmZsb3ciPgog
ICAgPC9kaXY+CjwvZGl2PgoKQjoKPGRpdiBjbGFzcz0iY29udGFpbmVyIiBzdHlsZT0iYm9yZGVy
LXRvcDogc29saWQ7Ij4KICAgIDxkaXYgY2xhc3M9ImZsb2F0Ij4KICAgIDwvZGl2PgogICAgPGRp
diBjbGFzcz0ib3ZlcmZsb3ciPgogICAgPC9kaXY+CjwvZGl2PgoKQzoKPGRpdiBjbGFzcz0iY29u
dGFpbmVyIj4KICAgIDxkaXYgY2xhc3M9ImZsb2F0Ij4KICAgIDwvZGl2PgogICAgPGRpdiBjbGFz
cz0ib3ZlcmZsb3ciPgogICAgPC9kaXY+CjwvZGl2Pgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>34922</attachid>
            <date>2009-08-16 02:02:12 -0700</date>
            <delta_ts>2009-08-16 10:45:42 -0700</delta_ts>
            <desc>Proposed site-specific hack</desc>
            <filename>28350_r0.diff</filename>
            <type>text/plain</type>
            <size>3103</size>
            <attacher>mitz</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0NzMzNSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMjMgQEAKKzIwMDktMDgtMTYgIERhbiBCZXJuc3RlaW4gIDxtaXR6QGFwcGxlLmNv
bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBSRUdS
RVNTSU9OIChyNDcyNTUpOiBNZWRpYVdpa2kncyAoaW5jbHVkaW5nIFdpa2lwZWRpYSkgbmF2aWdh
dGlvbiBwYW5lCisgICAgICAgIGFwcGVhcnMgYmVsb3cgdGhlIG1haW4gY29udGVudAorICAgICAg
ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjgzNTAKKworICAgICAg
ICBBIHBhcnRpY3VsYXIgdmVyc2lvbiBvZiB0aGUgTWVkaWFXaWtpIHNvZnR3YXJlIGRldGVjdHMg
V2ViS2l0IHRocm91Z2gKKyAgICAgICAgdXNlciBhZ2VudCBzbmlmZmluZyBhbmQgaW1wb3J0cyBh
IHN0eWxlIHNoZWV0IGNhbGxlZCBLSFRNTEZpeGVzLmNzcywKKyAgICAgICAgd2hpY2ggY29udGFp
bnMgYSBzaW5nbGUgcnVsZSB0aGF0IHdhcyBtZWFudCB0byB3b3JrIGFyb3VuZCBzb21lIEtIVE1M
CisgICAgICAgIGJ1ZywgYnV0IGN1cnJlbnRseSBoYXMgdGhlIHNvbGUgZWZmZWN0IG9mIGNhdXNp
bmcgdGhlIGZsb2F0IGNvbnRhaW5pbmcKKyAgICAgICAgdGhlIG1haW4gYXJ0aWNsZSBjb250ZW50
IHRvIGV4dGVuZCBhbGwgdGhlIHdheSB0byB0aGUgbGVmdCBhbmQgdGh1cyBwdXNoCisgICAgICAg
IGRvd24gdGhlIGxlZnQgbmF2aWdhdGlvbiBwYW5lLgorCisgICAgICAgICogY3NzL0NTU0ltcG9y
dFJ1bGUuY3BwOgorICAgICAgICAoV2ViQ29yZTo6Q1NTSW1wb3J0UnVsZTo6c2V0Q1NTU3R5bGVT
aGVldCk6IElmIHNpdGUgc3BlY2lmaWMgaGFja3MgYXJlCisgICAgICAgIGVuYWJsZWQsIGNoZWNr
IGlmIHRoZSBpbXBvcnRlZCBzdHlsZSBzaGVldCBpcyB0aGUgTWVkaWFXaWtpCisgICAgICAgIEtI
VE1MRml4ZXMuY3NzLiBJZiBzbywgcmVtb3ZlIHRoZSBvZmZlbmRpbmcgcnVsZS4KKwogMjAwOS0w
OC0xNSAgU2ltb24gRnJhc2VyICA8c2ltb24uZnJhc2VyQGFwcGxlLmNvbT4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBEYXZlIEh5YXR0CkluZGV4OiBXZWJDb3JlL2Nzcy9DU1NJbXBvcnRSdWxlLmNw
cAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2Nzcy9DU1NJbXBvcnRSdWxlLmNwcAkocmV2aXNpb24g
NDczMjMpCisrKyBXZWJDb3JlL2Nzcy9DU1NJbXBvcnRSdWxlLmNwcAkod29ya2luZyBjb3B5KQpA
QCAtMSw3ICsxLDcgQEAKIC8qCiAgKiAoQykgMTk5OS0yMDAzIExhcnMgS25vbGwgKGtub2xsQGtk
ZS5vcmcpCiAgKiAoQykgMjAwMi0yMDAzIERpcmsgTXVlbGxlciAobXVlbGxlckBrZGUub3JnKQot
ICogQ29weXJpZ2h0IChDKSAyMDAyLCAyMDA1LCAyMDA2LCAyMDA4IEFwcGxlIEluYy4gQWxsIHJp
Z2h0cyByZXNlcnZlZC4KKyAqIENvcHlyaWdodCAoQykgMjAwMiwgMjAwNSwgMjAwNiwgMjAwOCwg
MjAwOSBBcHBsZSBJbmMuIEFsbCByaWdodHMgcmVzZXJ2ZWQuCiAgKgogICogVGhpcyBsaWJyYXJ5
IGlzIGZyZWUgc29mdHdhcmU7IHlvdSBjYW4gcmVkaXN0cmlidXRlIGl0IGFuZC9vcgogICogbW9k
aWZ5IGl0IHVuZGVyIHRoZSB0ZXJtcyBvZiB0aGUgR05VIExpYnJhcnkgR2VuZXJhbCBQdWJsaWMK
QEAgLTI2LDYgKzI2LDggQEAKICNpbmNsdWRlICJEb2NMb2FkZXIuaCIKICNpbmNsdWRlICJEb2N1
bWVudC5oIgogI2luY2x1ZGUgIk1lZGlhTGlzdC5oIgorI2luY2x1ZGUgIlNldHRpbmdzLmgiCisj
aW5jbHVkZSA8d3RmL1N0ZExpYkV4dHJhcy5oPgogCiBuYW1lc3BhY2UgV2ViQ29yZSB7CiAKQEAg
LTYwLDcgKzYyLDIwIEBAIHZvaWQgQ1NTSW1wb3J0UnVsZTo6c2V0Q1NTU3R5bGVTaGVldChjb24K
IAogICAgIENTU1N0eWxlU2hlZXQqIHBhcmVudCA9IHBhcmVudFN0eWxlU2hlZXQoKTsKICAgICBi
b29sIHN0cmljdCA9ICFwYXJlbnQgfHwgcGFyZW50LT51c2VTdHJpY3RQYXJzaW5nKCk7Ci0gICAg
bV9zdHlsZVNoZWV0LT5wYXJzZVN0cmluZyhzaGVldC0+c2hlZXRUZXh0KHN0cmljdCksIHN0cmlj
dCk7CisgICAgU3RyaW5nIHNoZWV0VGV4dCA9IHNoZWV0LT5zaGVldFRleHQoc3RyaWN0KTsKKyAg
ICBtX3N0eWxlU2hlZXQtPnBhcnNlU3RyaW5nKHNoZWV0VGV4dCwgc3RyaWN0KTsKKworICAgIGlm
IChzdHJpY3QgJiYgcGFyZW50ICYmIHBhcmVudC0+ZG9jKCkgJiYgcGFyZW50LT5kb2MoKS0+c2V0
dGluZ3MoKSAmJiBwYXJlbnQtPmRvYygpLT5zZXR0aW5ncygpLT5uZWVkc1NpdGVTcGVjaWZpY1F1
aXJrcygpKSB7CisgICAgICAgIC8vIFdvcmsgYXJvdW5kIDxodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9MjgzNTA+LgorICAgICAgICBERUZJTkVfU1RBVElDX0xPQ0FMKGNv
bnN0IFN0cmluZywgc2xhc2hLSFRNTEZpeGVzRG90Q3NzLCAoIi9LSFRNTEZpeGVzLmNzcyIpKTsK
KyAgICAgICAgREVGSU5FX1NUQVRJQ19MT0NBTChjb25zdCBTdHJpbmcsIG1lZGlhV2lraUtIVE1M
Rml4ZXNTdHlsZVNoZWV0LCAoIi8qIEtIVE1MIGZpeCBzdHlsZXNoZWV0ICovXG4vKiB3b3JrIGFy
b3VuZCB0aGUgaG9yaXpvbnRhbCBzY3JvbGxiYXJzICovXG4jY29sdW1uLWNvbnRlbnQgeyBtYXJn
aW4tbGVmdDogMDsgfVxuXG4iKSk7CisgICAgICAgIGlmICh1cmwuZW5kc1dpdGgoc2xhc2hLSFRN
TEZpeGVzRG90Q3NzKSAmJiBzaGVldFRleHQgPT0gbWVkaWFXaWtpS0hUTUxGaXhlc1N0eWxlU2hl
ZXQpIHsKKyAgICAgICAgICAgIEFTU0VSVChtX3N0eWxlU2hlZXQtPmxlbmd0aCgpID09IDEpOwor
ICAgICAgICAgICAgRXhjZXB0aW9uQ29kZSBlYzsKKyAgICAgICAgICAgIG1fc3R5bGVTaGVldC0+
ZGVsZXRlUnVsZSgwLCBlYyk7CisgICAgICAgIH0KKyAgICB9CisKICAgICBtX2xvYWRpbmcgPSBm
YWxzZTsKIAogICAgIGlmIChwYXJlbnQpCg==
</data>
<flag name="review"
          id="19103"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>