Closed Bug 903965 Opened 7 years ago Closed 7 years ago

nsWebNavigationInfo loads plugins for docShells with allowPlugins=false

Categories

(Core :: Plug-ins, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: markh, Assigned: markh)

References

Details

Attachments

(1 file)

nsWebNavigationInfo::IsTypeSupported() loads plugins (and thus the blocklist) even if the request is for a docShell with allowPlugins=false.  While later code avoids using the plugin, we should skip the load in this case.

Normally this shouldn't cause any problems, but currently this causes warnings when done in a child process - see bug 901294.  The attached patch fixes this.

(The evidence of this is the following message printed to the console in debug builds:

***********************************************************
*
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIProperties.get]"
nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "JS frame :: resource://gre/modules/FileUtils.jsm :: FileUtils_getDir :: line 63"  data: no]
************************************************************

and further instrumentation shows this is being called by the plugin manager)
Comment on attachment 788827 [details] [diff] [review]
Skip loading plugins if the docShell isn't going to use them

Georg,
  You were suggested to me as good reviewer of this given Ben's away - I hope that's OK!
Attachment #788827 - Flags: review?(georg.fritzsche)
Comment on attachment 788827 [details] [diff] [review]
Skip loading plugins if the docShell isn't going to use them

Sorry, i've no insight into docshell.
I see that bz is also on PTO, maybe try jst or smaug as docshell peers?
Attachment #788827 - Flags: review?(georg.fritzsche) → review?
Comment on attachment 788827 [details] [diff] [review]
Skip loading plugins if the docShell isn't going to use them

Olli, any chance you could take a look at this or pass it on?
Attachment #788827 - Flags: review? → review?(bugs)
Comment on attachment 788827 [details] [diff] [review]
Skip loading plugins if the docShell isn't going to use them

The code should be above IsTypeSupportedInternal.
Attachment #788827 - Flags: review?(bugs) → review-
er, let me look again.
Attachment #788827 - Flags: review- → review?(bugs)
Attachment #788827 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/7e2f89d98860
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
Assignee: nobody → mhammond
You need to log in before you can comment on or make changes to this bug.