Replace makeURI with Services.io.newURI in pageInfo.js

RESOLVED FIXED in Firefox 63

Status

()

P5
normal
RESOLVED FIXED
4 months ago
4 months ago

People

(Reporter: johannh, Assigned: arshadkazmi42, Mentored)

Tracking

(Blocks: 1 bug)

unspecified
Firefox 63
Points:
---

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(1 attachment)

We should replace all occurrences of the deprecated makeURI function with Services.io.newURI here:

https://searchfox.org/mozilla-central/search?q=makeURI&case=false&regexp=false&path=pageInfo.js

The parameters should stay the same.
(Assignee)

Comment 1

4 months ago
I want to take this up
Flags: needinfo?(jhofmann)
Awesome, let me know if you need any help! Thanks!
Assignee: nobody → arshadkazmi42
Status: NEW → ASSIGNED
Flags: needinfo?(jhofmann)
(Assignee)

Comment 3

4 months ago
It needs to be changed only in that file? Or I need to check other files also for this change?
Flags: needinfo?(jhofmann)
(Assignee)

Comment 4

4 months ago
Done with the changes.

Added to phabricator and added you as reviewer

https://phabricator.services.mozilla.com/D4389
(Assignee)

Comment 5

4 months ago
Created attachment 9004362 [details]
Bug 1486313 makeURI replaced with Services.io.newURI
(Assignee)

Comment 6

4 months ago
Hey Johann,
Any updates on this? I am not sure who is going to do review for this. I have added you as reviewer for this.
Comment on attachment 9004362 [details]
Bug 1486313 makeURI replaced with Services.io.newURI

Johann Hofmann [:johannh] has approved the revision.
Attachment #9004362 - Flags: review+
(In reply to Arshad Kazmi from comment #6)
> Hey Johann,
> Any updates on this? I am not sure who is going to do review for this. I
> have added you as reviewer for this.

Hey Arshad, sorry for the delay, the last days were a bit busy for me.

Thank you for the reminder, that patch looks great!

You now have editbugs access (which means you can edit all bugs), would you like to try it out and set the checkin-needed flag yourself? :)

Thanks!
Flags: needinfo?(jhofmann)
(Assignee)

Updated

4 months ago
Flags: needinfo?(jhofmann)
Attachment #9004362 - Flags: checkin?(jhofmann)
(Assignee)

Comment 9

4 months ago
I have added 'checkin flag'

Is this the right way?
Adding '?'

Comment 10

4 months ago
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5deec145c04d
makeURI replaced with Services.io.newURI r=johannh
(Assignee)

Comment 11

4 months ago
Hey Johann,
I think code is merged now. 
What is the next step for this bug? Do I have to close it? or there is some flag for it?
arshadkazmi42, can you please remove the checkin? because this patch is still in the list on checkin-needed bugs. 
Next time you can just add the 'checkin-needed' in the keywords section. Thanks!
Flags: needinfo?(arshadkazmi42)
(Assignee)

Updated

4 months ago
Flags: needinfo?(arshadkazmi42)
Attachment #9004362 - Flags: checkin?(jhofmann)
(Assignee)

Comment 13

4 months ago
Hi Eliza,
Updated the flag. I got it now from my other bug flow, that checkin-needed needs to be added in the keywords.

So can I add checkin-needed keyword in this bug now? Or its already checkedin?
Flags: needinfo?(ebalazs)
This patch is already landed. Please see Comment 10.
Flags: needinfo?(ebalazs)
(Assignee)

Comment 15

4 months ago
Oh. yes. missed that comment. 
So what is the next step? This bug needs to be closed?
Do I have to close the bug? Or there is someone else who handles this?
Flags: needinfo?(ebalazs)

Comment 16

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5deec145c04d
Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
status-firefox63: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
No need to take any other action, as you can see in the previous comment your patch was merged in mozilla-central and it was closed as Resolved fixed.
Flags: needinfo?(ebalazs)
(Assignee)

Comment 18

4 months ago
Ok. thank your for the details :)

Updated

4 months ago
Flags: needinfo?(jhofmann)
You need to log in before you can comment on or make changes to this bug.