[Webkit-unassigned] [Bug 29896] Runtime configurable WebSocket code is detectable when turned off

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=29896


David Levin <levin at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #40934|review?                     |review-
               Flag|                            |




--- Comment #11 from David Levin <levin at chromium.org>  2009-10-15 10:07:08 PDT ---
(From update of attachment 40934)
Summary: The right fix is to remove change done in
https://bugs.webkit.org/show_bug.cgi?id=28941 and replace it with the mechanism
from https://bugs.webkit.org/show_bug.cgi?id=30240

Details:
The change done in 28941 was to allow for runtime enabling of web sockets, but
it is fundamentally broken because it allows for the feature to be detected
even when it isn't enabled which breaks websites. This patch attempts to repair
28941 by enabling web sockets by default, but it does not address the core
issue that this approach is broken.

There is now a better way to do what is desired:
https://bugs.webkit.org/show_bug.cgi?id=30240

So instead of trying to patch code that will still be broken, it should be
ripped out and replaced with a proper solution.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.