Last Comment Bug 590961 - [Mac] OOPP: Don't get crash reports from 32-bit plugins in 64-bit Firefox
: [Mac] OOPP: Don't get crash reports from 32-bit plugins in 64-bit Firefox
Product: Toolkit
Classification: Components
Component: Breakpad Integration (show other bugs)
: Trunk
: x86_64 Mac OS X
-- critical with 1 vote (vote)
: mozilla2.0b9
Assigned To: Ted Mielczarek [:ted.mielczarek]
: Ted Mielczarek [:ted.mielczarek]
Depends on:
  Show dependency treegraph
Reported: 2010-08-26 10:27 PDT by Marcia Knous [:marcia - use ni]
Modified: 2010-12-19 06:32 PST (History)
17 users (show)
ted: in‑testsuite?
mozillamarcia.knous: in‑litmus?
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Screenshot of crash page (163.47 KB, image/png)
2010-08-26 10:27 PDT, Marcia Knous [:marcia - use ni]
no flags Details

Description User image Marcia Knous [:marcia - use ni] 2010-08-26 10:27:30 PDT
Created attachment 469509 [details]
Screenshot of crash page

Using Flash Version: and Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b5pre) Gecko/20100826 Minefield/4.0b5pre, I crashed but I got a "No Report Available." 

1. Crash while playing a video. I might have "Popped Out" the video player before the crash.
2. Get the attached screenshot

Some stuff from the console:

8/26/10 9:51:49 AM	[0x0-0x1e01e].org.mozilla.firefox	Thu Aug 26 09:51:49 firefox-bin[6582] <Error>: kCGErrorInvalidOperation: _CGSFindSharedWindow: WID 934
8/26/10 9:51:49 AM	[0x0-0x1e01e].org.mozilla.firefox	Thu Aug 26 09:51:49 firefox-bin[6582] <Error>: kCGErrorFailure: Set a breakpoint @ CGErrorBreakpoint() to catch errors as they are logged.
8/26/10 9:51:49 AM	[0x0-0x1e01e].org.mozilla.firefox	Thu Aug 26 09:51:49 firefox-bin[6582] <Error>: kCGErrorInvalidOperation: _CGSFindSharedWindow: WID 934
8/26/10 9:56:43 AM	plugin-container[6583]	Warning once: This application, or a library it uses, is using NSQuickDrawView, which has been deprecated. Apps should cease use of QuickDraw and move to Quartz.
Comment 1 User image Ted Mielczarek [:ted.mielczarek] 2010-08-27 05:29:14 PDT
If you can find STR I can investigate this.
Comment 2 User image Johnny Stenback (:jst, 2010-09-23 15:21:42 PDT
I don't think we want to ship Firefox 4 w/o reliable crash reporting for plugins, blocking.
Comment 3 User image Marcia Knous [:marcia - use ni] 2010-09-23 16:14:09 PDT
Unfortunately I haven't seen this since the initial report. But I test OOPP every day on the Mac so I will keep a watchful eye.
Comment 4 User image Ted Mielczarek [:ted.mielczarek] 2010-09-24 04:17:00 PDT
Without STR, I think it's very hard to block on this.
Comment 5 User image d.a. 2010-11-01 06:57:39 PDT
This bug says x86, but for me it only happens on x86-64 Firefox process. I can't recall ever having a crash report for Flash since firefox-bin became a i386/x86-64 binary.


While watching my YouTube subscriptions earlier today the plugin would crash without any report if I navigated to the next video (or any new page) while the video was still playing.

Running Firefox in 32-bit mode instead of the default 64-bit gave me this crash report doing the same thing:

Also happens when viewing a playlist, like this one:


My Flash version is 10,1,85,3
Comment 6 User image Ted Mielczarek [:ted.mielczarek] 2010-11-01 07:28:40 PDT
Thanks, that's useful info. I don't know that crash reporting was tested with a 64-bit Firefox and a 32-bit plugin. I'll look into this.
Comment 7 User image d.a. 2010-11-01 07:38:43 PDT
FWIW I didn't get any crash reports with 64-bit Firefox and 64-bit plugin either while I was testing it.

I went back to the 32-bit plugin in part because the fullscreen didn't work (see bug 607023)
Comment 8 User image Ben Turner (not reading bugmail, use the needinfo flag!) 2010-11-01 07:57:58 PDT
Ted, that's my setup too (64-bit Firefox, 32-bit plugins).
Comment 9 User image juan becerra [:juanb] 2010-11-01 11:04:38 PDT
Marcia, I'll add the manual test here and in litmus later when we have a good set in the spreadsheet:

(anyone else is welcome to add tests there or comment)
Comment 10 User image Maniac Vlad Florin (:vladmaniac) 2010-11-02 02:09:44 PDT
Hy everyone, these are my proposals for Litmus tests

unfortunately it needs more investigating. 

I can't reproduce Marcia's problem since videos are not available outside the US...
Comment 11 User image juan becerra [:juanb] 2010-11-04 15:50:47 PDT
We should try to fix this in one of the betas to get an idea of these types of crashes.

While they are not the same as a browser crash, they are disruptive to the user experience, and there might be things we can do to improve that, or contact the right people at Adobe to alert them before a lot more users see this.

It's not uncommon for me to experience this problem (I can't reproduce on command).
Comment 13 User image Ted Mielczarek [:ted.mielczarek] 2010-11-30 13:51:21 PST
Okay, so after doing some tests with the test plugin, it looks like we don't actually get crash reports at all from a 32-bit plugin running with a 64-bit browser.
Comment 14 User image chris hofmann 2010-11-30 15:05:34 PST
should this be moved up to beta9 ?   betaN means we not have time to get this working right in time to fix any problems that are hiding in missed crash reports.
Comment 15 User image Ted Mielczarek [:ted.mielczarek] 2010-12-06 11:47:08 PST
I'm about 50% through this. There are a large number of places in Breakpad where the OOP minidump generation code assumes it's dumping a process of the same CPU architecture, so fixing this involves refactoring a large amount of code.
Comment 16 User image Ted Mielczarek [:ted.mielczarek] 2010-12-08 13:33:35 PST
Patch is up for review upstream:
Comment 17 User image Ted Mielczarek [:ted.mielczarek] 2010-12-15 13:56:57 PST
Landed upstream:

Will land on m-c shortly.
Comment 18 User image Ted Mielczarek [:ted.mielczarek] 2010-12-16 14:25:22 PST
I tested this in the browser and it wasn't working, so I started writing more unit tests and found that I had missed a bit. That patch is up for review upstream:
Comment 19 User image Ted Mielczarek [:ted.mielczarek] 2010-12-16 14:53:33 PST
Landed upstream:
Comment 20 User image Ted Mielczarek [:ted.mielczarek] 2010-12-16 16:16:01 PST
Pushed to m-c:
Comment 21 User image Henrik Skupin (:whimboo) [away 02/18 - 02/27] 2010-12-19 06:32:45 PST
Verified fixed based on my testcase on bug 619757.

Note You need to log in before you can comment on or make changes to this bug.