Pinning disabled publisher sets % value to 1 and not to actual value

[srirambv]

Description

Pinning disabled publisher sets % value to 1 and not to actual value

Steps to Reproduce

1. Enable payments and visit 7-8 sites to add it in the ledger table
2. Ensure each site has a decent amount of % value available
3. Pin one of site with max time spent so that the major chunk of % value is set
4. Pin second publisher which is enabled, ensure the % value remains as it was when it was enabled
5. Exclude third publisher, ensure the % value is set to 0 from original calculated value
6. Pin the excluded publisher, % value is set to 1
7. Unpin the publisher pinned in step 6, shows actual value for %

Actual result:

Expected result:
Pinning excluded publisher should show the actual % value based on time visited and not set to 1

Reproduces how often:
100%

Brave Version

about:brave info:

Brave	0.19.95
rev	cc0ebad
Muon	4.5.16
libchromiumcontent	62.0.3202.94
V8	6.2.414.42
Node.js	7.9.0
Update Channel	Release
OS Platform	Microsoft Windows
OS Release	10.0.15063
OS Architecture	x64

Reproducible on current live release:
Yes, same behaviour on 0.19.88

Additional Information

#11369 #11238 #10534
cc: @NejcZdovc @bsclifton

[NejcZdovc]

This is not a bug, we want to set it to 1%, because it was disabled so we don't know the real value that it has right now.