Observe network changes from network thread in DomainReliabilityMonitor.

Observe network changes from network thread in DomainReliabilityMonitor.

From observer_list_threadsafe.h:
  // If the observer to be removed is in the list, RemoveObserver MUST
  // be called from the same thread which called AddObserver.

This rule is currently not respected by DomainReliabilityMonitor, which
prevents the addition of a DCHECK.

With this CL, the rule is respected and |last_network_change_time_|
is read/written from the same thread (no data race).

BUG=

Review-Url: https://codereview.chromium.org/2627523003
Cr-Commit-Position: refs/heads/master@{#443283}
Committed: https://chromium.googlesource.com/chromium/src/+/b0002b84acf5c71a6c257a068fb56d588e5e9d47

[fdoray, 2017-01-10 13:35:09]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-10 13:35:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fdoray, 2017-01-10 13:35:47]

Description was changed from

==========
Observe network changes from network thread in DomainReliabilityMonitor.

From observer_list_threadsafe.h:
  // If the observer to be removed is in the list, RemoveObserver MUST
  // be called from the same thread which called AddObserver.

This rule is currently not respected by DomainReliabilityMonitor, which
prevents the addition of a DCHECK.

BUG=
==========

to

==========
Observe network changes from network thread in DomainReliabilityMonitor.

From observer_list_threadsafe.h:
  // If the observer to be removed is in the list, RemoveObserver MUST
  // be called from the same thread which called AddObserver.

This rule is currently not respected by DomainReliabilityMonitor, which
prevents the addition of a DCHECK.

With this CL, the rule is respected and |last_network_change_time_|
is read/written from the same thread.

BUG=
==========

[fdoray, 2017-01-10 13:40:02]

Description was changed from

==========
Observe network changes from network thread in DomainReliabilityMonitor.

From observer_list_threadsafe.h:
  // If the observer to be removed is in the list, RemoveObserver MUST
  // be called from the same thread which called AddObserver.

This rule is currently not respected by DomainReliabilityMonitor, which
prevents the addition of a DCHECK.

With this CL, the rule is respected and |last_network_change_time_|
is read/written from the same thread.

BUG=
==========

to

==========
Observe network changes from network thread in DomainReliabilityMonitor.

From observer_list_threadsafe.h:
  // If the observer to be removed is in the list, RemoveObserver MUST
  // be called from the same thread which called AddObserver.

This rule is currently not respected by DomainReliabilityMonitor, which
prevents the addition of a DCHECK.

With this CL, the rule is respected and |last_network_change_time_|
is read/written from the same thread (no data race).

BUG=
==========

[commit-bot: I haz the power, 2017-01-10 14:33:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-10 14:33:05]

Dry run: This issue passed the CQ dry run.

[fdoray, 2017-01-10 15:24:04]

fdoray@chromium.org changed reviewers:
+ juliatuttle@chromium.org

[fdoray, 2017-01-10 15:24:04]

PTAL

[Julia Tuttle, 2017-01-12 16:46:30]

lgtm, thanks.

[fdoray, 2017-01-12 16:47:26]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2017-01-12 16:47:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-12 17:52:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484239646891190, "parent_rev":
"1fc7b35ee14c30ec55fbdb6612821804f49c3afa", "commit_rev":
"b0002b84acf5c71a6c257a068fb56d588e5e9d47"}

[commit-bot: I haz the power, 2017-01-12 17:53:25]

Description was changed from

==========
Observe network changes from network thread in DomainReliabilityMonitor.

From observer_list_threadsafe.h:
  // If the observer to be removed is in the list, RemoveObserver MUST
  // be called from the same thread which called AddObserver.

This rule is currently not respected by DomainReliabilityMonitor, which
prevents the addition of a DCHECK.

With this CL, the rule is respected and |last_network_change_time_|
is read/written from the same thread (no data race).

BUG=
==========

to

==========
Observe network changes from network thread in DomainReliabilityMonitor.

From observer_list_threadsafe.h:
  // If the observer to be removed is in the list, RemoveObserver MUST
  // be called from the same thread which called AddObserver.

This rule is currently not respected by DomainReliabilityMonitor, which
prevents the addition of a DCHECK.

With this CL, the rule is respected and |last_network_change_time_|
is read/written from the same thread (no data race).

BUG=

Review-Url: https://codereview.chromium.org/2627523003
Cr-Commit-Position: refs/heads/master@{#443283}
Committed:
https://chromium.googlesource.com/chromium/src/+/b0002b84acf5c71a6c257a068fb5...
==========

[commit-bot: I haz the power, 2017-01-12 17:53:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b0002b84acf5c71a6c257a068fb5...