Remove unneeded virtual functions in ExtensionAppShimHandler

These get in the way of refactoring, and aren't used anyway.

TBR=raymes

Bug: 982024
Change-Id: Ifa87a5e7af30a8529bf531409b49307f499a64fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761714Reviewed-by: ccameron <ccameron@chromium.org>
Commit-Queue: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689124}

chrome/browser/apps/app_shim/extension_app_shim_handler_mac.h

@@ -93,8 +93,8 @@ class ExtensionAppShimHandler : public AppShimHandler,
   ~ExtensionAppShimHandler() override;
 
   // Get the host corresponding to a profile and app id, or null if there is
-  // none. Virtual for tests.
-  virtual AppShimHost* FindHost(Profile* profile, const std::string& app_id);
+  // none.
+  AppShimHost* FindHost(Profile* profile, const std::string& app_id);
 
   // Return the host corresponding to |profile| and |app_id|, if one exists.
   // If one does not exist, create one, and launch the app shim (so that the
@@ -179,7 +179,6 @@ class ExtensionAppShimHandler : public AppShimHandler,
 
   // Exposed for testing.
   void set_delegate(Delegate* delegate);
-  HostMap& hosts() { return hosts_; }
   content::NotificationRegistrar& registrar() { return registrar_; }
 
  private:

chrome/browser/apps/app_shim/extension_app_shim_handler_mac_unittest.cc

@@ -152,11 +152,6 @@ class TestingExtensionAppShimHandler : public ExtensionAppShimHandler {
     ExtensionAppShimHandler::OnShimFocus(host, focus_type, files);
   }
 
-  AppShimHost* FindHost(Profile* profile, const std::string& app_id) {
-    HostMap::const_iterator it = hosts().find(make_pair(profile, app_id));
-    return it == hosts().end() ? NULL : it->second;
-  }
-
   void SetAcceptablyCodeSigned(bool is_acceptable_code_signed) {
     is_acceptably_code_signed_ = is_acceptable_code_signed;
   }