Closed Bug 461810 Opened 16 years ago Closed 16 years ago

Video content menu should operate on .currentSrc, not .src

Categories

(Firefox :: Menus, defect)

Product:
Firefox
Component:
Menus
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 3.1b2

People

(Reporter: Dolske, Assigned: Dolske)

References

Details

Attachments

(1 file)

Patch v.1
1.25 KB, patch
Gavin
: review+
Details | Diff | Splinter Review
Attached patch Patch v.1 — — Splinter Review 
From bug 449522 comment 40:

> Should this.target.src be this.target.currentSrc?
>
> .currentSrc is the actual URL playing. It can be different from .src if the
> video/audio has <source> elements.
Attachment #344957 - Flags: review?(gavin.sharp)
Attachment #344957 - Flags: review?(gavin.sharp) → review+
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1b2
Pushed changeset 831818cf2865.
Blocks: 461136
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: