[webkit-reviews] review requested: [Bug 52974] [Qt] enum QWebPage::WebAction doesn't have value QWebPage::CopyImageUrl : [Attachment 80756] patch2
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Feb 1 06:35:03 PST 2011
qi <qi.2.zhang at nokia.com> has asked for review:
Bug 52974: [Qt] enum QWebPage::WebAction doesn't have value
QWebPage::CopyImageUrl
https://bugs.webkit.org/show_bug.cgi?id=52974
Attachment 80756: patch2
https://bugs.webkit.org/attachment.cgi?id=80756&action=review
------- Additional Comments from qi <qi.2.zhang at nokia.com>
Based on comments to make some changes.
For the last issue "at(4)", because we already have another 3 menu items for
image, they are "Open Image, Save Image, and Copy Image", and I want to put
"Copy Image Address" after them, that' why. By the way, the other 3 actions
come from WebKit common code, currently "Copy Image Address" is only for Qt, so
we have to manually add into ContextMenu like "Open in Default Browser" did.
More information about the webkit-reviews
mailing list