[Reland] [Media Router Contextual Menu] Test 'checked' state of cloud services item.

chrome/browser/ui/toolbar/media_router_contextual_menu.h

Index: chrome/browser/ui/toolbar/media_router_contextual_menu.h
diff --git a/chrome/browser/ui/toolbar/media_router_contextual_menu.h b/chrome/browser/ui/toolbar/media_router_contextual_menu.h
index 234f7852597e4be46c1ef75b09ec94a17e130c61..1fdb949c495505b575e648dd3f8501d494bc1348 100644
--- a/chrome/browser/ui/toolbar/media_router_contextual_menu.h
+++ b/chrome/browser/ui/toolbar/media_router_contextual_menu.h
@@ -20,6 +20,11 @@ class MediaRouterContextualMenu : public ui::SimpleMenuModel::Delegate {
   ui::MenuModel* menu_model() { return &menu_model_; }
 
  private:
+#if defined(GOOGLE_CHROME_BUILD)
+  FRIEND_TEST_ALL_PREFIXES(MediaRouterContextualMenuUnitTest,

[msw, 2016/08/08 18:07:11]

nit: I'd rather see a static cast to a ui::SimpleMenuModel::Delegate in the
test, or making those overrides public, than adding a friend test like this.
It's not a big deal, go with your preference.

[apacible, 2016/08/09 06:21:39]

Thanks. We've been using friend tests for other Media Router tests, so keeping
as is.

+                           ToggleCloudServicesItem);
+#endif  // GOOGLE_CHROME_BUILD
+
   // ui::SimpleMenuModel::Delegate:
   bool IsCommandIdChecked(int command_id) const override;
   bool IsCommandIdEnabled(int command_id) const override;