Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Silverlight fails to run on 20150730030208 Nightly

VERIFIED FIXED in Firefox 41

Status

()

Core
IPC
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: Ekanan Ketunuti, Assigned: aklotz)

Tracking

({regression})

42 Branch
mozilla43
x86
Windows 10
regression
Points:
---

Firefox Tracking Flags

(firefox41+ verified, firefox42+ verified, firefox43+ verified)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

2 years ago
Url: 
http://www.dcs-digital.com/setweb/ondemandnew.php?onid=3082&&size=b
http://demos.telerik.com/silverlight/
http://bubblemark.com/silverlight2.html

The first bad revision is 

changeset:   255050:f290fbb44af6
user:        Aaron Klotz <aklotz@mozilla.com>
date:        Tue Jul 21 01:21:51 2015 -0600
files:       ipc/glue/MessageChannel.cpp ipc/glue/MessageChannel.h ipc/glue/Neutering.h ipc/glue/WindowsMessageLoop.cpp ipc/glue/moz.build
description:
Bug 1185639 - Allow deferred message processing to happen between consecutive IPC message dispatches. r=jimm
(Reporter)

Updated

2 years ago
Keywords: regression

Comment 1

2 years ago
confirming, I can reproduce.

Comment 2

2 years ago
Dupe of https://bugzilla.mozilla.org/show_bug.cgi?id=1187005

Regression introduced by https://bugzilla.mozilla.org/show_bug.cgi?id=1165981

Comment 3

