Closed
Bug 1495180
Opened 7 years ago
Closed 7 years ago
Add linking LOG to Http2PushedStream::SetConsumerStream
Categories
(Core :: Networking: HTTP, enhancement, P3)
Core
Networking: HTTP
Tracking
()
RESOLVED
FIXED
mozilla64
| Tracking | Status | |
|---|---|---|
| firefox64 | --- | fixed |
People
(Reporter: mayhemer, Assigned: mayhemer)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
|
870 bytes,
patch
|
u408661
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•7 years ago
|
||
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)
|
|
Assignee | |
Comment 2•7 years ago
|
||
OK, I think I can do it w/o this log relatively decently. No need for this patch.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
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 → ---
|
|
Assignee | |
Comment 4•7 years ago
|
||
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
|
|
||
Comment 6•7 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•