Fix for discrepancies in presenting results of Smart Blocking actions. Fix for GH-952.

app/setup/components/Views/AdditionalFeaturesView.jsx

@@ -45,7 +45,9 @@ class AdditionalFeaturesView extends Component {
 	 * @param  {Object} event The event created by the onChange property
 	 */
 	_handleAntiTrack = (event) => {
-		this.props.actions.updateAntiTrack(event.target.checked);
+		if (!IS_CLIQZ) {
+			this.props.actions.updateAntiTrack(event.target.checked);
+		}
 	}
 
 	/**
@@ -61,7 +63,9 @@ class AdditionalFeaturesView extends Component {
 	 * @param  {Object} event The event created by the onChange property
 	 */
 	_handleAdBlock = (event) => {
-		this.props.actions.updateAdBlock(event.target.checked);
+		if (!IS_CLIQZ) {
+			this.props.actions.updateAdBlock(event.target.checked);
+		}
 	}
 
 	/**

app/setup/reducers/additionalFeatures.js

@@ -40,6 +40,9 @@ const initialState = {
 export default (state = initialState, action) => {
 	switch (action.type) {
 		case UPDATE_ANTITRACK: {
+			if (IS_CLIQZ) {
+				return state;
+			}
 			msg.sendMessage('updateAntiTrack', action.data);
 			return Object.assign({}, state, {
 				antiTrack: action.data,
@@ -52,6 +55,9 @@ export default (state = initialState, action) => {
 			});
 		}
 		case UPDATE_ADBLOCK: {
+			if (IS_CLIQZ) {
+				return state;
+			}
 			msg.sendMessage('updateAdBlock', action.data);
 			return Object.assign({}, state, {
 				adBlock: action.data,

src/background.js

@@ -45,6 +45,7 @@ import { allowAllwaysC2P } from './utils/click2play';
 import * as common from './utils/common';
 import * as utils from './utils/utils';
 
+
 // class instantiation
 const button = new Button();
 const events = new Events();
@@ -92,7 +93,6 @@ function setCliqzModuleEnabled(module, enabled) {
 	log('MODULE IS ALREADY IN CORRECT STATE', module, enabled);
 	return Promise.resolve(CORRECT_STATE);
 }
-
 /**
  * Check and fetch (if needed) a new tracker library every 12 hours
  * @memberOf Background
@@ -1015,6 +1015,7 @@ offers.on('enabled', () => {
 		setCliqzModuleEnabled(messageCenter, true);
 	});
 });
+
 /**
  * Set listener for 'enabled' event for Offers module.
  * It registers message handler for messages with the offers.

src/classes/PolicySmartBlock.js

@@ -50,28 +50,23 @@ class PolicySmartBlock {
 	shouldUnblock(appId, catId, tabId, pageURL, requestType) {
 		if (!this.shouldCheck(tabId, appId)) { return false; }
 
-		// allow if tracker is in compatibility list
-		if (this._appHasKnownIssue(tabId, appId, pageURL)) {
-			return true;
-		}
-
-		// allow if tracker is in breaking category
-		if (this._allowedCategories(tabId, appId, catId)) {
-			return true;
-		}
+		let reason;
 
-		// allow if tracker is in breaking type
-		if (this._allowedTypes(tabId, appId, requestType)) {
-			return true;
+		if (this._appHasKnownIssue(tabId, appId, pageURL)) {
+			reason = 'hasIssue'; 		// allow if tracker is in compatibility list
+		} else if (this._allowedCategories(tabId, appId, catId)) {
+			reason = 'allowedCategory'; // allow if tracker is in breaking category
+		} else if (this._allowedTypes(tabId, appId, requestType)) {
+			reason = 'allowedType'; 	// allow if tracker is in breaking type
+		} else if (this._pageWasReloaded(tabId, appId)) {
+			reason = 'pageReloaded'; 	// allow if page has been reloaded recently
 		}
 
-		// allow if page has been reloaded recently
-		if (this._pageWasReloaded(tabId, appId)) {
+		if(reason) {
+			tabInfo.setTabSmartBlockAppInfo(tabId, appId, reason, false);
 			return true;
 		}
 
-		// @TODO tabInfo.setTabSmartBlockAppInfo() should happen here
-
 		return false;
 	}
 	/**
@@ -87,41 +82,45 @@ class PolicySmartBlock {
 	shouldBlock(appId, catId, tabId, pageURL, requestType, requestTimestamp) {
 		if (!this.shouldCheck(tabId, appId)) { return false; }
 
+		let reason;
+
 		// block if it's been more than 5 seconds since page load started
 		if (this._requestWasSlow(tabId, appId, requestTimestamp)) {
-			// allow if tracker is in compatibility list
-			if (this._appHasKnownIssue(tabId, appId, pageURL)) {
-				return true;
-			}
+			reason = 'slow';
 
-			// allow if tracker is in breaking category
-			if (this._allowedCategories(tabId, appId, catId)) {
-				return true;
-			}
-
-			// allow if tracker is in breaking type
-			if (this._allowedTypes(tabId, appId, requestType)) {
-				return true;
-			}
-
-			// allow if page has been reloaded recently
-			if (this._pageWasReloaded(tabId, appId)) {
-				return true;
+			if (this._appHasKnownIssue(tabId, appId, pageURL)) {
+				reason = 'hasIssue'; 		// allow if tracker is in compatibility list
+			} else if (this._allowedCategories(tabId, appId, catId)) {
+				reason = 'allowedCategory'; // allow if tracker is in breaking category
+			} else if (this._allowedTypes(tabId, appId, requestType)) {
+				reason = 'allowedType'; 	// allow if tracker is in breaking type
+			} else if (this._pageWasReloaded(tabId, appId)) {
+				reason = 'pageReloaded'; 	// allow if page has been reloaded recently
 			}
 		}
 
-		// TODO tabInfo.setTabSmartBlockAppInfo() should happen here
+		const result = (reason === 'slow');
+		if(result) {
+			//We don't want record in tabInfo reasons other than 'slow'
+			//Smart blocking should not claim that it unblock trackers which were unlocked
+			//for other reasons before shouldBlock was called for them.
+			tabInfo.setTabSmartBlockAppInfo(tabId, appId, 'slow', true);
+		}
 
-		return false;
+		return result;
 	}
 
+
 	/**
 	 * Check if Smart Block should proceed based on:
 	 * 1. Smart Block is enabled
 	 * 2. Paused blocking is disabled
 	 * 3. Page is neither whitelisted or blacklisted
 	 * 4. Tracker is not site-specific unblocked
 	 * 5. Tracker is not site-specific blocked
+	 * 
+	 * @param  {number} 			tabId 	tab id
+	 * @param  {string | boolean} 	appId 	tracker id
 	 * @return {boolean}
 	 */
 	shouldCheck(tabId, appId = false) {
@@ -141,6 +140,7 @@ class PolicySmartBlock {
 	// and requestHost is "some.other.subdomain.domain.com"
 	/**
 	 * Check if request host matches page host
+	 * @param  {number} tabId			tab id
 	 * @param  {string} pageHost		host of the page url
 	 * @param  {string} requestHost		host of the request url
 	 * @return {boolean}
@@ -164,75 +164,62 @@ class PolicySmartBlock {
 	}
 
 	/**
-	 * Check if tab was reloaded
-	 * @param  {string} tabId		tab id
+	 * Check if tab was reloaded.
+	 * @param  {number} tabId		tab id
+	 * @param  {string} appId 		app id
 	 * @return {boolean}
 	 */
 	_pageWasReloaded(tabId, appId) {
-		const checks = tabInfo.getTabInfo(tabId, 'reloaded') || false;
-		if (checks) {
-			tabInfo.setTabSmartBlockAppInfo(tabId, appId, 'pageReloaded', false);
-		}
-
-		return checks;
+		return tabInfo.getTabInfo(tabId, 'reloaded') || false;
 	}
 
 	/**
-	 * Check if app has a known issue with a URL
-	 * @param  {string} appId		tracker id
-	 * @param  {string} pageURL		tab url
-	 * @return {boolean}
+	 * Check if app has a known issue with a URL.
+	 * @param 	{number} tabId 		tab id
+	 * @param  	{string} appId		tracker id
+	 * @param  	{string} pageURL	tab url
+	 * @return 	{boolean}
 	 */
 	_appHasKnownIssue(tabId, appId, pageURL) {
-		const checks = compDb.hasIssue(appId, pageURL);
-		if (checks) {
-			tabInfo.setTabSmartBlockAppInfo(tabId, appId, 'hasIssue', false);
-		}
-
-		return checks;
+		return compDb.hasIssue(appId, pageURL);
 	}
 
 	/**
 	 * Check if HTTP or WS (insecure web socket) request is loading on a HTTPS page
-	 * @param  {string} pageProtocol		protocol of the tab url
-	 * @param  {string} requestProtocol		protocol of the request url
-	 * @return {boolean}
+	 * @param 	{number} tabId 				tab id
+	 * @param  	{string} pageProtocol		protocol of the tab url
+	 * @param  	{string} requestProtocol	protocol of the request url
+	 * @return 	{boolean}
 	 */
 	isInsecureRequest(tabId, pageProtocol, requestProtocol) {
 		if (!this.shouldCheck(tabId)) { return false; }
 
-		const checks = (
+		return (
 			pageProtocol === 'https' &&
 			(requestProtocol === 'http' || requestProtocol === 'ws') || false
 		);
-		if (checks) {
-			// tabInfo.setTabSmartBlockInfo(tabId, 'isInsecure');
-		}
-
-		return checks;
 	}
 
 	/**
 	 * Check if given category is in the list of whitelisted categories
-	 * @param  {string} catId  		category id
+	 * @param 	{number} 	tabId 		tab id
+	 * @param  	{string} 	appId		tracker id
+	 * @param  	{string} 	catId  		category id
 	 * @return {boolean}
 	 */
 	_allowedCategories(tabId, appId, catId) {
-		const checks = this.allowedCategoriesList.includes(catId);
-		if (checks) {
-			tabInfo.setTabSmartBlockAppInfo(tabId, appId, 'allowedCategory', false);
-		}
-
-		return checks;
+		return this.allowedCategoriesList.includes(catId);
 	}
 
+	/**
+	 * Check if given request type is in the list of whitelisted requests
+	 * @param 	{number} 	tabId 				tab id
+	 * @param  	{string} 	appId				tracker id
+	 * @param  	{string} 	requestType  		request type
+	 * @return {boolean}
+	 */
 	_allowedTypes(tabId, appId, requestType) {
-		const checks = this.allowedTypesList.includes(requestType);
-		if (checks) {
-			tabInfo.setTabSmartBlockAppInfo(tabId, appId, 'allowedType', false);
-		}
-
-		return checks;
+		return this.allowedTypesList.includes(requestType);
 	}
 
 	/**
@@ -252,20 +239,16 @@ class PolicySmartBlock {
 	}
 
 	/**
-	 * Check if request loaded after a threshhold time since page load
-	 * @param  {string} tabId			tab id
-	 * @param  {number} requestTimestamp   timestamp of the request
-	 * @return {boolean}
+	 * Check if request loaded after a threshhold time since page load.
+	 * @param  	{string}	tabId				tab id
+	 * @param  	{string} 	appId				tracker id
+	 * @param  	{number} 	requestTimestamp   	timestamp of the request
+	 * @return 	{boolean}
 	 */
 	_requestWasSlow(tabId, appId, requestTimestamp) {
 		const THRESHHOLD = 5000; // 5 seconds
 		const pageTimestamp = tabInfo.getTabInfo(tabId, 'timestamp');
-		const checks = (requestTimestamp - pageTimestamp > THRESHHOLD) || false;
-		if (checks) {
-			tabInfo.setTabSmartBlockAppInfo(tabId, appId, 'slow', true);
-		}
-
-		return checks;
+		return (requestTimestamp - pageTimestamp > THRESHHOLD) || false;
 	}
 }