Visited links are not reset when closing Brave

[warwickmm]

Test plan

1. Launch Brave on Linux
2. Under Preferences -> Security -> Private Data, set Brave to clear all data upon closing (flip all the switches to true).
3. Go to https://www.google.com/#q=test and click on the first result (or multiple, doesn't matter).
4. Close that tab; open a new tab; Go to https://www.google.com/#q=test again, and see that the first result is rendered as a visited link (ex: it's purple in color).
5. Close the Google tab and close Brave.
6. Launch Brave, and note that the browsing history has been cleared (visit about:history to confirm).
7. In a new tab, open https://www.google.com/#q=test
8. The link(s) you clicked in step 3 should be un-visited (ex: blue in color)

Original issue description

Under Preferences -> Security -> Private Data, I have selected all data to be cleared when closing Brave. However, visited links are not reset when closing Brave. If I explicitly go to History -> Clear Browsing Data, then the visited links are reset.

Steps to Reproduce

1. Under Preferences -> Security -> Private Data, set Brave to clear all data upon closing.
2. Go to https://www.google.com/#q=test and click on the first result.
3. Go to https://www.google.com/#q=test again, and see that the first result is rendered as a visited link.
4. Close Brave.
5. Launch Brave, and note that the browsing history has been cleared.
6. Go to https://www.google.com/#q=test, and note that the first result is still rendered as a visited link.
7. Go to History -> Clear Browsing Data, and clear all data.
8. Go to https://www.google.com/#q=test, and note that the first result is no longer rendered as a visited link.

Reproduces how often:
100%

Brave Version

Name	Version
Brave	0.23.19
V8	6.7.288.46
rev	178c3fb
Muon	7.1.3
OS Release	4.17.2-1-ARCH
Update Channel	Release
OS Architecture	x64
OS Platform	Linux
Node.js	7.9.0
Tor	0.3.3.7 (git-035a35178c92da94)
Brave Sync	v1.4.2
libchromiumcontent	67.0.3396.87

Reproducible on current live release:
This is reproducible on version 0.23.19 (178c3fb)
This is reproducible on version 0.23.31 (3148ace)
This is reproducible on version 0.23.73 (50bdb6d)

Additional Information

This is related to issue #9351.

[diracdeltas]

marking this as regression of #9351

[bsclifton]

Trying to reproduce:

• macOS: can't reproduce
• Ubuntu 14/16 w/ Gnome: can't reproduce
• Ubuntu 14 w/ KDE can't reproduce
• Ubuntu 16 w/ KDE: able to reproduce
• Windows 10 x64: can't reproduce

I verified the patch that originally fixed this issue is in place (with our current Muon):
brave/muon#219

[bsclifton]

@warwickmm are you using KDE by chance? Do you have more information about your setup? (distro, window manager version, etc)

[diracdeltas]

maybe the history delete callback in muon is not called before the browser shuts down?

[warwickmm]

Sorry, I should have included more details about my setup in the description. I'm running Arch Linux with the Cinnamon desktop environment.

[warwickmm]

I should perhaps mention that I also have an installation of Chromium on the machine. Not sure if that matters?

[bsclifton]

Based on my results above, one intermittent case I didn't mention (browsing on KDE, closing Brave, opening in Gnome; links showed as visited - Ubuntu 14), and that @rebron was able to reproduce once (but not again), I think this might be exactly the case @diracdeltas is mentioning... which might be a race condition

[bsclifton]

Should be fixed with #15028