Closed Bug 893191 Opened 11 years ago Closed 11 years ago

test-content-events.test multiple tabs, fails on Fennec

Categories

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

x86
Android
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jsantell, Assigned: jsantell)

References

Details

Attachments

(1 file)

May be a byproduct of 863502
Priority: -- → P1
Assignee: nobody → jsantell
Attached file GH PR 1137
This may be the last change for all tests passing on Fennec.. *fingers crossed*
Attachment #782762 - Flags: review?(evold)
Fixes content-events tests, but bug 899357 preventing tabs/windows from passing, so almost all tests!
Also ignores HTTP requests made by about:home on Fennec which queries possible add-ons to install
Attachment #782762 - Flags: review?(evold) → review+
(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #3)
> Also ignores HTTP requests made by about:home on Fennec which queries
> possible add-ons to install

We should make another bug and find a way to disable/avoid this.
New bug for disabling AMO requests during testing: bug 899747
Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/0a390cb20c14c123836a49b3ffff2287b014b4e8
Bug 893191, fix content-events test for Fennec, ignoring about:home and http addon requests

https://github.com/mozilla/addon-sdk/commit/49bf73d99222bfaf56d8c85534344dbda5a83c2d
Merge pull request #1137 from jsantell/893131-fix-content-events-test

Fix Bug 893191 Fix test-content-events for Fennec, r+=@erikvold
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: