Commit e7f09286 authored by thestig's avatar thestig Committed by Commit bot

Windows: Fix the plugin metadata check to determine if Adobe Reader is secure.

BUG=412242
TEST=manual, see bug.

Review URL: https://codereview.chromium.org/549313003

Cr-Commit-Position: refs/heads/master@{#294034}
parent d6def967
...@@ -94,10 +94,13 @@ AdobeReaderPluginInfo GetReaderPlugin( ...@@ -94,10 +94,13 @@ AdobeReaderPluginInfo GetReaderPlugin(
reader_info.is_enabled = plugin_status != PluginPrefs::POLICY_DISABLED; reader_info.is_enabled = plugin_status != PluginPrefs::POLICY_DISABLED;
} }
PluginMetadata::SecurityStatus security_status = // Adobe Reader will likely always come up as "requires_authorization".
// See http://crbug.com/311655.
PluginMetadata::SecurityStatus security_stat =
plugin_metadata->GetSecurityStatus(plugins[i]); plugin_metadata->GetSecurityStatus(plugins[i]);
reader_info.is_secure = reader_info.is_secure =
security_status == PluginMetadata::SECURITY_STATUS_UP_TO_DATE; security_stat == PluginMetadata::SECURITY_STATUS_UP_TO_DATE ||
security_stat == PluginMetadata::SECURITY_STATUS_REQUIRES_AUTHORIZATION;
reader_info.plugin_info = plugins[i]; reader_info.plugin_info = plugins[i];
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment