[Webkit-unassigned] [Bug 12422] [S60] Disabling Java/ECMA script on www.viamichelin.com crashes Browser

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=12422


zbujtas at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #12692|review?(yongjun.zhang at nokia.|review-
               Flag|com)                        |




------- Comment #3 from zbujtas at gmail.com  2007-01-28 20:01 PDT -------
(From update of attachment 12692)
fix looks great but you dont have to declare the proxy variable unless you
really use it.
so either 
if (!KJSProxy::proxy(window->m_part))
   {
   return;
   }
or

KJSProxy *proxy = KJSProxy::proxy(window->m_part);
if (!proxy)
   {
   return;
   }
ScriptInterpreter *interpreter = static_cast<ScriptInterpreter
*>(proxy->interpreter());


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