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)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: shrir, Assigned: mscott)

References

()

Details

(Whiteboard: [nsbeta2+])

Attachments

(1 file)

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.
nominating: nsbeta2
Keywords: nsbeta2
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
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.
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
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]
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.
Attached file stack trace (deleted) —
I crash when loading real audio on today's commercial release build. Stack trace (from amazon.com) attached above.
Putting on [NEED INFO] radar. Michael, can we ship pr2 without realplayer?
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.
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.
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+]
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
looks like that packaging problem was the culrpit. I'm getting the dialog on my windows release build just fine now!
yeah..verified that real layer launches as a helper app on today's windows (2000072108)
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: