Status

()

Core
Plug-ins
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Matheus Kerschbaum, Assigned: Matheus Kerschbaum)

Tracking

Trunk
mozilla9
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 554322 [details] [diff] [review]
patch
Attachment #554322 - Flags: review?(joshmoz)

Updated

6 years ago
Assignee: nobody → matjk7

Updated

6 years ago
Attachment #554322 - Flags: review?(joshmoz) → review+
(Assignee)

Updated

6 years ago
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: in-testsuite-
Keywords: checkin-needed
http://hg.mozilla.org/integration/mozilla-inbound/rev/279d6114667d
Keywords: checkin-needed
Target Milestone: --- → mozilla9
http://hg.mozilla.org/mozilla-central/rev/279d6114667d
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Latest hourly with the latest merge from MI to M-C is causing flash plugin to crash, turning off OOP causes browser to crash on start up if there is flash on the tab that is loading. 

Since its an hourly I'm not sure crash-reports will be of any value: 

https://crash-stats.mozilla.com/report/index/32583d7c-a907-46a7-a498-598322110821
http://crash-stats.mozilla.com/report/index/bp-88b80f0e-a995-4047-9e7c-c0d062110821

this is the only patch I 'think' that may have affected plugins, and not even sure what this patch is doing, so maybe its not the cause. 

Latest merge from mi->m-c based on cset:
http://hg.mozilla.org/mozilla-central/rev/482742e6fff7
some 73 patches, so finding the cause of the flash plugin crash is going to be fun.  

I am using flash 10.3.183.5 32 bit 


Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0a1) Gecko/20110821 Firefox/9.0a1

Comment 5

6 years ago
(In reply to Jim Jeffery not reading bug-mail 1/2/11 from comment #4)
> this is the only patch I 'think' that may have affected plugins, and not
> even sure what this patch is doing, so maybe its not the cause. 

This patch just removed a no longer used file for xpcshell plugin tests (ie would only affect testing, not the builds), so won't have been the cause.

Perhaps the mozilla-inbound tinderbox builds can help narrow the range?
http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-win32/

Comment 6

6 years ago
Matheus, have just spotted this:
http://mxr.mozilla.org/mozilla-central/source/testing/xpcshell/xpcshell.ini#8
...which can presumably be removed now too? :-)
(Assignee)

Comment 7

6 years ago
Created attachment 554777 [details] [diff] [review]
follow-up patch

(In reply to Ed Morley [:edmorley] from comment #6)
> Matheus, have just spotted this:
> http://mxr.mozilla.org/mozilla-central/source/testing/xpcshell/xpcshell.ini#8
> ...which can presumably be removed now too? :-)

Well spotted. Can you land this patch for me?
Attachment #554777 - Flags: review+
(Assignee)

Updated

6 years ago
Attachment #554777 - Flags: review+ → review?(joshmoz)
And can we make that an error?

Updated

6 years ago
Attachment #554777 - Flags: review?(joshmoz) → review+

Comment 9

6 years ago
Follow-up:
http://hg.mozilla.org/integration/mozilla-inbound/rev/ac57320746e8
Whiteboard: [follow up patch on inbound]
Pushed:
http://hg.mozilla.org/mozilla-central/rev/ac57320746e8
Whiteboard: [follow up patch on inbound]
(In reply to Ms2ger from comment #8)
> And can we make that an error?

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