Handle pinned sites separately when doing contribution

[NejcZdovc]

Test Plan

#13530

Description

Pinned sites should be excluded from the statistical voting.

[LaurenWags]

@NejcZdovc @mrose17 How 'off' is acceptable for pinned sites (per the test plan)? one of mine is off by almost 20:

cc @kjozwiak @btlechowski

[NejcZdovc]

@LaurenWags can you please provide monthly amount, ledger-state and which version you are on

[LaurenWags]

monthly amount is 7.5 and I was using 0.22.7. I don't have ledger state.

[NejcZdovc]

10 BAT (15 votes)

ledger-state.json.zip

Publisher	Pinned	% table	Votes	% votes
slo-tech.com	Yes	30	5	33.33
Brave on YouTube	Yes	20	4	26.67
TechGumbo on YouTube	Yes	10	3	20
24ur.com	Yes	5	1	6.67
Chris' Tutorials on YouTube	No	17	0	0
ThioJoe on YouTube	No	8	1	6.67
Popesites on YouTube	No	6	1	6.67
CNET on YouTube	No	4	0	0

---

7.5 BAT (11 votes)

ledger-state.json.zip

Publisher	Pinned	% table	Votes	% votes
slo-tech.com	Yes	30	4	36.36
Brave on YouTube	Yes	20	3	27.27
TechGumbo on YouTube	Yes	10	1	9.09
24ur.com	Yes	5	1	9.09
Chris' Tutorials on YouTube	No	17	2	18.18
ThioJoe on YouTube	No	8	0	0
Popesites on YouTube	No	6	0	0
CNET on YouTube	No	4	0	0

[NejcZdovc]

Closing this as this was implemented, but we will need to fine tune it a little bit more. Will create another issue for that. It's better, but we can still improve it.

Issue opened #13620

[LaurenWags]

Thanks for opening up the new issue @NejcZdovc !

[LaurenWags]

Verified on macOS 10.12.6 x64 using the following build:

• 0.22.8 3ae27f2
• libchromiumcontent: 65.0.3325.181
• muon: 5.1.2

Verified on Mint 18.3 x64 using the following build:

• 0.22.11 56de947
• libchromiumcontent: 65.0.3325.181
• muon: 5.1.2

Verified on Win 10 x64 using the following build:

• 0.22.11 56de947
• libchromiumcontent: 65.0.3325.181
• muon: 5.1.2