<div class="gmail_quote">On Thu, Jan 12, 2012 at 7:24 PM, Kenneth Rohde Christiansen <span dir="ltr"><<a href="mailto:kenneth.christiansen@gmail.com">kenneth.christiansen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hey Jarred, Are you going to upstream these patches then?<br>
<br>
Kenneth<br></blockquote><div><br></div><div>I'd be happy to if Leo doesn't have the time, giving him author credit of course.  It's not obvious to me if there are any layout tests for remote inspector support, and I certainly don't see any relevant skipped tests in Qt.  Anyone familiar?</div>

<div><br></div><div>Thanks,</div><div>Jarred</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5"><br>
On Thu, Jan 12, 2012 at 10:05 PM, Jarred Nicholls <<a href="mailto:jarred@webkit.org">jarred@webkit.org</a>> wrote:<br>
> On Wed, Jan 11, 2012 at 1:05 PM, Jarred Nicholls <<a href="mailto:jarred@webkit.org">jarred@webkit.org</a>> wrote:<br>
>><br>
>> On Tue, Jan 10, 2012 at 11:08 PM, Leo Franchi <<a href="mailto:lfranchi@kde.org">lfranchi@kde.org</a>> wrote:<br>
>>><br>
>>> On Jan 10, 2012, at 2:45 PM, Jarred Nicholls wrote:<br>
>>><br>
>>> Hey qtwebkittens,<br>
>>><br>
>>> My company has charged me with resolving a very nasty issue with QtWebKit<br>
>>> 2.2.1's web inspector interface (built from Qt 4.8.0 source), which locks up<br>
>>> when one pauses script at a breakpoint.<br>
>>><br>
>>> <a href="https://bugs.webkit.org/show_bug.cgi?id=71271" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=71271</a><br>
>>><br>
>>> CPU on the process goes to 100% and it looks like the EventLoop just runs<br>
>>> on and on (calling QCoreApplication::processEvents), but isn't processing UI<br>
>>> events in the inspector interface at all.<br>
>>><br>
>>> Before spending much more time trying to analyze this issue and scouring<br>
>>> changesets, has anyone experienced this themself and/or have any indication<br>
>>> of what the issue is or how to fix it?  This appears to have been an issue<br>
>>> in the past (<a href="https://bugs.webkit.org/show_bug.cgi?id=41142" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=41142</a>) but I have a<br>
>>> trunk build of QtWebKit @ r84622 (against Qt 4.7.4) that does not have this<br>
>>> issue.<br>
>>><br>
>>><br>
>>> Hi Jarred,<br>
>>><br>
>>> It might be the issue I fixed recently in my qtwebkit fork---I added<br>
>>> support for phantomjs remote debugging, and ran into this exact same thing.<br>
>>> Turns out in a few places setting a breakpoint caused processing of the<br>
>>> incoming events from the JS side of the debugger to fail.<br>
>>><br>
>>> I mentioned my fixes to jturcotte but didn't get much interest---if there<br>
>>> is interest I will gladly clean up my patches and get these fixes upstream.<br>
>>><br>
>>> Here's the work to see if it fixes for you:<br>
>>><br>
>>> 1) Updated Websocket implementation to latest spec to work against a<br>
>>> recent chrome/webkit browser:<br>
>>><br>
>>><br>
>>> <a href="https://gitorious.org/~lfranchi/webkit/lfranchis-qtwebkit/commit/93a72f2657c2a8bb6863d5d2ffe8ce6faedd4598" target="_blank">https://gitorious.org/~lfranchi/webkit/lfranchis-qtwebkit/commit/93a72f2657c2a8bb6863d5d2ffe8ce6faedd4598</a><br>


>>><br>
>>> 2) 2 fixes for hanging on breakpoint (one requires the above commit, but<br>
>>> is easy enough to apply against trunk if you read the code)<br>
>>><br>
>>><br>
>>> <a href="https://gitorious.org/~lfranchi/webkit/lfranchis-qtwebkit/commit/4e104bc438c21ef63a6a4227a142d36bf2401b2e" target="_blank">https://gitorious.org/~lfranchi/webkit/lfranchis-qtwebkit/commit/4e104bc438c21ef63a6a4227a142d36bf2401b2e</a><br>


>>><br>
>>> <a href="https://gitorious.org/~lfranchi/webkit/lfranchis-qtwebkit/commit/1606e202891b856ad0fdae80288ccdbf8d6765e1" target="_blank">https://gitorious.org/~lfranchi/webkit/lfranchis-qtwebkit/commit/1606e202891b856ad0fdae80288ccdbf8d6765e1</a><br>


>>><br>
>>> The latter two patches I hope should fix your issue. All 3 are also<br>
>>> available as a standalone patch against qt 4.8 here:<br>
>>><br>
>>><br>
>>> <a href="https://github.com/ariya/phantomjs/blob/master/deploy/qt48_fix_inspector.patch" target="_blank">https://github.com/ariya/phantomjs/blob/master/deploy/qt48_fix_inspector.patch</a><br>
>>><br>
>>> cheers,<br>
>>> leo<br>
>>><br>
>>> --<br>
>>> <a href="mailto:leo@kdab.com">leo@kdab.com</a>                                 KDAB (USA), LLC<br>
>>> <a href="mailto:lfranchi@kde.org">lfranchi@kde.org</a>                             The KDE Project<br>
>>><br>
>>> --<br>
>>> <a href="mailto:leo@kdab.com">leo@kdab.com</a>                                 KDAB (USA), LLC<br>
>>> <a href="mailto:lfranchi@kde.org">lfranchi@kde.org</a>                             The KDE Project<br>
>>><br>
>><br>
>> Hi Leo,<br>
>><br>
>> I tried the patch on vanilla Qt 4.8 and unfortunately it did not resolve<br>
>> the issue for me :(  I will keep digging.  Thanks!<br>
>><br>
>> Jarred<br>
><br>
><br>
> Hi Leo,<br>
><br>
> FYI, I did get remote debugging to work, it was a stupid mistake on my part.<br>
>  I narrowed down the issue of local debugging; it seems the inspector's JS<br>
> context is paused when the Frame's context is paused.  I added some<br>
> debugging to WebInspector.dispatch() in inspector.js, and it seems that<br>
> execution of the inspector itself is paused.  I resumed the code execution<br>
> manually in ScriptDebugServer via GDB and the inspector's code resumes.<br>
>  It's a little bizarre, but since I got remote debugging to work (thanks to<br>
> your patch!) this will suffice for now.<br>
><br>
> Cheers,<br>
> Jarred<br>
><br>
</div></div>> _______________________________________________<br>
> webkit-qt mailing list<br>
> <a href="mailto:webkit-qt@lists.webkit.org">webkit-qt@lists.webkit.org</a><br>
> <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-qt</a><br>
><br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
--<br>
Kenneth Rohde Christiansen<br>
Senior Engineer<br>
Nokia Mobile Phones, Browser / WebKit team<br>
Phone  <a href="tel:%2B45%204093%200598" value="+4540930598">+45 4093 0598</a> / E-mail kenneth at <a href="http://webkit.org" target="_blank">webkit.org</a><br>
<br>
<a href="http://codeposts.blogspot.com" target="_blank">http://codeposts.blogspot.com</a> ﹆﹆﹆<br>
</font></span></blockquote></div><br>