Closed Bug 1146555 Opened 9 years ago Closed 4 years ago

[e10s] browser_ssl_error_reports.js causes unsafe CPOW usage warnings

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME
Tracking Status
e10s + ---

People

(Reporter: Kwan, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [unsafe-cpow-usage])

+++ This bug was initially created as a clone of Bug #1133577 +++

Mined from test logs

In browser/base/content/test/general/browser_ssl_error_reports.js:

      onStateChange: function(aWebProgress, aRequest, aStateFlags, aStatus) {
        let doc = aBrowser.contentDocument;

        if (doc && doc.getElementById("reportCertificateError")) { <- Causes CPOW warning
          // Wait until the documentURI changes (from about:blank) this should
          // be the error page URI.
          let documentURI = doc.documentURI; <- Causes CPOW warning
          if (documentURI == originalDocumentURI) {
            return;
          }

[...]

let testSendReportManual = function*(testURL, suffix) {
  setup();
  Services.prefs.setBoolPref("security.ssl.errorReporting.enabled", true);
  Services.prefs.setCharPref("security.ssl.errorReporting.url",
    "https://example.com/browser/browser/base/content/test/general/pinning_reports.sjs?" + suffix);

  let tab = gBrowser.addTab(testURL, {skipAnimation: true});
  let browser = tab.linkedBrowser;
  let mm = browser.messageManager;
  mm.loadFrameScript(ROOT + "browser_ssl_error_reports_content.js", true);

  gBrowser.selectedTab = tab;

  // ensure we have the correct error message from about:neterror
  let netError = createNetworkErrorMessagePromise(browser);
  yield netError;
  netError.then(function(val){
    is(val.startsWith("An error occurred during a connection to"), true,
                      "ensure the correct error message came from about:neterror");
  });

  let btn = browser.contentDocument.getElementById("reportCertificateError"); <- Causes CPOW warning
  let deferredReportSucceeds = Promise.defer();
Component: Menus → General
Blocks: e10s-tests
No longer blocks: 1109869
Blocks: 1109869
tracking-e10s: m8+ → ---
Whiteboard: [unsafe-cpow-usage]
This is test only.
tracking-e10s: --- → +

This looks done to me.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.