784224 - page-mod tests hang for Fennec

Status: RESOLVED FIXED

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

Comment 1

[Matteo Ferretti [:zer0] [:matteo]]

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 2

[Erik Vold [:erikvold] (please needinfo? me)]

Attachment: Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/793

Pointer to Github pull-request

Comment 3

[Erik Vold [:erikvold] (please needinfo? me)]

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

Comment 4

[Alexandre Poirot [:ochameau]]

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.

Comment 5

[Alexandre Poirot [:ochameau]]

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!

Comment 6

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/bb751291552b5b529d69f50be35564a015f36557
Bug 784224: Page-mod tests now work on Fennec

https://github.com/mozilla/addon-sdk/commit/04ca75e0d8bd6dd2d8e7f309c99b1280b86a1bf6
bug 784224: addressing review comments

https://github.com/mozilla/addon-sdk/commit/084d13d6294e775044a9a4e07bba636fe40a1aff
bug 784224: removing unnecessary use of rawTabNS

https://github.com/mozilla/addon-sdk/commit/39e72837c57b3080838d0c02c69cc0de27252574
bug 784224: making a more clear separation between tabs/utils and tabs/helpers

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

Fix Bug 784224: Page-mod tests now work on Fennec r=@ochameau

Comment 7

[[github robot]]

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