[Extensions Toolbar] Move some popup logic to be platform-agnostic

chrome/browser/ui/views/extensions/extension_action_platform_delegate_views.h

 // Copyright 2014 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_ACTION_PLATFORM_DELEGATE_VIEWS_H_
 #define CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_ACTION_PLATFORM_DELEGATE_VIEWS_H_
 
 #include "chrome/browser/ui/extensions/extension_action_platform_delegate.h"
 #include "chrome/browser/ui/views/extensions/extension_popup.h"
 #include "content/public/browser/notification_observer.h"
@@ skipping 43 matching lines @@
   ExtensionActionPlatformDelegateViews(
       ExtensionActionViewController* controller);
   ~ExtensionActionPlatformDelegateViews() override;
 
  private:
   // ExtensionActionPlatformDelegate:
   gfx::NativeView GetPopupNativeView() override;
   bool IsMenuRunning() const override;
   void RegisterCommand() override;
   void OnDelegateSet() override;
-  bool IsShowingPopup() const override;
   void CloseActivePopup() override;
   void CloseOwnPopup() override;
-  bool ShowPopupWithUrl(
+  extensions::ExtensionViewHost* ShowPopupWithUrl(
       ExtensionActionViewController::PopupShowAction show_action,
       const GURL& popup_url,
       bool grant_tab_permissions) override;
 
   // content::NotificationObserver:
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override;
 
   // ui::AcceleratorTarget:
@@ skipping 47 matching lines @@
   base::Closure followup_context_menu_task_;
 
   content::NotificationRegistrar registrar_;
 
   base::WeakPtrFactory<ExtensionActionPlatformDelegateViews> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionActionPlatformDelegateViews);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_ACTION_PLATFORM_DELEGATE_VIEWS_H_