Bug 33591 – [Qt/Win] Flash in QGraphicsWebView does not process hover correctly.

• Component: Plug-ins
• Status: RESOLVED FIXED

[Girish Ramakrishnan]

Mouse hover does not work as expected with the flash in some sites.
        - http://www.bbc.co.uk/ Hover over the map
        - http://www.barbie.com/ Hover over the menu items (Games, Videos)

[Girish Ramakrishnan]

Created attachment 46451 [details]
Fix mouse hover for flash in QGraphicsWebView

[Girish Ramakrishnan]

Comment on attachment 46451 [details]
Fix mouse hover for flash in QGraphicsWebView

> commit 9b2485eb48ef403f6609334fc4d00b0aec901e8a
> Author: Girish Ramakrishnan <[e-mail]>
> Date:   Wed Jan 13 19:25:11 2010 +0530
> 
>     [Qt/Win] Flash in QGraphicsWebView does not process hover correctly.
>     
>     https://bugs.webkit.org/show_bug.cgi?id=33591
>     
>     Mouse hover does not work as expected with the flash in some sites.
>         - http://www.bbc.co.uk/ Hover over the map
>         - http://www.barbie.com/ Hover over the menu items (Games, Videos)
>     The problem appears to be that Flash queries NPNVnetscapeWindow on every
>     mouse hover. I do not how flash uses this value but returning 0 when flash
>     is in windowless mode solves the problem (When using QGraphicsWebView we
>     inject wmode opaque, thereby putting the plugin in windowless mode).
> 
> diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
> index 1968af0..34cf27a 100644
> --- a/WebCore/ChangeLog
> +++ b/WebCore/ChangeLog
> @@ -1,3 +1,22 @@
> [phone]  Girish Ramakrishnan  <[e-mail]>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        [Qt/Win] Flash in QGraphicsWebView does not process hover correctly.
> +        
> +        https://bugs.webkit.org/show_bug.cgi?id=33591
> +        
> +        Mouse hover does not work as expected with the flash in some sites.
> +        - http://www.bbc.co.uk/ Hover over the map
> +        - http://www.barbie.com/ Hover over the menu items (Games, Videos)
> +        The problem appears to be that Flash queries NPNVnetscapeWindow on every
> +        mouse hover. I do not how flash uses this value but returning 0 when flash
> +        is in windowless mode solves the problem (When using QGraphicsWebView we
> +        inject wmode opaque, thereby putting the plugin in windowless mode).
> +
> +        * plugins/win/PluginViewWin.cpp:
> +        (windowHandleForPageClient):
> +
>  2009-12-29  Kent Tamura  <[e-mail]>
>  
>          Reviewed by Maciej Stachowiak.
> diff --git a/WebCore/plugins/win/PluginViewWin.cpp b/WebCore/plugins/win/PluginViewWin.cpp
> index 763536c..336681e 100644
> --- a/WebCore/plugins/win/PluginViewWin.cpp
> +++ b/WebCore/plugins/win/PluginViewWin.cpp
> @@ -94,7 +94,9 @@ static inline HWND windowHandleForPageClient(PlatformPageClient client)
>  #if PLATFORM(QT)
>      if (!client)
>          return 0;
> -    return client->ownerWidget()->winId();
> +    if (QWidget *pluginParent = qobject_cast<QWidget*>(client->pluginParent()))
> +        return pluginParent->winId();
> +    return 0;

Reviewing my own patch, there is a coding style violation. Other than that, does this look ok?

[Girish Ramakrishnan]

Created attachment 46640 [details]
Code style fixes

[Simon Hausmann]

Comment on attachment 46640 [details]
Code style fixes

r=me

[Girish Ramakrishnan]

Landed in r53327

[Simon Hausmann]

Cherry-picked into qtwebkit-4.6 with commit 24c008da2c13b72c441a2a47a3b17627c4fc4a9e