Revert "WebShare: Windows Implementation"

This reverts commit 83e275fd.

Reason for revert: ShareServiceUnitTest.Multimedia started failing on
multiple Windows bots. This is likely the culprit.

Original change's description:
> WebShare: Windows Implementation
>
> Completing the implementation of navigator.share() for Windows by
> hooking in the relevant components and updating the corresponding tests.
>
> Originally committed as part of https://crrev.com/c/2443757, but
> reverted due to test issues. Patchset 1 is a cherry pick of this
> original change (reduced to the files relevant to this portion) for
> easy comparison.
>
> Bug: 1035527
> Change-Id: Ic83fa002601cce5e6c03eee9ec214f7c4c209096
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505603
> Reviewed-by: Eric Willigers <ericwilligers@chromium.org>
> Commit-Queue: Hoch Hochkeppel <mhochk@microsoft.com>
> Cr-Commit-Position: refs/heads/master@{#821941}

TBR=ericwilligers@chromium.org,mhochk@microsoft.com

Change-Id: I04727e14ee724e8526b4ee8b79ed0705c37dd8b5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1035527
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2506574Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#822084}

chrome/browser/webshare/share_service_browsertest.cc

@@ -13,47 +13,17 @@
 #include "content/public/test/browser_test_utils.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 
-#if defined(OS_WIN)
-#include "chrome/browser/webshare/win/scoped_share_operation_fake_components.h"
-#endif
-
 class ShareServiceBrowserTest : public InProcessBrowserTest {
  public:
   ShareServiceBrowserTest() {
     feature_list_.InitAndEnableFeature(features::kWebShare);
   }
 
-#if defined(OS_WIN)
-  void SetUpOnMainThread() override {
-    InProcessBrowserTest::SetUpOnMainThread();
-    if (!IsSupportedEnvironment())
-      return;
-
-    ASSERT_NO_FATAL_FAILURE(scoped_fake_components_.SetUp());
-  }
-#endif
-
- protected:
-#if defined(OS_WIN)
-  bool IsSupportedEnvironment() {
-    return webshare::ScopedShareOperationFakeComponents::
-        IsSupportedEnvironment();
-  }
-#endif
-
  private:
   base::test::ScopedFeatureList feature_list_;
-#if defined(OS_WIN)
-  webshare::ScopedShareOperationFakeComponents scoped_fake_components_;
-#endif
 };
 
 IN_PROC_BROWSER_TEST_F(ShareServiceBrowserTest, Text) {
-#if defined(OS_WIN)
-  if (!IsSupportedEnvironment())
-    return;
-#endif
-
   ASSERT_TRUE(embedded_test_server()->Start());
   ui_test_utils::NavigateToURL(
       browser(), embedded_test_server()->GetURL("/webshare/index.html"));

chrome/browser/webshare/share_service_impl.cc

@@ -5,7 +5,6 @@
 #include "chrome/browser/webshare/share_service_impl.h"
 
 #include <algorithm>
-#include <memory>
 
 #include "base/feature_list.h"
 #include "base/strings/string_piece.h"
@@ -14,10 +13,6 @@
 #include "content/public/browser/web_contents.h"
 #include "mojo/public/cpp/bindings/self_owned_receiver.h"
 
-#if defined(OS_WIN)
-#include "chrome/browser/webshare/win/share_operation.h"
-#endif
-
 // IsDangerousFilename() and IsDangerousMimeType() should be kept in sync with
 // //third_party/blink/renderer/modules/webshare/FILE_TYPES.md
 // //components/browser_ui/webshare/android/java/src/org/chromium/components/browser_ui/webshare/ShareServiceImpl.java
@@ -170,17 +165,10 @@ void ShareServiceImpl::Share(const std::string& title,
 #if defined(OS_CHROMEOS)
   sharesheet_client_.Share(title, text, share_url, std::move(files),
                            std::move(callback));
-#elif defined(OS_WIN)
-  auto share_operation = std::make_unique<webshare::ShareOperation>(
-      title, text, share_url, std::move(files), web_contents);
-  share_operation->Run(base::BindOnce(
-      [](std::unique_ptr<webshare::ShareOperation> share_operation,
-         ShareCallback callback,
-         blink::mojom::ShareError result) { std::move(callback).Run(result); },
-      std::move(share_operation), std::move(callback)));
 #else
-  NOTREACHED();
-  std::move(callback).Run(blink::mojom::ShareError::INTERNAL_ERROR);
+  // TODO(crbug.com/1035527): Add implementation for OS_WIN
+  NOTIMPLEMENTED();
+  std::move(callback).Run(blink::mojom::ShareError::OK);
 #endif
 }
 

chrome/browser/webshare/share_service_unittest.cc

@@ -29,9 +29,6 @@ using blink::mojom::ShareError;
 #include "chrome/browser/sharesheet/sharesheet_types.h"
 #include "chrome/browser/webshare/chromeos/sharesheet_client.h"
 #endif
-#if defined(OS_WIN)
-#include "chrome/browser/webshare/win/scoped_share_operation_fake_components.h"
-#endif
 
 class ShareServiceUnitTest : public ChromeRenderViewHostTestHarness {
  public:
@@ -47,22 +44,9 @@ class ShareServiceUnitTest : public ChromeRenderViewHostTestHarness {
 #if defined(OS_CHROMEOS)
     webshare::SharesheetClient::SetSharesheetCallbackForTesting(
         base::BindRepeating(&ShareServiceUnitTest::AcceptShareRequest));
-#endif
-#if defined(OS_WIN)
-    if (!IsSupportedEnvironment())
-      return;
-
-    ASSERT_NO_FATAL_FAILURE(scoped_fake_components_.SetUp());
 #endif
   }
 
-#if defined(OS_WIN)
-  bool IsSupportedEnvironment() {
-    return webshare::ScopedShareOperationFakeComponents::
-        IsSupportedEnvironment();
-  }
-#endif
-
   ShareError ShareGeneratedFileData(const std::string& extension,
                                     const std::string& content_type,
                                     unsigned file_length = 100,
@@ -139,19 +123,11 @@ class ShareServiceUnitTest : public ChromeRenderViewHostTestHarness {
   }
 #endif
 
-#if defined(OS_WIN)
-  webshare::ScopedShareOperationFakeComponents scoped_fake_components_;
-#endif
   base::test::ScopedFeatureList feature_list_;
   std::unique_ptr<ShareServiceImpl> share_service_;
 };
 
 TEST_F(ShareServiceUnitTest, FileCount) {
-#if defined(OS_WIN)
-  if (!IsSupportedEnvironment())
-    return;
-#endif
-
   EXPECT_EQ(ShareError::OK, ShareGeneratedFileData(".txt", "text/plain", 1234,
                                                    kMaxSharedFileCount));
   EXPECT_EQ(ShareError::PERMISSION_DENIED,
@@ -196,11 +172,6 @@ TEST_F(ShareServiceUnitTest, Multimedia) {
 }
 
 TEST_F(ShareServiceUnitTest, PortableDocumentFormat) {
-#if defined(OS_WIN)
-  if (!IsSupportedEnvironment())
-    return;
-#endif
-
   // TODO(crbug.com/1006055): Support sharing of pdf files.
   // The URL will be checked using Safe Browsing.
   EXPECT_EQ(ShareError::PERMISSION_DENIED,

chrome/test/BUILD.gn

@@ -347,7 +347,6 @@ static_library("test_support") {
       "//components/crash/core/app",
       "//third_party/wtl",
     ]
-    libs = [ "runtimeobject.lib" ]
   }
 
   if (is_chromeos) {