Click to activate message is displayed even if plugins are disabled

RESOLVED FIXED in Firefox 10

Status

()

Core
Plug-ins
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: blassey, Assigned: blassey)

Tracking

unspecified
mozilla11
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox10+ fixed)

Details

(Whiteboard: [inbound])

Attachments

(1 attachment, 2 obsolete attachments)

Created attachment 575613 [details] [diff] [review]
patch
Attachment #575613 - Flags: review?(doug.turner)
(Assignee)

Updated

6 years ago
status-firefox10: --- → affected
tracking-firefox10: --- → +
Created attachment 575614 [details] [diff] [review]
patch
Attachment #575614 - Flags: review?(doug.turner)
(Assignee)

Updated

6 years ago
Attachment #575613 - Attachment is obsolete: true
Attachment #575613 - Flags: review?(doug.turner)

Comment 2

6 years ago
Comment on attachment 575614 [details] [diff] [review]
patch

Review of attachment 575614 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/base/src/nsObjectLoadingContent.cpp
@@ +1965,5 @@
>  #ifdef ANDROID
> +   nsCOMPtr<nsIPrefBranch> prefService = do_GetService(NS_PREFSERVICE_CONTRACTID);
> +  bool pluginsDisabled = false;
> +  if (prefService) {
> +    nsresult rv = prefService->GetBoolPref("plugin.disable", &pluginsDisabled);

use Preferences::GetBool()

@@ +1969,5 @@
> +    nsresult rv = prefService->GetBoolPref("plugin.disable", &pluginsDisabled);
> +    if (NS_FAILED(rv))
> +      pluginsDisabled = false;
> +  }
> +  if (!pluginsDisabled && XRE_GetProcessType() == GeckoProcessType_Content)

Why do we need to check for the content process?
Attachment #575614 - Flags: review?(doug.turner) → review-
(In reply to Doug Turner (:dougt) from comment #2)
> Comment on attachment 575614 [details] [diff] [review] [diff] [details] [review]
> @@ +1969,5 @@
> > +    nsresult rv = prefService->GetBoolPref("plugin.disable", &pluginsDisabled);
> > +    if (NS_FAILED(rv))
> > +      pluginsDisabled = false;
> > +  }
> > +  if (!pluginsDisabled && XRE_GetProcessType() == GeckoProcessType_Content)
> 
> Why do we need to check for the content process?

Because we only use this version of click to play in the content process
Created attachment 575696 [details] [diff] [review]
patch
Attachment #575614 - Attachment is obsolete: true
Attachment #575696 - Flags: review?(doug.turner)

Comment 5

6 years ago
> Because we only use this version of click to play in the content process
Don't we want to click to play in the native client.
(In reply to Doug Turner (:dougt) from comment #5)
> > Because we only use this version of click to play in the content process
> Don't we want to click to play in the native client.

yes, that is being implemented in bug 549697. This is a bug in our tab-restarting version of click to play for e10s fennec which must be resolved for 10 so we can disable plugins and not look awful.

Comment 7

6 years ago
Comment on attachment 575696 [details] [diff] [review]
patch

Review of attachment 575696 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with a comment.  Please land this on m-c.

::: content/base/src/nsObjectLoadingContent.cpp
@@ +1963,4 @@
>  nsObjectLoadingContent::GetPluginDisabledState(const nsCString& aContentType)
>  {
>  #ifdef ANDROID
> +  if (!mozilla::Preferences::GetBool("plugin.disable", false) && 

Please add a comment here to say something like:

// If plugin.disable is false or not set,  we want to show click-to-play.
Attachment #575696 - Flags: review?(doug.turner) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/6afb61406ead
Whiteboard: [inbound]
(Assignee)

Updated

6 years ago
Attachment #575696 - Flags: approval-mozilla-aurora?

Comment 9

6 years ago
https://hg.mozilla.org/mozilla-central/rev/6afb61406ead
Assignee: nobody → blassey.bugs
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
(Assignee)

Updated

6 years ago
Blocks: 704612

Updated

6 years ago
Attachment #575696 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/f0b2f0baf91d
status-firefox10: affected → fixed
You need to log in before you can comment on or make changes to this bug.