Open Bug 1530209 Opened 1 year ago Updated 11 months ago

Remove context argument from remaining methods

Categories

(Core :: DOM: Core & HTML, enhancement, P3)

65 Branch
enhancement

Tracking

()

People

(Reporter: jkt, Assigned: kanurag94, Mentored)

References

Details

(Keywords: good-first-bug)

Methods include:

DoOnStartRequest
nsIndexedToHTML::DoOnStartRequest
DispatchContent
InterceptStreamListener::OnStatus
InterceptStreamListener::OnProgress
InterceptStreamListener::DoOnDataAvailable
OnImageDataAvailable
nsDeflateConverter::PushAvailableData
do_OnDataAvailable
FireListenerNotifications
nsDirIndexParser::ProcessData
SendToListener

The intent is to remove mContext once these are resolved also.

Also:

imgRequest::OnDataAvailable
OnImageDataComplete
PushAvailableData
OnDownloadComplete
nsDirIndexParser::ProcessData
nsIndexedToHTML::SendToListener
FireListenerNotifications
OnStopRequestInternal

Priority: -- → P3
Component: DOM → DOM: Core & HTML

Also nsInputStreamPump::AsyncRead I guess?

Oh, I see now that someone already filed bug 1528649 for that.

I did have a patch that resolved this however I lost the rebasing battle, it's easy enough to resolve for a good first bug though.

Mentor: jkt
Keywords: good-first-bug
OS: Unspecified → All
Hardware: Unspecified → All

Hi @jkt,
I would like to work on this bug

Hi @jkt is anyone working on this? Can i work on it?

Flags: needinfo?(jkt)

Sorry Anurag I missed your message, could you confirm if you are working on this?

Flags: needinfo?(jkt) → needinfo?(kanurag94)

Hi Jonathan, Sorry for late reply, didn't log in in last couple of days.
I will be working on this starting today.
Thanks!

Flags: needinfo?(kanurag94)
Assignee: nobody → kanurag94
You need to log in before you can comment on or make changes to this bug.