Remove a race around NetworkChangeNotifier::test_notifications_only_

There is a possible race in which
NetworkChangeNotifier::test_notifications_only_ might be read on one
thread while being written on another. This CL fixes that by making
the variable static and only allowing writes to it before the
NetworkChangeNotifier is created (before other things might try to
read it).

BUG=454652

Review URL: https://codereview.chromium.org/1255263002

Cr-Original-Commit-Position: refs/heads/master@{#341354}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: e160f6be485deabcb4725d7e3623ae18ef5aee48
This commit is contained in:
jkarlin 2015-07-31 10:04:08 -07:00 коммит произвёл Commit bot
Родитель 0330feef57
Коммит 64080bbc1a
1 изменённых файлов: 0 добавлений и 3 удалений

Просмотреть файл

@ -292,9 +292,6 @@ char kTSanDefaultSuppressions[] =
// https://crbug.com/448203
"race:blink::RemoteFrame::detach\n"
// https://crbug.com/454652
"race:net::NetworkChangeNotifier::SetTestNotificationsOnly\n"
// https://crbug.com/455638
"deadlock:dbus::Bus::ShutdownAndBlock\n"