<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/104890/">http://git.reviewboard.kde.org/r/104890/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 9th, 2012, 7:37 a.m., <b>David Faure</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This makes no sense to me. Popping up a context menu has nothing to do with keyboard focus, and the actual cause for the bug hasn't been identified, apparently. So this is a "blind workaround", which I cannot approve.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Well this is most definitely not about a keyboard focus issue. Rather it is an attempt to find a way to make QWebView the parent of the popup menu instead of the KPart's widget. The KPart's widget in most cases (at least in khtml and kwebkitpart), uses the actual content viewing widget its focus proxy. That is why this patch attempts to use the focus proxy widget when available.
However, you are correct. This is a workaround for a yet to be identified issue with the entire QWebView flickering and losing the flash plugin display when you attempt to popup a context menu using the KPart's widget as the parent of the popup menu. That problem does not occur if the QWebView is used as the parent widget of the context menu. No clue why this happens though. What is even more bizarre is if you simply reload the page and right click again, you will not see the flickering again. It only happens the first time the context menu is shown a page embedding a flash plugin.</pre>
<br />
<p>- Dawit</p>
<br />
<p>On May 9th, 2012, 7:13 a.m., Dawit Alemayehu wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Base Apps and David Faure.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated May 9, 2012, 7:13 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The following patch changes the parent widget used when creating a context menu in Konqueror. This is done to prevent unnecessary redraw that seems to occur when clicking on a kwebkitpart component that is currently displaying a flash movie. See bug#298744 for further details.
What makes the problem even more confounding is the fact that if the page playing the flash movie is reloaded, either before or after clicking the RMB, then clicking the RMB button to display the context menu afterwards works just fine. It is just the first click that causes the bug.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Made sure the bug reported in bug #298744 is gone after testing the patch.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=298744">298744</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>konqueror/src/konqmainwindow.cpp <span style="color: grey">(ea1678b)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104890/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>