Closed Bug 462294 Opened 11 years ago Closed 11 years ago

Add "View Video" to context menu for <video> elements

Categories

(Firefox :: Menus, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 3.1b2

People

(Reporter: Dolske, Assigned: Dolske)

References

Details

Attachments

(1 file)

This menu item got bumped from bug 449522 because the backend didn't support it. Now it does, so we should add it.
Attached patch Patch v.1Splinter Review
Attachment #345451 - Flags: ui-review?(jboriss)
Attachment #345451 - Flags: review?(gavin.sharp)
Attachment #345451 - Flags: review?(gavin.sharp) → review+
Comment on attachment 345451 [details] [diff] [review]
Patch v.1

Looks good
Attachment #345451 - Flags: ui-review?(jboriss) → ui-review+
Pushed changeset bf8ec3b200b9.
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: Firefox 3.1 → Firefox 3.1b2
Blocks: 461136
Maybe the "View Video" item should not be visible when the video is playing as a document?
Blocks: 462714
Depends on: 462892
Yeah, filed bug 462892.
You need to log in before you can comment on or make changes to this bug.