open.source/stylus
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Show active NTP icon correctly on startup

Browse Source
This commit is contained in:
tophf 2017-04-17 18:56:37 +03:00
parent b3b1d4a628
commit dca6aecd20
1 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
background.js
View File

@ -17,7 +17,6 @@ chrome.webNavigation.onReferenceFragmentUpdated.addListener(data => {
webNavigationListener('styleReplaceAll', data);
});
function webNavigationListener(method, data) {
getStyles({matchUrl: data.url, enabled: true, asHash: true}, styles => {
// we can't inject chrome:// and chrome-extension:// pages
@ -263,10 +262,7 @@ function refreshAllTabs() {
function updateIcon(tab, styles) {
// while NTP is still loading only process the request for its main frame with a real url
// (but when it's loaded we should process style toggle requests from popups, for example)
const isNTP = tab.url == 'chrome://newtab/';
if (isNTP && tab.status != 'complete' || tab.id < 0) {
if (tab.id < 0) {
return;
}
if (styles) {
@ -278,12 +274,9 @@ function updateIcon(tab, styles) {
});
return;
}
if (isNTP) {
getTabRealURL(tab).then(url =>
getStyles({matchUrl: url, enabled: true, asHash: true}, stylesReceived));
} else {
getStyles({matchUrl: tab.url, enabled: true, asHash: true}, stylesReceived);
}
getStyles({matchUrl: url, enabled: true, asHash: true},
stylesReceived));
function stylesReceived(styles) {
let numStyles = styles.length;