[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