845387 - private-browsing isPrivate should take a content script instance

Status: RESOLVED FIXED

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

Description

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

For example:

PageMod({
  ...
  onAttach: function(mod) {
    require('private-browsing').isPrivate(mod); // should be true when the window is private
    // require('private-browsing').isPrivate(mod.tab) is already true, but should have a test
  }
})

Comment 1

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

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

Pointer to Github pull-request

Comment 2

[[github robot]]

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

https://github.com/mozilla/addon-sdk/commit/e14076eff7bee97d70b29e2939eb2231d03c5597
Bug 845387: private-browsing isPrivate should take a content script instance

https://github.com/mozilla/addon-sdk/commit/86d1660710cd4436c8e076a4142ddfc923af4140
Merge pull request #853 from erikvold/845387

Bug 845387: private-browsing isPrivate should take a content script instance