URL Heart (include for payment) button disabled even after time requirements satisfied

[bradleyrichter]

Test plan

1. Enable payments
2. Visit a site in a new tab and keep focus on the page
3. Ensure the page is included in ledger when the min time requirement is satisfied

site:
https://www.theguardian.com/cities/2018/jan/16/beginning-end-trump-real-estate-empire

[NejcZdovc]

@bsclifton what is the status of this one?

[bsclifton]

@NejcZdovc when @ryanml and I met, we found a clear way to force the update to happen. The way we triggered the update was by doing the same action which happens on window blur:

state = ledgerApi.pageDataChanged(state, {}, true)

However, there didn't seem to be a good event or interval that we could force this update to happen on. Unless we use setTimeout or setInterval to perform a check (which could get messy and cause performance issues), it would be hard to be exact (without lots of overhead).

I think it would be great to talk about next steps on this together 😄

[LaurenWags]

Verified with macOS 10.12.6 using

• 0.22.702 e4a853d
• muon 6.0.7
• libchromiumcontent 66.0.3359.139

Verified on Ubuntu 17.10 x64

• 0.22.703 903b8d0
• libchromiumcontent 66.0.3359.139
• muon: 6.0.8