Commit 9a08b8c6 authored by alexmos's avatar alexmos Committed by Commit bot

Remove unnecessary top()->document() null check from PluginListChanged.

This was introduced in r135340 in an attempt to fix a crash in issue
125969, which it ended up not fixing, but nobody reverted it
afterward.  It doesn't appear to be necessary.

BUG=466297

Review-Url: https://codereview.chromium.org/2145353004
Cr-Commit-Position: refs/heads/master@{#406005}
parent c67c6db6
...@@ -260,9 +260,6 @@ void ChromePluginPlaceholder::OnCancelledDownloadingPlugin() { ...@@ -260,9 +260,6 @@ void ChromePluginPlaceholder::OnCancelledDownloadingPlugin() {
void ChromePluginPlaceholder::PluginListChanged() { void ChromePluginPlaceholder::PluginListChanged() {
if (!GetFrame() || !plugin()) if (!GetFrame() || !plugin())
return; return;
blink::WebDocument document = GetFrame()->top()->document();
if (document.isNull())
return;
ChromeViewHostMsg_GetPluginInfo_Output output; ChromeViewHostMsg_GetPluginInfo_Output output;
std::string mime_type(GetPluginParams().mimeType.utf8()); std::string mime_type(GetPluginParams().mimeType.utf8());
......
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