Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Commit

Permalink
Browse files Browse the repository at this point in the history
Deal with lack of MOZ_PHOTON_THEME on 57+
  • Loading branch information
gijsk committed Aug 14, 2017
1 parent d64d224 commit 3e9eba3
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion addon/bootstrap.js
Expand Up @@ -230,7 +230,9 @@ let photonPageAction;
// Does nothing otherwise. Ideally, in the future, WebExtension page actions
// and Photon page actions would be one in the same, but they aren't right now.
function initPhotonPageAction(api) {
if (!AppConstants.MOZ_PHOTON_THEME) {
// The MOZ_PHOTON_THEME ifdef got removed, but we need to support 55 and 56 as well,
// so check if the property exists *and* is false before bailing.
if (typeof AppConstants.MOZ_PHOTON_THEME != "undefined" && !AppConstants.MOZ_PHOTON_THEME) {
// Photon not supported. Use the WebExtension's browser action.
return;
}
Expand Down

0 comments on commit 3e9eba3

Please sign in to comment.