2 years ago
(In reply to David H. from comment #2)
> Dupe of https://bugzilla.mozilla.org/show_bug.cgi?id=1187005
> 
> Regression introduced by https://bugzilla.mozilla.org/show_bug.cgi?id=1165981

That's a bug specific to 64-bit edition.
(Reporter)

Comment 4

2 years ago
(In reply to David H. from comment #2)
> Dupe of https://bugzilla.mozilla.org/show_bug.cgi?id=1187005
>
> Regression introduced by https://bugzilla.mozilla.org/show_bug.cgi?id=1165981

Nothing related. This bug is NOT about "Silverlight is not recognized as installed".

Bug 1165981 landed on m-c on 2015-07-22 (Nightly 20150722030205) which is prior to Bug 1185639. Silverlight works fine on that build.
(Reporter)

Updated

2 years ago
Hardware: Unspecified → x86
Assignee: nobody → aklotz
(Reporter)

Comment 5

2 years ago
Merge day is coming up.
Flags: needinfo?(aklotz)
(Reporter)

Comment 6

2 years ago
I can't reproduce it in Nightly 8/9/2015 and I found a fix range.

Last bad: mozilla-inbound revision 3d180b8dcf42 
http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-win32/1438932581/

First good: mozilla-inbound revision ce43b991306f 
http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-win32/1438933961/

Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=3d180b8dcf42&tochange=ce43b991306f

Fixed by Bob Owen, Bug 1185529.
Assignee: aklotz → nobody
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox42: affected → fixed
Flags: needinfo?(aklotz)
Resolution: --- → FIXED
Whiteboard: [fixed by 1185529]
Target Milestone: --- → mozilla42
(Reporter)

Comment 7

2 years ago
(In reply to Ekanan Ketunuti from comment #6)
> I can't reproduce it in Nightly 8/9/2015 and I found a fix range.

Sorry, I meant Nightly 8/8/2015 (20150808030207) :(
Whiteboard: [fixed by 1185529] → [fixed by bug 1185529]

Comment 8

2 years ago
I'm still getting frequent problems with SilverLight in Nightly on Amazon where I get a white tab, sometimes with sound in the background.
I don't get the issue on any other sites using SilverLight that I've tried.

I can't reproduce this if I back out the changeset for bug 1185639.
Status: RESOLVED → REOPENED
Flags: needinfo?(aklotz)
Resolution: FIXED → ---
Looks like Silverlight is doing some fun stuff with window subclassing and the like during initialization. This is not currently compatible with window neutering which now happens sooner in IPC calls.
Flags: needinfo?(aklotz)
(Reporter)

Comment 10

2 years ago
[Tracking Requested - why for this release]: regression


(In reply to Bob Owen (:bobowen) from comment #8)
> I'm still getting frequent problems with SilverLight in Nightly on Amazon
> where I get a white tab, sometimes with sound in the background.
> I don't get the issue on any other sites using SilverLight that I've tried.
> 

I've been playing around with this for a while and seeing this too.

STR:
1. Open any link in comment 0 and activate Silverlight.
2. Switch to the other tab.
3. Wait for a while (5 or 10 seconds maybe, I can't really tell.)
4. Switch back to tab opened in step 1.

AR:
Silverlight became white.
Assignee: nobody → aklotz
Status: REOPENED → NEW
status-firefox42: fixed → affected
status-firefox43: --- → affected
tracking-firefox42: --- → ?
tracking-firefox43: --- → ?
Whiteboard: [fixed by bug 1185529]
Target Milestone: mozilla42 → ---
tracking-firefox42: ? → +
tracking-firefox43: ? → +
Created attachment 8653192 [details] [diff] [review]
Patch

The problem here was that we were neutering too aggressively in IPC Call()s. When the plugin tried to subclass the plugin window, it was inadvertently subclassing the NeuteredWindowProc instead.

Intercepting SetWindowLong* and playing around with that was too fragile, so I opted to reduce the scope of the NeuteredWindowRegion and add DeferredMessagePlaybackSuppressor to hold back deferred message playback until the end the call.

This fixes the problem with all three URLs described in the but report.
Attachment #8653192 - Flags: review?(jmathies)
Comment on attachment 8653192 [details] [diff] [review]
Patch

I'm going to update this patch, so cancelling the review request for now.
Attachment #8653192 - Flags: review?(jmathies)
Created attachment 8653593 [details] [diff] [review]
Patch (r2)
Attachment #8653192 - Attachment is obsolete: true
Attachment #8653593 - Flags: review?(jmathies)

Updated

2 years ago
Attachment #8653593 - Flags: review?(jmathies) → review+
Keywords: checkin-needed

Comment 14

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/993384a9dc2e
Keywords: checkin-needed
Comment on attachment 8653593 [details] [diff] [review]
Patch (r2)

Approval Request Comment
[Feature/regressing bug #]: bug 1185639
[User impact if declined]: Silverlight won't start
[Describe test coverage new/current, TreeHerder]: Plugin test suite, manually tested with silverlight.
[Risks and why]: Low, this patch reduces the scope of the changes in the regressing bug.
[String/UUID change made/needed]: None
Attachment #8653593 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/993384a9dc2e
Status: NEW → RESOLVED
Last Resolved: 2 years ago2 years ago
status-firefox43: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Comment on attachment 8653593 [details] [diff] [review]
Patch (r2)

Sure, taking it for 42.
Attachment #8653593 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/97c700862ca4
status-firefox42: affected → fixed
(Reporter)

Comment 19

2 years ago
verified fixed in nightly 8/31
Status: RESOLVED → VERIFIED
status-firefox43: fixed → verified
(Reporter)

Comment 20

2 years ago
verified fixed in aurora 9/01.
status-firefox42: fixed → verified
Comment on attachment 8653593 [details] [diff] [review]
Patch (r2)

Approval Request Comment
[Feature/regressing bug #]: bug 1185639
[User impact if declined]: Silverlight busted. Must be landed as a follow-up to bug 1185639.
[Describe test coverage new/current, TreeHerder]: Plugin test suite
[Risks and why]: Low, has been working well on aurora and beta
[String/UUID change made/needed]: None
Attachment #8653593 - Flags: approval-mozilla-release?

Updated

2 years ago
status-firefox41: --- → affected
tracking-firefox41: --- → +
Comment on attachment 8653593 [details] [diff] [review]
Patch (r2)

Approving for uplift to moz-release to be included in 41.0.1 build2. This patch is needed to fix the recent flash hangs (bug 1207766).
Attachment #8653593 - Flags: approval-mozilla-release? → approval-mozilla-release+
https://hg.mozilla.org/releases/mozilla-release/rev/989787953aca
status-firefox41: affected → fixed
During our Firefox 41.0.1 build 2 smoke testing we ran a number of plugin tests that included Silverlight across platforms (Windows 10 32bit, Windows 7 64bit, Mac OS X 10.10.5) and we can say that this issue is verified fixed.
status-firefox41: fixed → verified
You need to log in before you can comment on or make changes to this bug.