Plugin Placeholders: Fix "blank first menu item" bug when no mime type.

chrome/renderer/plugins/chrome_plugin_placeholder.cc

 // Copyright 2013 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/renderer/plugins/chrome_plugin_placeholder.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/common/prerender_messages.h"
 #include "chrome/common/render_messages.h"
@@ skipping 313 matching lines @@
   context_menu_request_id_ = 0;
 }
 
 void ChromePluginPlaceholder::ShowContextMenu(const WebMouseEvent& event) {
 #if !defined(OS_ANDROID)  // The context menu is not applicable on Android.
   if (context_menu_request_id_)
     return;  // Don't allow nested context menu requests.
 
   content::ContextMenuParams params;
 
-  content::MenuItem name_item;
-  name_item.label = title_;
-  params.custom_items.push_back(name_item);
+  if (!title_.empty()) {
+    content::MenuItem name_item;
+    name_item.label = title_;
+    params.custom_items.push_back(name_item);
 
-  content::MenuItem separator_item;
-  separator_item.type = content::MenuItem::SEPARATOR;
-  params.custom_items.push_back(separator_item);
+    content::MenuItem separator_item;
+    separator_item.type = content::MenuItem::SEPARATOR;
+    params.custom_items.push_back(separator_item);
+  }
 
   if (!GetPluginInfo().path.value().empty()) {
     content::MenuItem run_item;
     run_item.action = chrome::MENU_COMMAND_PLUGIN_RUN;
     // Disable this menu item if the plugin is blocked by policy.
     run_item.enabled = LoadingAllowed();
     run_item.label = l10n_util::GetStringUTF16(IDS_CONTENT_CONTEXT_PLUGIN_RUN);
     params.custom_items.push_back(run_item);
   }
 
@@ skipping 21 matching lines @@
   v8::Handle<v8::Object> global = context->Global();
   global->Set(gin::StringToV8(isolate, "plugin"),
               gin::CreateHandle(isolate, this).ToV8());
 }
 
 gin::ObjectTemplateBuilder ChromePluginPlaceholder::GetObjectTemplateBuilder(
     v8::Isolate* isolate) {
   return LoadablePluginPlaceholder::GetObjectTemplateBuilder(isolate).SetMethod(
       "openAboutPlugins", &ChromePluginPlaceholder::OpenAboutPluginsCallback);
 }