554905 - Asp.net Ajax Telerik Radwindow not working

Status: VERIFIED WORKSFORME

(Core :: JavaScript Engine, defect)

Description

[christoph.jarisch]

User-Agent:       Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0; GTB6.4; .NET CLR 1.1.4322; .NET CLR 2.0.50727; .NET CLR 3.0.04506.648; .NET CLR 3.5.21022; .NET CLR 3.0.4506.2152; .NET CLR 3.5.30729; InfoPath.2)
Build Identifier: MozillaDeveloperPreview/3.7a3

Telerik 2008.3 Asp.net ajax - javascript "window.radopen" does not work (in all prev. versions of firefox it worked!) -> Error: Sys.invalidoperationexception "Der Vorgang ist aufgrund des aktuellen Zustands des Objekts ungültig"

Reproducible: Always

Actual Results:  
Ajax Request wich does nothing...

Expected Results:  
Open a Telerik Radwindow

Comment 1

[Nickolay_Ponomarev]

Thanks for your report and sorry for the late reply. Unfortunately there's not enough information here to lead to a code fix. At the very least we'd need a link to the problematic page.

I would recommend to report the problem to the ajax toolkit vendor, since they should also be interested in having their widgets work with the upcoming Firefox version.

To figure out what caused the change in Firefox behavior and whether it was an intended change or a regression it's useful to figure out a "regression window" - http://new.quality.mozilla.org/documents-home/bugs-docs/bug-triaging-guidelines/finding-regression-windows

Comment 2

[christoph.jarisch]

Please go to http://intern.inforaum.com/login.aspx !
There click on "Forgotten password?"!
There should be opened a window (iframe)! So it does in IE, Safari, Chrome and Opera! The actual Firefox version (3.6.8) also does it!

It seems that you don't allow to give this object an id (js)! Why not?

Comment 3

[Nickolay_Ponomarev]

Thanks for the link (I can reproduce the problem on the current trunk build, works in Firefox 3.6.8 on Mac). Someone will still have to figure out when exactly this regressed and/or try to create a smaller testcase.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

I can also reproduce it with Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b6pre) Gecko/20100902 Firefox/4.0b6pre

Not really sure how widespread this toolkit is under use but it's definitely a regression which could affect a lot of people.

Christoph, could you test some older beta releases of Firefox 4.0 and check when the problem has been started? Narrowing down the regression range is necessary for us to find the causing patch. Thanks.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

A quick check showed me that's an already older regression. Even alpha 1 (20100208) doesn't handle it correctly anymore.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Looking out for the window. Results should be available soon. Regressed sometime between 09090103 and 09100103.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Regression between 09091603 and 09091703

WORK: http://hg.mozilla.org/mozilla-central/rev/38754465ffde
FAIL: http://hg.mozilla.org/mozilla-central/rev/3079370d6597

Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=38754465ffde&tochange=3079370d6597

Could this be a regression from the tracemonkey landing that day?
http://hg.mozilla.org/mozilla-central/rev/5ac5a4d5563e

Comment 8

[Boris Zbarsky [:bzbarsky]]

OK, I can reproduce the issue.  What happens is that something in fcOpenPW_Win ends up throwing:

Error: Sys.InvalidOperationException: Der Vorgang ist aufgrund des aktuellen Zustands des Objekts ungültig.
Source File: http://intern.inforaum.com/Telerik.Web.UI.WebResource.axd?_TSM_HiddenField_=ScriptManager1_HiddenField&compress=1&_TSM_CombinedScripts_=%3b%3bSystem.Web.Extensions%2c+Version%3d3.5.0.0%2c+Culture%3dneutral%2c+PublicKeyToken%3d31bf3856ad364e35%3ade-AT%3ad0c4ca6e-6b5d-49b6-922d-5244924fb100%3a52817a7d%3a67c678a8%3bTelerik.Web.UI%2c+Version%3d2008.3.1105.35%2c+Culture%3dneutral%2c+PublicKeyToken%3d121fae78165ba3d4%3ade-AT%3a99dd44ed-99f9-4eed-a93e-6ebfddaf3eb2%3aa9723048%3a393f5085%3a9703c1f0%3aa3f85c94%3a819163f9%3abdca08ad%3a526d426%3a80c4ad6d%3a527acb41
Line: 6

The TM merge is definitely the most likely culprit.

Comment 9

[Boris Zbarsky [:bzbarsky]]

bisect says:

The first bad revision is:
changeset:   32658:842e6c09e35a
user:        Brendan Eich <[e-mail]>
date:        Thu Sep 03 14:41:19 2009 -0700
summary:     Join lambdas assigned or initialized as methods to the compiler-created function object if we can, with a read barrier to clone on method value extractions other than call expressions (471214, r=jorendorff).

Comment 10

[Brendan Eich [:brendan]]

Does this reproduce in the latest tm hourly? Earlier today I pushed a fix for bug 592412.

/be

Comment 11

[Boris Zbarsky [:bzbarsky]]

I have no idea about hourly, but I did try that patch locally and it doesn't seem to help.  Assuming I rebuilt all the stuff I needed to correctly, that is...

Comment 12

[David Mandelin [:dmandelin]]

WFM today.

Comment 13

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Verified with Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b8pre) Gecko/20101201 Firefox/4.0b8pre ID:20101201030318