Make all SafeBrowsing* unit_tests work under AddressSanitizer on Windows

Make all SafeBrowsing* unit_tests work under AddressSanitizer on Windows

AddressSanitizer (ASan) uses code patching of every module of an application at startup / module load.
As a result, many tests that check the number of modified functions misbehave.
I'm disabling most of them and adjusting one that's easy to generalize.

I've also replaced some EXPECT_* macros with ASSERT_* per
https://code.google.com/p/googletest/wiki/V1_7_Primer#Assertions
"you should use ASSERT_* if it doesn't make sense to continue when the assertion in question fails"

BUG=345874
Committed: https://crrev.com/12a841b4008156472c958074ed491e11acbc247e
Cr-Commit-Position: refs/heads/master@{#311452}

[Timur Iskhodzhanov, 2015-01-13 20:07:45]

timurrrr@chromium.org changed reviewers:
+ grt@chromium.org, robertshield@chromium.org

[grt (UTC plus 2), 2015-01-13 20:15:51]

lgtm w/ a nit

https://codereview.chromium.org/848923004/diff/1/chrome/browser/safe_browsing...
File
chrome/browser/safe_browsing/incident_reporting/module_integrity_verifier_win_unittest.cc
(right):

https://codereview.chromium.org/848923004/diff/1/chrome/browser/safe_browsing...
chrome/browser/safe_browsing/incident_reporting/module_integrity_verifier_win_unittest.cc:155:
#endif
nit:
#endif  // ADDRESS_SANITIZER

[robertshield, 2015-01-13 20:25:56]

lgtm

[Timur Iskhodzhanov, 2015-01-14 10:20:47]

Thanks for the quick review!

https://codereview.chromium.org/848923004/diff/1/chrome/browser/safe_browsing...
File
chrome/browser/safe_browsing/incident_reporting/module_integrity_verifier_win_unittest.cc
(right):

https://codereview.chromium.org/848923004/diff/1/chrome/browser/safe_browsing...
chrome/browser/safe_browsing/incident_reporting/module_integrity_verifier_win_unittest.cc:155:
#endif
Oops, I always forget this bit :(

[Timur Iskhodzhanov, 2015-01-14 10:21:23]

The CQ bit was checked by timurrrr@chromium.org

[commit-bot: I haz the power, 2015-01-14 10:22:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/848923004/20001

[commit-bot: I haz the power, 2015-01-14 11:21:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-14 11:22:11]

Patchset 2 (id:??) landed as
https://crrev.com/12a841b4008156472c958074ed491e11acbc247e
Cr-Commit-Position: refs/heads/master@{#311452}