Skip to content
This repository has been archived by the owner. It is now read-only.

when the idle state changes, flush the timer #4186

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

when the idle state changes, flush the timer

fixes #4051
  • Loading branch information
mrose17 committed Sep 22, 2016
commit ffcac8570ec7b8a122ab41358276b7fddc8ffc77
@@ -110,7 +110,24 @@ let notificationTimeout = null
const doAction = (action) => {
var i, publisher

/* TBD: handle

This comment has been minimized.

Copy link
@ayumi

ayumi Sep 22, 2016

Contributor

what's this– does it pertain to the idle state

This comment has been minimized.

Copy link
@mrose17

mrose17 Sep 22, 2016

Author Member

yes, at some time in the future, we're not going to count idle when you are on an audio(/video) page... so, i've just put in a comment as a reminder that a check can start there.

{ actionType: "window-set-blocked-by"
, frameProps:
{ audioPlaybackActive: true
...
}
, ...
}
*/
if (publisherInfo._internal.verboseP) {
console.log('\napplication event: ' + JSON.stringify(underscore.pick(action, [ 'actionType', 'key' ]), null, 2))
}
switch (action.actionType) {
case appConstants.APP_IDLE_STATE_CHANGED:
visit('NOOP', underscore.now(), null)
break

case appConstants.APP_CHANGE_SETTING:
if (action.key === settings.PAYMENTS_ENABLED) return initialize(action.value)
if (action.key === settings.PAYMENTS_CONTRIBUTION_AMOUNT) return setPaymentInfo(action.value)
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.