1181351 - [B2G] Some service worker info properties are empty in the list of service workers shown by about:sw for b2g (follow up bug 1174140)

Status: RESOLVED WONTFIX

(Core :: DOM: Service Workers, defect)

Description

[José Antonio Olivera Ortega [:jaoo]]

Follow up bug 1174140 for B2G.

Comment 1

[José Antonio Olivera Ortega [:jaoo]]

Attachment: WIP

In B2G about:sw is chrome process and there is no way to make it content process (like we did for bug 1174140 in desktop). about:sw panel in B2G should work well even being chrome process. The issue here is that some properties don't show up so after following the code I see that the issue is gone by commenting a piece of code. Andrea, could we remove that block?, I don't see the reason of having it.

Comment 2

[Noemí Freire (:noemi)]

Hi,

Checking Bug 1174140 just landed in m-c, we have realized that a new property called "Push Endpoint" has been added to about:sw properties list so we will dig into it and try to add it in about:sw in b2g within this bug. Thanks!

Comment 3

[Noemí Freire (:noemi)]

Hi Catalin,

As commented yesterday during the bi-weekly sync meeting, since SWM refactor will take a while, would you mind to take a look at this bug to see if any kind of workaround could be applied here in the meanwhile?. Thanks!

Comment 4

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Comment on attachment 8630768 [details] [diff] [review]
WIP

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

Sorry this took time. The problem with allowing this to go through is that the active worker ServiceWorkerInfo's URL will change under any actual WorkerPrivate instance running for the ServiceWorkerInfo.

Comment 5

[José Antonio Olivera Ortega [:jaoo]]

(In reply to Nikhil Marathe [:nsm] (please needinfo?) from comment #4)
> Comment on attachment 8630768 [details] [diff] [review]
> WIP
> 
> Review of attachment 8630768 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Sorry this took time. The problem with allowing this to go through is that
> the active worker ServiceWorkerInfo's URL will change under any actual
> WorkerPrivate instance running for the ServiceWorkerInfo.

Thanks for your feedback. I've been trying to figure out another different fix for this but honestly I'm out of ideas. Could you share what you might have in mind for a fix here please? Maybe It's not worth to work on this until bug 1182117 get fixed? Thanks.

Comment 6

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

If you want to update the active URL, then yes, you'll have to wait. This state mirroring was always going to be a stop-gap at the best.