Remove conditional compile statements in ShareServiceImpl (and test).

chrome/browser/webshare/share_service_impl.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/webshare/share_service_impl.h"
 
 #include <algorithm>
 #include <functional>
 #include <utility>
 
@@ skipping 105 matching lines @@
       start_index_to_copy, url_template.size() - start_index_to_copy));
 
   *url_template_filled = JoinString(split_template);
   return true;
 }
 
 void ShareServiceImpl::ShowPickerDialog(
     const std::vector<std::pair<base::string16, GURL>>& targets,
     const base::Callback<void(base::Optional<std::string>)>& callback) {
 // TODO(mgiuca): Get the browser window as |parent_window|.
-#if defined(OS_LINUX) || defined(OS_WIN)
   chrome::ShowWebShareTargetPickerDialog(nullptr /* parent_window */, targets,
                                          callback);
-#else
-  callback.Run(base::nullopt);
-#endif
 }
 
 Browser* ShareServiceImpl::GetBrowser() {
-// TODO(constantina): Prevent this code from being run/compiled in android.
-#if defined(OS_LINUX) || defined(OS_WIN)
   return BrowserList::GetInstance()->GetLastActive();
-#else
-  return nullptr;
-#endif
 }
 
 void ShareServiceImpl::OpenTargetURL(const GURL& target_url) {
-// TODO(constantina): Prevent this code from being run/compiled in android.
-#if defined(OS_LINUX) || defined(OS_WIN)
   Browser* browser = GetBrowser();
   chrome::AddTabAt(browser, target_url,
                    browser->tab_strip_model()->active_index() + 1, true);
-#endif
 }
 
 std::string ShareServiceImpl::GetTargetTemplate(
     const std::string& target_url,
     const base::DictionaryValue& share_targets) {
   const base::DictionaryValue* share_target_info_dict = nullptr;
   share_targets.GetDictionaryWithoutPathExpansion(target_url,
                                                   &share_target_info_dict);
 
   std::string url_template;
   share_target_info_dict->GetString("url_template", &url_template);
   return url_template;
 }
 
 PrefService* ShareServiceImpl::GetPrefService() {
-// TODO(constantina): Prevent this code from being run/compiled in android.
-#if defined(OS_LINUX) || defined(OS_WIN)
   return GetBrowser()->profile()->GetPrefs();
-#else
-  return nullptr;
-#endif
 }
 
 blink::mojom::EngagementLevel ShareServiceImpl::GetEngagementLevel(
     const GURL& url) {
-// TODO(constantina): Prevent this code from being run/compiled in android.
-#if defined(OS_LINUX) || defined(OS_WIN)
   SiteEngagementService* site_engagement_service =
       SiteEngagementService::Get(GetBrowser()->profile());
   return site_engagement_service->GetEngagementLevel(url);
-#else
-  return blink::mojom::EngagementLevel::NONE;
-#endif
 }
 
 // static
 std::vector<std::pair<base::string16, GURL>>
 ShareServiceImpl::GetTargetsWithSufficientEngagement(
     const base::DictionaryValue& share_targets) {
   constexpr blink::mojom::EngagementLevel kMinimumEngagementLevel =
       blink::mojom::EngagementLevel::LOW;
 
   std::vector<std::pair<base::string16, GURL>> sufficiently_engaged_targets;
@@ skipping 16 matching lines @@
 
   return sufficiently_engaged_targets;
 }
 
 void ShareServiceImpl::Share(const std::string& title,
                              const std::string& text,
                              const GURL& share_url,
                              const ShareCallback& callback) {
   std::unique_ptr<base::DictionaryValue> share_targets;
 
-// TODO(constantina): Prevent this code from being run/compiled in android.
-#if defined(OS_LINUX) || defined(OS_WIN)
   share_targets = GetPrefService()
                       ->GetDictionary(prefs::kWebShareVisitedTargets)
                       ->CreateDeepCopy();
-#else
-  return;
-#endif
 
   std::vector<std::pair<base::string16, GURL>> sufficiently_engaged_targets =
       GetTargetsWithSufficientEngagement(*share_targets);
 
   ShowPickerDialog(
       sufficiently_engaged_targets,
       base::Bind(&ShareServiceImpl::OnPickerClosed, base::Unretained(this),
                  base::Passed(&share_targets), title, text, share_url,
                  callback));
 }
@@ skipping 29 matching lines @@
   const GURL target(url_base.as_string() + url_template_filled);
   if (!target.is_valid()) {
     callback.Run(base::Optional<std::string>(
         "Error: url of share target is not a valid url."));
     return;
   }
   OpenTargetURL(target);
 
   callback.Run(base::nullopt);
 }