Windows

Quick explanation of the non trivial change: this removes a checksum test and a test for a string which we don't use. This is code that was brought over with ABP. The reason for removing this is that it was preventing the cookie block list from updating. It doesn't seem necessary to me, and since I am trying to rid this code base of ABP anyway I think it's best just to delete it. cc @jsha for review
Also I believe it's jsdoc style to not include the function name at the top of the comment block, but just start right in with the description.

By the way, if these are local style things, feel free to keep them matching local style.