GitLab

Implemented Strategy Pattern so that the class can be tested more easily.
Moved all external dependencies into an injectable class (SplitCompatStrategy).
Removed "spying" from unit tests.

Bug: 949729
Change-Id: If35af947f207a429e1d9badeb5b1b1fdb978e5ed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1752504Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Commit-Queue: Fred Mello <fredmello@chromium.org>
Cr-Commit-Position: refs/heads/master@{#688191}