[Media Router] Add experiment control logic.

chrome/browser/renderer_context_menu/render_view_context_menu.cc

 // 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.
 
 #include "chrome/browser/renderer_context_menu/render_view_context_menu.h"
 
 #include <algorithm>
 #include <set>
 #include <utility>
 
@@ skipping 105 matching lines @@
 #if defined(ENABLE_PRINTING)
 #include "chrome/browser/printing/print_view_manager_common.h"
 #include "components/printing/common/print_messages.h"
 
 #if defined(ENABLE_PRINT_PREVIEW)
 #include "chrome/browser/printing/print_preview_context_menu_observer.h"
 #include "chrome/browser/printing/print_preview_dialog_controller.h"
 #endif  // defined(ENABLE_PRINT_PREVIEW)
 #endif  // defined(ENABLE_PRINTING)
 
+#if defined(ENABLE_MEDIA_ROUTER)
+#include "chrome/browser/media/router/media_router_feature.h"
+#endif  // defined(ENABLE_MEDIA_ROUTER)
+
 using base::UserMetricsAction;
 using blink::WebContextMenuData;
 using blink::WebMediaPlayerAction;
 using blink::WebPluginAction;
 using blink::WebString;
 using blink::WebURL;
 using content::BrowserContext;
 using content::ChildProcessSecurityPolicy;
 using content::DownloadManager;
 using content::DownloadUrlParameters;
@@ skipping 811 matching lines @@
 
 void RenderViewContextMenu::AppendPrintItem() {
   if (GetPrefs(browser_context_)->GetBoolean(prefs::kPrintingEnabled) &&
       (params_.media_type == WebContextMenuData::MediaTypeNone ||
        params_.media_flags & WebContextMenuData::MediaCanPrint)) {
     menu_model_.AddItemWithStringId(IDC_PRINT, IDS_CONTENT_CONTEXT_PRINT);
   }
 }
 
 void RenderViewContextMenu::AppendMediaRouterItem() {
-  if (switches::MediaRouterEnabled() && !browser_context_->IsOffTheRecord())
+#if defined(ENABLE_MEDIA_ROUTER)
+  if (media_router::MediaRouterEnabled() &&
+      !browser_context_->IsOffTheRecord()) {
     menu_model_.AddItemWithStringId(IDC_ROUTE_MEDIA,
                                     IDS_MEDIA_ROUTER_MENU_ITEM_TITLE);
+  }
+#endif  // defined(ENABLE_MEDIA_ROUTER)
 }
 
 void RenderViewContextMenu::AppendRotationItems() {
   if (params_.media_flags & WebContextMenuData::MediaCanRotate) {
     menu_model_.AddSeparator(ui::NORMAL_SEPARATOR);
     menu_model_.AddItemWithStringId(IDC_CONTENT_CONTEXT_ROTATECW,
                                     IDS_CONTENT_CONTEXT_ROTATECW);
     menu_model_.AddItemWithStringId(IDC_CONTENT_CONTEXT_ROTATECCW,
                                     IDS_CONTENT_CONTEXT_ROTATECCW);
   }
@@ skipping 438 matching lines @@
       return true;
 #endif
 
     case IDC_SPELLCHECK_MENU:
     case IDC_CONTENT_CONTEXT_OPENLINKWITH:
     case IDC_CONTENT_CONTEXT_PROTOCOL_HANDLER_SETTINGS:
     case IDC_CONTENT_CONTEXT_FORCESAVEPASSWORD:
       return true;
 
     case IDC_ROUTE_MEDIA: {
-      DCHECK(switches::MediaRouterEnabled());
+#if defined(ENABLE_MEDIA_ROUTER)
+      DCHECK(media_router::MediaRouterEnabled());
 
       // Disable the command if there is an active modal dialog.
       Browser* browser =
           chrome::FindBrowserWithWebContents(source_web_contents_);
       if (!browser || browser->profile()->IsOffTheRecord())
         return false;
 
       WebContents* web_contents =
           browser->tab_strip_model()->GetActiveWebContents();
       if (!web_contents)
         return false;
 
       const web_modal::WebContentsModalDialogManager* manager =
           web_modal::WebContentsModalDialogManager::FromWebContents(
               web_contents);
       return !manager || !manager->IsDialogActive();
+#else
+      return false;

[msw, 2015/10/23 20:18:42]

nit: add NOTREACHED(); in this block?

[imcheng, 2015/10/23 21:39:26]

Done.

+#endif  // defined(ENABLE_MEDIA_ROUTER)
     }
 
     default:
       NOTREACHED();
       return false;
   }
 }
 
 bool RenderViewContextMenu::IsCommandIdChecked(int id) const {
   if (RenderViewContextMenuBase::IsCommandIdChecked(id))
@@ skipping 261 matching lines @@
 
       printing::StartPrint(
           source_web_contents_,
           GetPrefs(browser_context_)->GetBoolean(prefs::kPrintPreviewDisabled),
           !params_.selection_text.empty());
 #endif  // ENABLE_PRINTING
       break;
     }
 
     case IDC_ROUTE_MEDIA: {
-      DCHECK(switches::MediaRouterEnabled());
+#if defined(ENABLE_MEDIA_ROUTER)
+      DCHECK(media_router::MediaRouterEnabled());
 
       Browser* browser =
           chrome::FindBrowserWithWebContents(source_web_contents_);
       DCHECK(browser && !browser->profile()->IsOffTheRecord());
 
       media_router::MediaRouterDialogController* dialog_controller =
           media_router::MediaRouterDialogController::GetOrCreateForWebContents(
               source_web_contents_);
 
       if (!dialog_controller)
         return;
 
       dialog_controller->ShowMediaRouterDialog();
+#endif  // defined(ENABLE_MEDIA_ROUTER)

[msw, 2015/10/23 20:18:43]

nit: add an else block with NOTREACHED();?

[imcheng, 2015/10/23 21:39:26]

Done.

       break;
     }
 
     case IDC_VIEW_SOURCE:
       embedder_web_contents_->ViewSource();
       break;
 
     case IDC_CONTENT_CONTEXT_INSPECTELEMENT:
       Inspect(params_.x, params_.y);
       break;
@@ skipping 255 matching lines @@
   source_web_contents_->GetRenderViewHost()->
       ExecuteMediaPlayerActionAtLocation(location, action);
 }
 
 void RenderViewContextMenu::PluginActionAt(
     const gfx::Point& location,
     const WebPluginAction& action) {
   source_web_contents_->GetRenderViewHost()->
       ExecutePluginActionAtLocation(location, action);
 }