Fix RemoteCopyBrowserTest on macOS

This is a fix after crrev.com/c/2125047 that checks the notification
icon instead of the image on macOS. Ran the browser_tests manually on a
Mac this time as they don't seem to run on the CQ.

TBR=mvanouwerkerk@chromium.org

Bug: 1065882
Change-Id: I5e9e7ec544612073258565cc5ce2fa7ac277a91a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127033Reviewed-by: Richard Knoll <knollr@chromium.org>
Commit-Queue: Richard Knoll <knollr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#754534}

chrome/browser/ui/views/sharing/remote_copy_browsertest.cc

@@ -12,6 +12,7 @@
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/metrics/histogram_tester.h"
 #include "base/test/scoped_feature_list.h"
+#include "build/build_config.h"
 #include "chrome/browser/notifications/notification_display_service_tester.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sharing/shared_clipboard/feature_flags.h"
@@ -237,8 +238,14 @@ IN_PROC_BROWSER_TEST_F(RemoteCopyBrowserTest, ImageUrl) {
                 base::ASCIIToUTF16(kDeviceName)),
             notification.title());
   ASSERT_EQ(message_center::NOTIFICATION_TYPE_IMAGE, notification.type());
+#if defined(OS_MACOSX)
+  // We show the image in the notification icon on macOS.
+  ASSERT_EQ(640, notification.icon().Width());
+  ASSERT_EQ(480, notification.icon().Height());
+#else
   ASSERT_EQ(640, notification.rich_notification_data().image.Width());
   ASSERT_EQ(480, notification.rich_notification_data().image.Height());
+#endif  // defined(OS_MACOSX)
   histograms_.ExpectUniqueSample(kStatusCodeHistogram, net::HTTP_OK, 1);
   histograms_.ExpectTotalCount(kLoadTimeHistogram, 1);
   histograms_.ExpectUniqueSample(kImageSizeBeforeDecodeHistogram, 810490, 1);