Unnecessary 'translate to English' option is displayed in context menu in PDF.

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 1149 matching lines @@
       if (embedder_web_contents_->GetController().GetVisibleEntry() == NULL)
         return false;
       // Disabled if no browser is associated (e.g. desktop notifications).
       if (chrome::FindBrowserWithWebContents(embedder_web_contents_) == NULL)
         return false;
       return true;
 
     case IDC_CONTENT_CONTEXT_TRANSLATE: {
       ChromeTranslateClient* chrome_translate_client =
           ChromeTranslateClient::FromWebContents(embedder_web_contents_);
-      if (!chrome_translate_client)
+      // If no |chrome_translate_client| attached with this webcontent or we're

[lazyboy, 2015/09/29 04:41:58]

s/webcontent/WebContents

[Deepak, 2015/09/29 07:57:23]

Done.

+      // viewing in a MimeHandlerViewGuest translate will be disabled.
+      if (!chrome_translate_client ||
+          guest_view::GuestViewBase::IsGuest(source_web_contents_)) {

[lazyboy, 2015/09/29 04:41:58]

GuestViewBase seems too broad, to narrow it to mime handler views only, you'd
need !!MimeHandlerViewGuest::FromWebContents(..)

[Deepak, 2015/09/29 07:57:23]

Done.

         return false;
+      }
       std::string original_lang =
           chrome_translate_client->GetLanguageState().original_language();
       std::string target_lang = g_browser_process->GetApplicationLocale();
       target_lang =
           translate::TranslateDownloadManager::GetLanguageCode(target_lang);
       // Note that we intentionally enable the menu even if the original and
       // target languages are identical.  This is to give a way to user to
       // translate a page that might contains text fragments in a different
       // language.
       return ((params_.edit_flags & WebContextMenuData::CanTranslate) != 0) &&
@@ skipping 777 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);
 }