Closed Bug 1476552 Opened Last year Closed Last year

Minor improvements to GeckoSession.WebResponseInfo


(GeckoView :: General, enhancement, P2)



(geckoview62 wontfix, firefox-esr52 wontfix, firefox-esr60 wontfix, firefox61 wontfix, firefox62 wontfix, firefox63 fixed)

Tracking Status
geckoview62 --- wontfix
firefox-esr52 --- wontfix
firefox-esr60 --- wontfix
firefox61 --- wontfix
firefox62 --- wontfix
firefox63 --- fixed


(Reporter: sebastian, Assigned: raynaldm, Mentored)


(Whiteboard: [good first bug])


(1 file)

1) WebResponseInfo should be static. Right now it has an implicit reference to the outer class and can access everything in it. However this doesn't seem to be needed. I'm trying to create a WebResponseInfo object in a test case and it's complicated because I need to create it from inside a GeckoSession object and can't create it independently.

2) It looks like only "uri" is a required value. The properties should have a @Nullable or @NonNull annotation to make Kotlin interoperability nicer.
Jim says this would be a good contributor bug. Setting Sebastian as the mentor because he best knows what needs to be fixed.
Mentor: s.kaspari
Priority: -- → P2
Whiteboard: [good first bug]
MozReview-Commit-ID: 1HWEStKaGVl

Bug 1476552 part 1 - Add @NonNull/@Nullable annotations to GeckoSession::WebResponseInfo for Kotlin interoperability

Bug 1476552 part 2 - Make GeckoSession::WebResponseInfo static

MozReview-Commit-ID: 4ce92G7i2nz
I submitted a patch for review here:
Comment on attachment 8998328 [details]
Bug 1476552 - GeckoSession::WebResponseInfo Kotlin annotations and static

Sebastian Kaspari (:sebastian) has approved the revision.
Attachment #8998328 - Flags: review+
Assignee: nobody → raynald.mirville
Great! What would be the next step from here?
Jim, Snorp: How do I land phabricator patches? I may have lost commit access since I haven't landed anything in m-c for a while. Can you do that?
Flags: needinfo?(snorp)
Flags: needinfo?(nchen)
You can click the "View in Lando" link and land from there. I took care of it for this patch.
Flags: needinfo?(snorp)
Pushed by
GeckoSession::WebResponseInfo Kotlin annotations and static r=sebastian
Flags: needinfo?(nchen)
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Thanks for your fix, Raynald! If you're looking for another bug to fix, check out Mozilla's Codetribute bug site. You can search by product and programming language.
Product: Firefox for Android → GeckoView
Target Milestone: Firefox 63 → mozilla63
You need to log in before you can comment on or make changes to this bug.