<br><br><div class="gmail_quote">On Sun, Jan 15, 2012 at 9:52 PM, Adam Barth <span dir="ltr"><<a href="mailto:abarth@webkit.org">abarth@webkit.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

We've previously discussed this topic in Bug 67882 and Bug 75905.<br>
<br>
We should remove deprecatedFrameEncoding.  Removing the code has the<br>
following benefits:<br>
<br>
1) Standards compliance.  There was a discussion in the HTTP working<br>
group about whether the requesting context should be a factor in<br>
determining the character set used to decode the Content-Disposition<br>
header.  The working group decided that we shouldn't use that<br>
information for several reasons:<br>
<br>
  a) Varying the interpretation of an HTTP response based on the<br>
context of the request is contrary to the stateless nature of HTTP.<br>
The fact that HTTP request/response pairs can be understood<br>
irrespective of context is core to the design of HTTP.<br>
<br>
  b) Most user agents, including most browsers, do not have this<br>
behavior.  That means the compatibility risk for dropping the behavior<br>
in other user agents (e.g., Safari) is relatively low, especially for<br>
a feature likes this one that is not widely used on the mobile web.<br>
<br>
2) Stability.  This code crashes.  For example, in the most recent<br>
release of Chrome Canary on Windows, deprecatedFrameEncoding accounts<br>
for 3.5% of all renderer crashes.  While we could invest effort in<br>
fixing these crashes, we'd be better off removing this code and<br>
spending that effort improving stability elsewhere.<br></blockquote><div><br></div><div>For the sake of completeness: The crashes I've seen end in</div><div><br></div><div>[DocumentWriter.cpp:246]<span class="Apple-tab-span" style="white-space:pre">       </span>WebCore::DocumentWriter::deprecatedFrameEncoding()</div>

<div>[FrameLoader.cpp:2591]<span class="Apple-tab-span" style="white-space:pre">        </span>WebCore::FrameLoader::addExtraFieldsToRequest()</div><div>[FrameLoader.cpp:1191]<span class="Apple-tab-span" style="white-space:pre">        </span>WebCore::FrameLoader::loadURL()</div>

<div><br></div><div>implying that the FrameLoader doesn't have an active document loader at that point.</div><div><br></div><div>-jochen</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


If removing deprecatedFrameEncoding isn't possible at this time, we<br>
should revert <a href="http://trac.webkit.org/changeset/104723" target="_blank">http://trac.webkit.org/changeset/104723</a> because it<br>
causes crashes.  Rather than get into a revert war, however, I believe<br>
the project would be better served by talking out this issue.<br>
<br>
Adam<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
</blockquote></div><br>