Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(622)

Unified Diff: chrome/browser/ui/webui/media_router/media_router_dialog_controller_impl.h

Issue 2410553002: Show Media Router toolbar icon ephemerally for MR dialogs (Closed)
Patch Set: Deleted the first patch set on accident, responded to Mark's comments inline below Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/media_router/media_router_dialog_controller_impl.h
diff --git a/chrome/browser/ui/webui/media_router/media_router_dialog_controller_impl.h b/chrome/browser/ui/webui/media_router/media_router_dialog_controller_impl.h
index 1c19a127c3a7735939edf9aeae931cdd539b64c1..f5bc5d4b956805d7bb686a87692cecf9e99a7df4 100644
--- a/chrome/browser/ui/webui/media_router/media_router_dialog_controller_impl.h
+++ b/chrome/browser/ui/webui/media_router/media_router_dialog_controller_impl.h
@@ -15,6 +15,7 @@
FORWARD_DECLARE_TEST(MediaRouterActionUnitTest, IconPressedState);
class MediaRouterAction;
+class MediaRouterActionController;
namespace media_router {
@@ -75,6 +76,10 @@ class MediaRouterDialogControllerImpl :
// when the overflow menu is opened and destroyed when the menu is closed.
base::WeakPtr<MediaRouterAction> action_;
+ // |action_controller_| is responsible for showing and hiding the toolbar
+ // action. It's owned by MediaRouterUIService, which outlives |this|.
+ MediaRouterActionController* action_controller_;
+
base::WeakPtrFactory<MediaRouterDialogControllerImpl> weak_ptr_factory_;
DISALLOW_COPY_AND_ASSIGN(MediaRouterDialogControllerImpl);

Powered by Google App Engine
This is Rietveld 408576698