Commit 5c606106 authored by Meg Viar's avatar Meg Viar
Browse files

Bug 1787783 - return boolean for userEnabledActiveColorway targeting param r=emcminn 

For consistency, always return a boolean for `userEnabledActiveColorway` targeting param.

This change should have no effect on current functionality.

Differential Revision: https://phabricator.services.mozilla.com/D155847
parent 6f463ede

browser/components/newtab/lib/ASRouterTargeting.jsm

+1 −1
Original line number Diff line number Diff line
@@ -774,7 +774,7 @@ const TargetingGetters = { 
    }

    return QueryCache.getters.currentThemes.get().then(themes => {

      let themeId = themes.find(theme => theme.isActive)?.id;

      return (

      return !!(

        themeId && lazy.BuiltInThemes.isColorwayFromCurrentCollection(themeId)

      );

    });