Summary: | Make PlatformWindow return something else than PlatformWidget | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kenneth Rohde Christiansen <kenneth> | ||||
Component: | Platform | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Enhancement | CC: | aroben, dglazkov, eric, fishd, gustavo, hausmann, hyatt, kevin.rogovin, laszlo.gombos, tonikitoo, ukai, vestbo, yael | ||||
Priority: | P2 | Keywords: | Qt | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Kenneth Rohde Christiansen
2009-09-09 07:17:38 PDT
You should talk to Hyatt about this change. He added the PlatformWindow stuff, I believe. Seems fine. Created attachment 39712 [details]
Make PlatformWindow return something else than PlatformWidget
Comment on attachment 39712 [details]
Make PlatformWindow return something else than PlatformWidget
r=me
Eric, does this require any changes in Chromium? How can we synchronize the landing with the changes needed in Chromium? Adding Darin (fishd). This requires a small renaming change in the Chromium repository. Fortunately, we always return NULL for the platformWidget. So, the fixup on our end should just be about renaming the ChromeClient method. (In reply to comment #7) > This requires a small renaming change in the Chromium repository. Fortunately, > we always return NULL for the platformWidget. So, the fixup on our end should > just be about renaming the ChromeClient method. That said, is it OK with you that I commit this right away? We'll be able to cope. Thanks! Thansk! Landed in 48511 |