The default bug view has changed. See this FAQ.

Context menu should not handle OGG audio as OGV video.

RESOLVED FIXED in seamonkey2.13

Status

SeaMonkey
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.13

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
From Bug 557278:

> When I open an audio stream (e.g http://ct1.party107.com:8000/Party107-Q0.ogg)
> and then right-click on the player, I get options like:
> 'view video', 'copy video location', etc. Those should be 'play audio', 'copy
> audio location', 'save audio', 'send audio', etc.
(Assignee)

Comment 1

5 years ago
Created attachment 631671 [details] [diff] [review]
Patch v1.0 Fixit.

$ MOZ_NO_REMOTE=1 TEST_PATH=suite/browser/test/test_contextmenu.html  pymake -C ../objdir-sm/ mochitest-plain

Passed: 1713
Failed: 0
Todo: 0

Note this test is disabled on Linux due to random ORANGEs.
Attachment #631671 - Flags: review?(neil)

Comment 2

5 years ago
Comment on attachment 631671 [details] [diff] [review]
Patch v1.0 Fixit.

>-		test_contextmenu.html \
>-		subtst_contextmenu.html \
> 		ctxmenu-image.png \
> 		video.ogg \
Aren't these test files too?

>+# test_contextmenu.html is disabled on Linux due to bug 513558
This is disabled because a) we're already hitting this randomorange b) Firefox has already disabled it c) other?

>+		audio.ogg \
>+		test_contextmenu.html \
>+		subtst_contextmenu.html \
Any particular order to these files?
(Assignee)

Comment 3

5 years ago
Created attachment 633788 [details] [diff] [review]
Patch v1.1 rearrange Makefile.in

>>-		test_contextmenu.html \
>>-		subtst_contextmenu.html \
>> 		ctxmenu-image.png \
>> 		video.ogg \
> Aren't these test files too?
Hmm. Yes. Firefox uses them for other tests. Since we haven't ported those yet, I'll move them as well.

>>+# test_contextmenu.html is disabled on Linux due to bug 513558
> This is disabled because a) we're already hitting this randomorange b) Firefox has already disabled it c) other?
(b) Plus Serge filed Bug 513558 so I'll save him some work by disabling it on Linux for us as well.

>>+		audio.ogg \
>>+		test_contextmenu.html \
>>+		subtst_contextmenu.html \
> Any particular order to these files?
I think based on the original order in Firefox the main test file should go first followed by the resources used so it should be something like:

	test_contextmenu.html \
	subtst_contextmenu.html \
	ctxmenu-image.png \
	video.ogg \
	audio.ogg \
Attachment #631671 - Attachment is obsolete: true
Attachment #631671 - Flags: review?(neil)
Attachment #633788 - Flags: review?(neil)

Comment 4

5 years ago
Comment on attachment 633788 [details] [diff] [review]
Patch v1.1 rearrange Makefile.in

a) would have been the ideal reason, but b) will do ;-)
Attachment #633788 - Flags: review?(neil) → review+
(Assignee)

Comment 5

5 years ago
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/bca02d8d10bd
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.13
You need to log in before you can comment on or make changes to this bug.