Fix for plugin id is showing instead of plugin path in chrome://plugins.

chrome/browser/extensions/plugin_manager.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 "base/files/file_path.h"
 #include "base/lazy_instance.h"
 #include "base/path_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/plugin_manager.h"
 #include "chrome/browser/plugins/chrome_plugin_service_filter.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/extensions/api/plugins/plugins_handler.h"
 #include "chrome/common/extensions/manifest_handlers/mime_types_handler.h"
 #include "content/public/browser/plugin_service.h"
 #include "content/public/common/pepper_plugin_info.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/common/extension.h"
 #include "url/gurl.h"
 
 #if !defined(DISABLE_NACL)
 #include "components/nacl/common/nacl_constants.h"
 #endif
 
+const char kPDFPluginExtension[] = "pdf";
+const char kPDFPluginDescription[] = "Portable Document Format";
+
 using content::PluginService;
 
 namespace extensions {
 
 PluginManager::PluginManager(content::BrowserContext* context)
     : profile_(Profile::FromBrowserContext(context)),
       extension_registry_observer_(this) {
   extension_registry_observer_.Add(ExtensionRegistry::Get(profile_));
 }
 
@@ skipping 47 matching lines @@
   }
 #endif
 
   const MimeTypesHandler* handler = MimeTypesHandler::GetHandler(extension);
   if (handler && !handler->handler_url().empty()) {
     plugins_or_nacl_changed = true;
 
     content::WebPluginInfo info;
     info.type = content::WebPluginInfo::PLUGIN_TYPE_BROWSER_PLUGIN;
     info.name = base::UTF8ToUTF16(extension->name());
-    info.path = base::FilePath::FromUTF8Unsafe(handler->extension_id());
+    info.path = extension->path();

[raymes, 2015/01/22 03:23:40]

The path is sort of unimportant. We could display the URL of the extension if we
really wanted.

+    info.version = base::UTF8ToUTF16(extension->GetVersionForDisplay());

[raymes, 2015/01/22 03:23:40]

Since this is bundled with Chrome, let's not version it (note that there is
currently no version displayed)

 
     for (std::set<std::string>::const_iterator mime_type =
          handler->mime_type_set().begin();
          mime_type != handler->mime_type_set().end(); ++mime_type) {
       content::WebPluginMimeType mime_type_info;
       mime_type_info.mime_type = *mime_type;
       base::FilePath::StringType file_extension;
       if (net::GetPreferredExtensionForMimeType(*mime_type, &file_extension)) {
-        mime_type_info.file_extensions.push_back(
-            base::FilePath(file_extension).AsUTF8Unsafe());
+        std::string mime_file_extension =
+            base::FilePath(file_extension).AsUTF8Unsafe();
+        mime_type_info.file_extensions.push_back(mime_file_extension);
+        if (!mime_file_extension.compare(std::string(kPDFPluginExtension))) {
+          mime_type_info.description =
+              base::UTF8ToUTF16(std::string(kPDFPluginDescription));
+        }

[raymes, 2015/01/22 03:23:40]

We don't need to have a description (there currently isn't one). This isn't very
nice because it's PDF specific so let's avoid it.

       }
       info.mime_types.push_back(mime_type_info);
     }
 
     PluginService::GetInstance()->RefreshPlugins();
     PluginService::GetInstance()->RegisterInternalPlugin(info, true);
   }
 
   if (plugins_or_nacl_changed)
     PluginService::GetInstance()->PurgePluginListCache(profile_, false);
@@ skipping 111 matching lines @@
        iter != nacl_module_list_.end(); ++iter) {
     if (iter->url == url)
       return iter;
   }
   return nacl_module_list_.end();
 }
 
 #endif  // !defined(DISABLE_NACL)
 
 }  // namespace extensions