Sever the connection between the test support library and snapshot

This dependency was created in 107fb7631788 for macOS, but it’s not used
on other platforms. Since the test support library is broadly used to
test all of Crashpad, it’s useful even during early-stage porting. The
snapshot library is a higher-level module that builds upon other
components, and is not likely to be functional until the later stages of
porting. Expressing this dependency artifically makes it difficult to
test ports in development.

Change-Id: I9dc2e2c473c8519b4c2b0d774acc9c146ee4e121
Reviewed-on: https://chromium-review.googlesource.com/811564
Reviewed-by: Scott Graham <scottmg@chromium.org>
Commit-Queue: Mark Mentovai <mark@chromium.org>

test/BUILD.gn

@@ -90,7 +90,6 @@ static_library("test") {
 
   deps = [
     "../compat",
-    "../snapshot",
     "../util",
     "//base",
     "//testing/gtest",
@@ -98,7 +97,10 @@ static_library("test") {
 
   if (is_mac) {
     libs = [ "bsm" ]
-    deps += [ "../handler" ]
+    deps += [
+      "../handler",
+      "../snapshot",
+    ]
   }
 }
 

test/test.gyp

@@ -22,7 +22,6 @@
       'type': 'static_library',
       'dependencies': [
         '../compat/compat.gyp:crashpad_compat',
-        '../snapshot/snapshot.gyp:crashpad_snapshot',
         '../third_party/gtest/gtest.gyp:gtest',
         '../third_party/mini_chromium/mini_chromium.gyp:base',
         '../util/util.gyp:crashpad_util',
@@ -85,6 +84,7 @@
         ['OS=="mac"', {
           'dependencies': [
             '../handler/handler.gyp:crashpad_handler_lib',
+            '../snapshot/snapshot.gyp:crashpad_snapshot',
           ],
           'link_settings': {
             'libraries': [