[Webkit-unassigned] [Bug 34902] [Qt] Added methods to QWebFrame for setting and consulting the CSS2 media type

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 18 11:10:20 PST 2010


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


Simon Hausmann <hausmann at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #48656|review?, commit-queue?      |review-, commit-queue-
               Flag|                            |




--- Comment #6 from Simon Hausmann <hausmann at webkit.org>  2010-02-18 11:10:20 PST ---
(From update of attachment 48656)
Other than that I like the API. I think it should be a Q_PROPERTY.

It would be nice perhaps if the media type could be propagated down to child
frames. Theoretically the frame is the correct place, but given that child
frames can appear and disappear at will it only makes sense to set this
property on the persistent main frame, at which point to avoid disambigouties a
property on the page might be clearer.

I'm going to say r- because of the missing Q_PROPERTY and more importantly
because I feel it's not clear that this is the right place. Would love to hear
input from others on this. Mailing list? :)

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



More information about the webkit-unassigned mailing list