Summary: | [Qt] [WK2] QGraphicsWKView::showContextMenu() should be public | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Benjamin Poulain <benjamin> | ||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | Keywords: | Qt, QtTriaged | ||||
Priority: | P3 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Benjamin Poulain
2011-02-27 13:47:51 PST
Created attachment 83993 [details]
Patch
Comment on attachment 83993 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=83993&action=review Very nice to see documentation for our WebKit2 classes! > Source/WebKit2/UIProcess/API/qt/qgraphicswkview.cpp:335 > + actions possible for its current position, it is empty if there is no action for the position. I don't think it should be a comma here. Maybe ; would do, or you could add another sentence instead. Committed r79832: <http://trac.webkit.org/changeset/79832> |