page-mod tests hang for Fennec

RESOLVED FIXED

Status

P1
normal
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: evold, Assigned: evold)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
No longer blocks: 777632
Priority: -- → P1
Assignee: nobody → evold
Depends on: 772663
I consider this bug for "make page-mod unit test passes on Fennec" or similar (I need a bug number to add to skipped unit test in Fennec, see bug 804187)
Comment on attachment 714294 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/793

I figure we'll need these tests for bug 834960
Attachment #714294 - Flags: review?(poirot.alex)
Comment on attachment 714294 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/793

Tested on device, works fine.

I asked your various question for clarification, would need one more cycle.
Attachment #714294 - Flags: review?(poirot.alex)
Attachment #714294 - Flags: review?(poirot.alex)
Comment on attachment 714294 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/793

Looks good, thanks for your hard work on supporting mobile,
you make the difference!
Attachment #714294 - Flags: review?(poirot.alex) → review+

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Comment 7

6 years ago
Commit pushed to stabilization at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/0afaa2bdb47a7eb68f836e72f67d48731d490bf4
Merge pull request #793 from erikvold/784224-2

Fix Bug 784224: Page-mod tests now work on Fennec r=@ochameau(cherry picked from commit 133c45791de94125e39b98aa11e3d8a9153e0153)

Conflicts:
	lib/sdk/tabs/utils.js
	test/test-page-mod.js
You need to log in before you can comment on or make changes to this bug.