1093607 - e10s - Fix browser/base/content/test/general/browser_trackingUI.js

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[:Gijs (out for now; he/him)]

26 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_trackingUI.js | Tracking Content Doorhanger did appear when protection was ON and tracking was present - Didn't expect null, but got it
Stack trace:
    chrome://mochikit/content/browser-test.js:test_isnot:832
    chrome://mochitests/content/browser/browser/base/content/test/general/browser_trackingUI.js:testTrackingPage:65
    chrome://mochitests/content/browser/browser/base/content/test/general/browser_trackingUI.js:null:130
    self-hosted:next:883
    Tester_execTest@chrome://mochikit/content/browser-test.js:645:9
    Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:571:7
    SimpleTest.waitForFocus/maybeRunTests/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:683:49
    this.DOMApplicationRegistry.doGetList@resource://gre/modules/Webapps.jsm:1344:7
    this.DOMApplicationRegistry.receiveMessage@resource://gre/modules/Webapps.jsm:1147:1
27 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/general/browser_trackingUI.js | Uncaught exception - at chrome://mochitests/content/browser/browser/base/content/test/general/browser_trackingUI.js:66 - TypeError: notification is null
Stack trace:
    testTrackingPage@chrome://mochitests/content/browser/browser/base/content/test/general/browser_trackingUI.js:66:3
    @chrome://mochitests/content/browser/browser/base/content/test/general/browser_trackingUI.js:130:1
    Tester_execTest@chrome://mochikit/content/browser-test.js:645:9
    Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:571:7
    SimpleTest.waitForFocus/maybeRunTests/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:683:49
    this.DOMApplicationRegistry.doGetList@resource://gre/modules/Webapps.jsm:1344:7
    this.DOMApplicationRegistry.receiveMessage@resource://gre/modules/Webapps.jsm:1147:1
    Tester_execTest@chrome://mochikit/content/browser-test.js:645:9
    Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:571:7
    SimpleTest.waitForFocus/maybeRunTests/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:683:49
    this.DOMApplicationRegistry.doGetList@resource://gre/modules/Webapps.jsm:1344:7
    this.DOMApplicationRegistry.receiveMessage@resource://gre/modules/Webapps.jsm:1147:1
SUITE-END | took 4s


Doorhangers generally work, so there's something else wrong here.

Comment 1

[[:mmc] Monica Chew (no longer reading bugmail)]

https://bugzilla.mozilla.org/show_bug.cgi?id=1088457 enables the shield to show up in e10s.

Comment 2

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: Enable this test.

I forgot to check this in with the rest of the changes. I'll check it in (with the r=mmc/sworkman from the other bug) on Monday.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ec82fb8c9127

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ec82fb8c9127