Closed Bug 1495180 Opened Last year Closed Last year

Add linking LOG to Http2PushedStream::SetConsumerStream

Categories

(Core :: Networking: HTTP, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: mayhemer, Assigned: mayhemer)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

No description provided.
Attached patch 1495180.patchSplinter Review
If you want logan work also for logs w/o this line, it's doable too, but would be kinda hack...
Attachment #9013072 - Flags: review?(hurley)
OK, I think I can do it w/o this log relatively decently.  No need for this patch.
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → WONTFIX
Comment on attachment 9013072 [details] [diff] [review]
1495180.patch

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

So I know this isn't strictly needed for logan, but this could be useful for quick visual scans of logs. Honza, I'd like it if you could go ahead and land this.
Attachment #9013072 - Flags: review?(hurley) → review+
Status: RESOLVED → REOPENED
Flags: needinfo?(honzab.moz)
Resolution: WONTFIX → ---
Yeah, I'll go ahead.

(No need for a try push.)
Status: REOPENED → ASSIGNED
Flags: needinfo?(honzab.moz)
Keywords: checkin-needed
Priority: -- → P3
Whiteboard: [necko-triaged]
Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/36b041b6b55a
Add linking LOG to Http2PushedStream::SetConsumerStream. r=nwgh
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/36b041b6b55a
Status: ASSIGNED → RESOLVED
Closed: Last yearLast year
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.