Closed Bug 999443 Opened 10 years ago Closed 10 years ago

IE Tab Plug-in is not automatically activated


(Core Graveyard :: Plug-ins, defect, P2)

Windows 7


(firefox30+ verified, firefox31 verified)

Tracking Status
firefox30 + verified
firefox31 --- verified


(Reporter: pauly, Assigned: benjamin)



(Whiteboard: p=2 s=it-31c-30a-29b.3 [qa!])


(2 files)

1. Install
2. Restart FF
3. Open Addons Manager/Plugins

Actual result: IE Tab Plug-in is set to 'Ask to Activate'
Expected result: IE Tab Plug-in is set to 'Always Activate'

Path: C:\Users\paul.silaghi\AppData\Roaming\Mozilla\Firefox\Profiles\jhkta1cy.21\extensions\{1BC9BA34-1EED-42ca-A505-6D2F1A935BBB}\plugins\npietab2.dll
31.0a1 (2014-04-21), Win 7 x64
Flags: firefox-backlog?
Priority: -- → P2
Assignee: nobody → benjamin
Whiteboard: p=2
Flags: firefox-backlog? → firefox-backlog+
QA Whiteboard: [qa+]
Whiteboard: p=2 → p=2
Whiteboard: p=2 → p=2 s=it-31c-30a-29b.3 [qa+]
Attachment #8410409 - Flags: review?(georg.fritzsche)
Attachment #8410411 - Flags: review?(georg.fritzsche)
Comment on attachment 8410409 [details] [diff] [review]

Review of attachment 8410409 [details] [diff] [review]:

r+ with comments addressed.

::: dom/plugins/test/unit/head_plugins.js
@@ +12,5 @@
>  const gIsLinux = (";1" in Cc) ||
>    (";1" in Cc);
>  const gDirSvc = Cc[";1"].getService(Ci.nsIProperties);
> +const kPluginNames = [

k-prefixing for constants seems unusual in test js - use upper- or plain mixed-case without prefix?

@@ +29,5 @@
>    var pluginEnum = gDirSvc.get("APluginsDL", Ci.nsISimpleEnumerator);
>    while (pluginEnum.hasMoreElements()) {
>      let dir = pluginEnum.getNext().QueryInterface(Ci.nsILocalFile);
> +    let names = secondplugin ? kSecondNames : kPluginNames;
> +    for (let name of names) {

Can't this just use |get_platform_specific_plugin_name()| plus handling for the second plugin?
Attachment #8410409 - Flags: review?(georg.fritzsche) → review+
Comment on attachment 8410411 [details] [diff] [review]

Review of attachment 8410411 [details] [diff] [review]:

Ouch, that was quite an oversight.

::: dom/plugins/base/nsPluginHost.cpp
@@ +1633,4 @@
>  {
> +  for (uint32_t i = 0; i < extensionDirs.Length(); ++i) {
> +    bool contains;
> +    if (NS_FAILED(extensionDirs[i]->Contains(pluginFile, true, &contains)) || !contains) {

So we now automatically activate plugins no matter where they are in extension dirs, not just <extension-dir>/plugins/.
So far i can't think of a problem with this though.

@@ +1660,5 @@
>    }
> +  bool more;
> +  nsCOMPtr<nsISupports> next;
> +  nsCOMPtr<nsIFile> file;

next & file don't need to be in this scope, can just move them where they are actually needed.
Attachment #8410411 - Flags: review?(georg.fritzsche) → review+
QA Contact: florin.mezei
Comment on attachment 8410411 [details] [diff] [review]

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 982101
User impact if declined: feature bug 982101 doesn't work, blocks bug 992995
Testing completed (on m-c, etc.): just landed, automated tests and we'll have manual QA verify also
Risk to taking this patch (and alternatives if risky): fairly low risk
String or IDL/UUID changes made by this patch: None
Attachment #8410411 - Flags: approval-mozilla-aurora?
QA Contact: florin.mezei → paul.silaghi
Hi Paul, confirming if this resolved bug can be verified before the end of the desktop iteration on Monday April 28?
Flags: needinfo?(paul.silaghi)
IE Tab Plug-in is now set to 'Always Activate' after installation.
Verified fixed 31.0a1 (2014-04-24), Win 7 x64.
The plugin is N/A for Mac and Linux.
Flags: needinfo?(paul.silaghi)
Whiteboard: p=2 s=it-31c-30a-29b.3 [qa+] → p=2 s=it-31c-30a-29b.3 [qa!]
I noticed 2 other issues with this addon (both reproducible on builds before landing bug 982101):

1. With the plugin 'Always activate', right click/open in IE Tab on a link, block the plugin from the doorhanger and then reload => plugin crash

2. Setting IT Tab to 'Ask to Activate' doesn't really block the plugin (right click/open in IE Tab on a link works just fine). Allowing from the doorhanger and then re-blocking finally blocks the plugin (the CTP overlay is displayed)

Let me know is these worth to be filed separately.
Flags: needinfo?(benjamin)
No, that's part of the addon and not something to track here.
Flags: needinfo?(benjamin)
Attachment #8410411 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Verified as fixed using the following environment:

FF 30 Aurora
Build Id:20140428004000
OS: Win7 x64
QA Whiteboard: [qa+] → [qa!]
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.