GitLab

The ImageLoader now std::moves the ImageFamily into the callback, rather
than holding onto a reference throughout the duration of the callback.

Fixes a data race in ScheduleCreatePlatformShortcut where the
ImageFamily could not be safely handed off to the other thread (because
the ImageLoader was still holding onto a reference).

Bug: 749342
Change-Id: I1452e47c892660e5fd06e27e3a0582240a5cbcec
Reviewed-on: https://chromium-review.googlesource.com/590338Reviewed-by: Ben Wells <benwells@chromium.org>
Commit-Queue: Matt Giuca <mgiuca@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490708}