1578191 - Follow up 1370077 - don't log warning when background.persistent is set to false

Status: RESOLVED WONTFIX

(WebExtensions :: Untriaged, enhancement)

Description

[Geoffrey De Belie (Smile4ever)]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:70.0) Gecko/20100101 Firefox/70.0

Steps to reproduce:

1. Install https://addons.mozilla.org/en-US/firefox/addon/view-image/
2. Go to about:debugging

Actual results:

A warning is displayed in a yellow box: "Reading manifest: Error processing background.persistent: Event pages are not currently supported. This will run as a persistent background page.".

background.persistent false is an event page, but why would you want a warning anyway? It's something that the WebExtensions team doesn't plan to implement soon I believe.

It seems like a totally useless warning since it has no impact on the functionality.

See bug 1253565 where this warning was added.

Comment 1

[Jim Mathies [:jimm]]

we won't accept patches for this. We alrewady updated this once to make it less severe.

Comment 2

[Geoffrey De Belie (Smile4ever)]

What about replacing the word "error" with "warning"?

Comment 3

[Rob Wu [:robwu]]

( "Error" became "Warning" in bug 1495908 )

Comment 4

[Geoffrey De Belie (Smile4ever)]

Thank you.