Closed Bug 616946 Opened 14 years ago Closed 13 years ago

jQuery does not work with JetPack

Categories

(Add-on SDK Graveyard :: General, defect, P1)

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 601295

People

(Reporter: erlingwl, Assigned: irakli)

References

()

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_5; en-US) AppleWebKit/534.10 (KHTML, like Gecko) Chrome/8.0.552.215 Safari/534.10
Build Identifier: 

Hi,

We are experiencing the exact same issues as discussed in this thread: http://groups.google.com/group/mozilla-labs-jetpack/browse_thread/thread/d1d3548145d527f3

I.e. we cannot easily port a Chrome/Safari extension to Firefox because jQuery does not work. Getting the jQuery tests mentioned in (http://groups.google.com/group/mozilla-labs-jetpack/browse_thread/thread/d1d3548145d527f3) to work with page mod in JetPack/Firefox would be great..

Reproducible: Always
Depends on: 636147
I've updated the test case that Cid gave in the Group.

So it appears to be mainly a problem with XRayWrappers that are automatically built on top of any content node. So jquery unit test reports a lot of failure around nodes or objects that are not equals.

Some of these errors will be fixed when we are going to remove system principal from content script sandbox. (I check with window object as principal and many tests stop failing)
-> bug 636147

But then it doesn't fix all tests and I didn't manage to find what cause these failures.

Currently content script can pollute document scope (bug 601295), it may be a source of error as jQuery.noConflict use to built dual jQuery instances are failing.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Target Milestone: --- → 1.0b5
OS: Mac OS X → All
Priority: -- → P1
Hardware: x86 → All
Target Milestone: 1.0b5 → 1.0
Assignee: nobody → rFobic
I have created a jetpack package that may be used to run all jQuery tests from the official jQuery repo: https://github.com/Gozala/jetpack-test4jquery

I'm planning to use this package to verify work both on this bug and 636147.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: