Closed
Bug 45689
Opened 24 years ago
Closed 24 years ago
Realplayer does not launch as a helper app
Categories
(Core Graveyard :: Plug-ins, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: shrir, Assigned: mscott)
References
()
Details
(Whiteboard: [nsbeta2+])
Attachments
(1 file)
(deleted),
text/plain
|
Details |
Build:2000071708
This was working in buld 2000071108. If you go to the above url and select any
link, say, 'news#1', then realplayer should launch and play the audio. But this
is not happening. The url bar just loads the file and nothing happens.
Assignee | ||
Comment 2•24 years ago
|
||
this works in debug builds. I think this is because the new helper app dialog
isn't part of the release build packages so we can't throw the dialog and we
fall out.
re-assiging to Bill.
Assignee: mscott → law
Assignee | ||
Comment 3•24 years ago
|
||
No what I said isn't true. Looking through my release builds all the dialog
files seem to be in the right place. Yet the dialog doesn't come up.
Assignee | ||
Comment 4•24 years ago
|
||
re-assigning back to me. I have a theory and we'll find out if it's correct
tomorrow. I noticed that the uriloader has an .xpt file uriloader.xpt which IS
part of the packages. However, as part of the helper app work I did, I added a
new module: exthandler. It's part of the uriloader dll; however it generates a
new xpt file: exthandler.xpt. This xpt file was not part of the packages.
I just added it. Since Bill's code is the only caller outside of the uriloader
to attempt to use an interface from the exthandler, that could explain why the
problem didn't manifest itself until now.
I'll take this bug back and we'll see if it works in tomorrows release builds.
Assignee: law → mscott
Comment 5•24 years ago
|
||
Putting on [NEED INFO] radar. PDT needs to know impact to user and risk of fix
to make a call on this bug. Is this still a bug? Let us know if this is fixed.
Whiteboard: [NEED INFO]
Assignee | ||
Comment 6•24 years ago
|
||
There's no risk the user. It's a packaging problem. We just haven't figured out
where yet. debug builds have been running just fine.
Comment 7•24 years ago
|
||
Comment 8•24 years ago
|
||
I crash when loading real audio on today's commercial release build. Stack trace
(from amazon.com) attached above.
Comment 9•24 years ago
|
||
Putting on [NEED INFO] radar. Michael, can we ship pr2 without realplayer?
Assignee | ||
Comment 10•24 years ago
|
||
It has nothing to do with real player. bringing up the helper app dialog crashes
the application in release builds.
It's a packaging problem. I'm sure of it.
Assignee | ||
Comment 11•24 years ago
|
||
Bill, you'll like this one. I think the problem is the fact that necko has a xpt
file that isn't part of the packages! mimetype.xpt. This xpt file contains the
interface information for nsIMIMEInfo.idl. This is of course one of the
interfaces you are using in your helper app dialog. It looks like you are the
first user of nsIMIMEInfo in java script.
I've been debugging a release install by adding debug statements throughout the
helper app dialog JS code and it does look like we are crashing every time we
try to access the nsIMIMEInfo object.
I think this is the problem.
PDT, you keep putting need info on this bug on me. It seems to be a pretty
serious problem. Invoking the helper app dialog causes a crash on release
builds. No if and's or buts. This dialog is invoked when the user clicks on
links that require external applications (like pdf, real audio, mp3, etc). I
don't see how we could ship a beta that did that. It's very common.
I'd like to check this packaging fix soon and not have to wait for another bug
review cycle tomorrow. I'll try to ping pdt directly.
Assignee | ||
Comment 12•24 years ago
|
||
per discussion with bug triage team (jar and dveditz), I explained that this bug
isn't real player, it's a problem with all helper apps. They agree to make it a
plus. I'm making that change with their permission
Whiteboard: [NEED INFO] → [nsbeta2+]
Assignee | ||
Comment 13•24 years ago
|
||
I checked in the packaging change for this. So we should be good to go in
tomorrow's release builds.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 14•24 years ago
|
||
looks like that packaging problem was the culrpit. I'm getting the dialog on my
windows release build just fine now!
Reporter | ||
Comment 15•24 years ago
|
||
yeah..verified that real layer launches as a helper app on today's windows
(2000072108)
Status: RESOLVED → VERIFIED
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•