Remove NOTREACHED() in ModuleSnapshot::AnnotationObjects() impls.

This is causing crashpad_handler_test to fail in Debug on Windows.

Bug: crashpad:192
Change-Id: Icf3ff387050ee2becf471f4e7c3a75394b1dd436
Reviewed-on: https://chromium-review.googlesource.com/749792
Reviewed-by: Mark Mentovai <mark@chromium.org>

snapshot/mac/module_snapshot_mac.cc

@@ -187,7 +187,6 @@ std::map<std::string, std::string> ModuleSnapshotMac::AnnotationsSimpleMap()
 
 std::vector<AnnotationSnapshot> ModuleSnapshotMac::AnnotationObjects() const {
   INITIALIZATION_STATE_DCHECK_VALID(initialized_);
-  NOTREACHED();
   return {};
 }
 

snapshot/win/module_snapshot_win.cc

@@ -192,7 +192,6 @@ std::map<std::string, std::string> ModuleSnapshotWin::AnnotationsSimpleMap()
 
 std::vector<AnnotationSnapshot> ModuleSnapshotWin::AnnotationObjects() const {
   INITIALIZATION_STATE_DCHECK_VALID(initialized_);
-  NOTREACHED();
   return {};
 }