GitLab

The original change was reverted due to the NearbyPreviewTest.All test
failing. Fixed by adding a missing script include.

Revert is here:
  https://chromium-review.googlesource.com/c/chromium/src/+/2592991

Original change is here:
  https://chromium-review.googlesource.com/c/chromium/src/+/2582523

Original description:
> Moves the logic for determining the ShareType into the *Attachment
> classes. This cleans up some static casts and also allows this logic
> to be reused when generating the preview for the receive dialog page,
> coming in a subsequent CL.
>
> To avoid a dependency cycle I've moved the ShareType mojo struct
> definition into its own build target, which required importing new
> mojo gen files in a number of places.

Fixed: 1144942
Change-Id: I012f682d52505420a2ae7063f95d7c4227d5bb03
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593808Reviewed-by: James Vecore <vecore@google.com>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Alex Gough <ajgo@chromium.org>
Commit-Queue: Michael Hansen <hansenmichael@google.com>
Cr-Commit-Position: refs/heads/master@{#837199}