Remove IsSyncActive() checks from ProfileSyncService's unit tests

As of https://crrev.com/c/1138613, IsSyncActive() is implemented in
terms of GetState(), in the base class SyncService. All the tests that
checked IsSyncActive() also check GetState(), so the IsSyncActive()
checks are now entirely redundant.

Bug: 839834
Change-Id: I1f3898a84643f6f8df7eef91dd038b6b6be30822
Reviewed-on: https://chromium-review.googlesource.com/1143397
Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#576811}
2 files changed