[Webkit-unassigned] [Bug 40472] New: [Qt] In Carbon event model, NPP_SetWindow is called too often

[bugzilla-daemon at webkit.org]

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

           Summary: [Qt] In Carbon event model, NPP_SetWindow is called
                    too often
           Product: WebKit
           Version: 420+
          Platform: Macintosh Intel
        OS/Version: Mac OS X 10.6
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P3
         Component: New Bugs
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: qt-info at nokia.com
                CC: vestbo at webkit.org


In the Carbon event model, NPP_SetWindow is called for every redraw. It should be called only when the window geometry has changed. 

There is an easy workaround: Check for equality of the NPP_SetWindow parameters and the parameters from the previous call before recomputing anything.

In PluginViewMac.cpp, there is a call to setNPWindowIfNeeded() inside the paint(...) function. I believe this may be the culprit. 

Basically, I'm seeing a call to NPP_SetWindow (with the same parameters each time) before each update event, which is completely unnecessary and different from every other browser I'm supporting so far.

(Bug reported against Qt version 4.6.2)

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