Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gh 1763/metrics feature/new triggers for broken page pings #430

Closed
Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
4ecd3d0
Update broken ping reload threshhold to 60 seconds
wlycdgr Jul 17, 2019
9c74083
Merge branch 'develop' into GH-1763/metrics-feature/new-triggers-for-…
wlycdgr Jul 19, 2019
1ed578e
Add startPossibleBrokenPageTimer method to Metrics, implement the del…
wlycdgr Jul 22, 2019
99305fc
Implement metrics broken page trigger handler that uses existing even…
wlycdgr Jul 22, 2019
52e9a7a
Add setup_pth and setup_block parameters to broken_page ping
wlycdgr Jul 22, 2019
311b498
Clear and reset broken page watcher setTimeout handler if a new trigg…
wlycdgr Jul 22, 2019
dd1d215
Add pause Ghostery trigger for Metrics broken page watcher
wlycdgr Jul 22, 2019
49281a8
Add local tracker unblock trigger for Metrics broken page watcher.
wlycdgr Jul 22, 2019
5e5d996
Handle new tab creation for broken page watcher. Add documentation co…
wlycdgr Jul 23, 2019
f2c49ab
Remove broken page metrics ping call from TabInfo#create
wlycdgr Jul 23, 2019
0664aa5
Revert smart unblock threshold to 30 seconds. Add comments explaining…
wlycdgr Jul 23, 2019
28da0df
Trigger metrics broken page watcher on individual tracker site specif…
wlycdgr Jul 23, 2019
e7cd1bc
Pacify linter
wlycdgr Jul 23, 2019
e0076c1
Only set broken page watcher on whitelist add, not both add and remove
wlycdgr Jul 23, 2019
b9cf1ee
Change this.BROKEN_APGE_NEW_TAB value to a less mysterious 5. RIP Age…
wlycdgr Jul 23, 2019
0f5a592
Merge branch 'develop' into GH-1763/metrics-feature/new-triggers-for-…
wlycdgr Jul 29, 2019
d286689
Correct timeout handler name in Metrics#_resetBrokenPageWatcher
wlycdgr Jul 29, 2019
aa76f53
Correct broken_page ping name in Metrics#handleBrokenPageTrigger
wlycdgr Jul 29, 2019
4e18aaa
Bind setTimeout handler to Metrics instance in getActiveTab callback …
wlycdgr Jul 29, 2019
6634a9b
Merge branch 'develop' into GH-1763/metrics-feature/new-triggers-for-…
christophertino Jul 29, 2019
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Add startPossibleBrokenPageTimer method to Metrics, implement the del…
…ayed off switch for the watcher, and set it to trigger on site whitelist
  • Loading branch information
wlycdgr committed Jul 22, 2019
commit 1ed578e1f4912b1530d93ea26d30d11ca716eb5a
@@ -1101,6 +1101,7 @@ function initializeDispatcher() {
button.update();
utils.flushChromeMemoryCache();
cliqz.modules.core.action('refreshAppState');
metrics.startPossibleBrokenPageTimer();
});
dispatcher.on('conf.save.enable_human_web', (enableHumanWeb) => {
if (!IS_CLIQZ) {
@@ -30,6 +30,7 @@ const FIRST_REWARD_METRICS = ['rewards_first_accept', 'rewards_first_reject', 'r
const { METRICS_SUB_DOMAIN, EXTENSION_VERSION, BROWSER_INFO } = globals;
const IS_EDGE = (BROWSER_INFO.name === 'edge');
const MAX_DELAYED_PINGS = 100;
const BROKEN_PAGE_WATCH_THRESHOLD = 60000; // 60 seconds
/**
* Class for handling telemetry pings.
* @memberOf BackgroundClasses
@@ -39,6 +40,9 @@ class Metrics {
this.utm_source = '';
this.utm_campaign = '';
this.ping_set = new Set();
this._pageMightBeBroken = false;
this._pageMightBeBrokenOffSwitch = null;
this._brokenPageOnNewTabListener = null;
}

/**
@@ -107,6 +111,21 @@ class Metrics {
});
}

startPossibleBrokenPageTimer() {
this._pageMightBeBroken = true;

// Stand down after a minute
if (this._pageMightBeBrokenOffSwitch) {
clearTimeout(this._pageMightBeBrokenOffSwitch);
}
this._pageMightBeBrokenOffSwitch = setTimeout(() => {
this._pageMightBeBroken = false;
},
BROKEN_PAGE_WATCH_THRESHOLD
);

}

/**
* Prepare data and send telemetry pings.
* @param {string} type type of the telemetry ping
ProTip! Use n and p to navigate between commits in a pull request.