Add pause Ghostery trigger for Metrics broken page watcher

src/background.js

@@ -1101,7 +1101,7 @@ function initializeDispatcher() {
 		button.update();
 		utils.flushChromeMemoryCache();
 		cliqz.modules.core.action('refreshAppState');
-		metrics.handleBrokenPageTrigger(globals.BROKEN_PAGE_WHITELISTED);
+		metrics.handleBrokenPageTrigger(globals.BROKEN_PAGE_WHITELIST);
 	});
 	dispatcher.on('conf.save.enable_human_web', (enableHumanWeb) => {
 		if (!IS_CLIQZ) {
@@ -1157,6 +1157,7 @@ function initializeDispatcher() {
 
 	dispatcher.on('globals.save.paused_blocking', () => {
 		// update content script state when blocking is paused/unpaused
+		if (globals.SESSION.paused_blocking) { metrics.handleBrokenPageTrigger(globals.BROKEN_PAGE_PAUSE); }
 		cliqz.modules.core.action('refreshAppState');
 	});
 }

src/classes/Globals.js

@@ -72,7 +72,8 @@ class Globals {
 
 		// Broken page metrics named constants
 		this.BROKEN_PAGE_REFRESH = 1;
-		this.BROKEN_PAGE_WHITELISTED = 2;
+		this.BROKEN_PAGE_WHITELIST = 2;
+		this.BROKEN_PAGE_PAUSE = 3;
 
 		// data stores
 		this.REDIRECT_MAP = new Map();

src/classes/Metrics.js

@@ -41,7 +41,7 @@ class Metrics {
 		this.utm_campaign = '';
 		this.ping_set = new Set();
 		this._brokenPageWatcher = {
-			flag: false,
+			on: false,
 			triggerId: '',
 			triggerTime: '',
 			timeoutId: null,
@@ -119,29 +119,41 @@ class Metrics {
 			case globals.BROKEN_PAGE_REFRESH:
 				if (this._brokenPageWatcher.flag) {
 					this.ping('broken-page');
-					this._clearBrokenPageWatcher();
+					this._unplugBrokenPageWatcher();
 				} else {
-					this._setBrokenPageWatcher(triggerId);
+					this._rebootBrokenPageWatcher(triggerId);
+				}
+				break;
+			case globals.BROKEN_PAGE_WHITELIST:
+				if (this._brokenPageWatcher.flag) {
+					this.ping('broken-page');
+					this._unplugBrokenPageWatcher();
+				}
+				break;
+			case globals.BROKEN_PAGE_PAUSE:
+				if (this._brokenPageWatcher.flag) {
+					this.ping('broken-page');
+					this._unplugBrokenPageWatcher();
 				}
 				break;
 			default:
 				break;
 		}
 	}
 
-	_clearBrokenPageWatcher() {
-		this._brokenPageWatcher = Object.assign({}, {
+	_unplugBrokenPageWatcher() {
+		this._clearBrokenPageWatcherTimeout();
+
+		this._brokenPageWatcher = Object.assign({},{
 			flag: false,
 			triggerId: '',
 			triggerTime: '',
 			timeoutId: null,
 		});
 	}
 
-	_setBrokenPageWatcher(triggerId) {
-		const { timeoutId } = this._brokenPageWatcher;
-
-		if (timeoutId) { clearTimeout(timeoutId); }
+	_rebootBrokenPageWatcher(triggerId) {
+		this._clearBrokenPageWatcherTimeout();
 
 		this._brokenPageWatcher = Object.assign({}, {
 			flag: true,
@@ -151,6 +163,11 @@ class Metrics {
 		});
 	}
 
+	_clearBrokenPageWatcherTimeout() {
+		const { timeoutId } = this._brokenPageWatcher;
+		if (timeoutId) { clearTimeout(timeoutId); }
+	}
+
 	/**
 	* Prepare data and send telemetry pings.
 	* @param {string} type    type of the telemetry ping