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

RESOLVED FIXED in Firefox 3.1b2

Status

()

Firefox
Menus
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Dolske, Assigned: Dolske)

Tracking

Trunk
Firefox 3.1b2
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
This menu item got bumped from bug 449522 because the backend didn't support it. Now it does, so we should add it.
(Assignee)

Comment 1

9 years ago
Created attachment 345451 [details] [diff] [review]
Patch v.1
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+
(Assignee)

Comment 3

9 years ago
Pushed changeset bf8ec3b200b9.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: Firefox 3.1 → Firefox 3.1b2

Updated

9 years ago
Blocks: 461136

Comment 4

9 years ago
Maybe the "View Video" item should not be visible when the video is playing as a document?

Updated

9 years ago
Blocks: 462714
(Assignee)

Updated

9 years ago
Depends on: 462892
(Assignee)

Comment 5

9 years ago
Yeah, filed bug 462892.
You need to log in before you can comment on or make changes to this bug.