Bug 55232 - Chromium: pass frame history state in the context menu parameters.
Summary: Chromium: pass frame history state in the context menu parameters.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Pavel Feldman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-25 10:35 PST by Pavel Feldman
Modified: 2011-02-27 23:29 PST (History)
1 user (show)

See Also:


Attachments
[PATCH] Proposed change. (2.52 KB, patch)
2011-02-25 10:37 PST, Pavel Feldman
fishd: review+
fishd: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Feldman 2011-02-25 10:35:42 PST
Patch to follow.
Comment 1 Pavel Feldman 2011-02-25 10:37:50 PST
Created attachment 83836 [details]
[PATCH] Proposed change.
Comment 2 Darin Fisher (:fishd, Google) 2011-02-25 11:01:08 PST
Comment on attachment 83836 [details]
[PATCH] Proposed change.

View in context: https://bugs.webkit.org/attachment.cgi?id=83836&action=review

> Source/WebKit/chromium/src/ContextMenuClientImpl.cpp:246
> +      RefPtr<HistoryItem> historyItem = selectedFrame->loader()->history()->currentItem();

indentation! :-(
Comment 3 Pavel Feldman 2011-02-27 23:29:11 PST
Committed r79843: <http://trac.webkit.org/changeset/79843>