TestProtocols includes unneeded nsICookieService.h header

RESOLVED FIXED

Status

()

Core
Networking
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: bbaetz, Assigned: bbaetz)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

1.24 KB, patch
Brian Ryner (not reading)
: review+
dbaron
: approval+
Details | Diff | Splinter Review
(Assignee)

Description

17 years ago
myotonic is red becauseit build w/o extentions, but TestProtocols includes
cookies headers. The code which uses it if #ifdef'd out. I'll attach a patch
which just removes it.
(Assignee)

Comment 1

17 years ago
Created attachment 65724 [details] [diff] [review]
patch
Comment on attachment 65724 [details] [diff] [review]
patch

r=bryner
Attachment #65724 - Flags: review+
Comment on attachment 65724 [details] [diff] [review]
patch

sr=jst
Attachment #65724 - Flags: superreview+
Comment on attachment 65724 [details] [diff] [review]
patch

a=dbaron for 0.9.8 (I won't note a + to the bug since we don't *need* to get it
in, so it won't cause you to get nag mail if you don't get it in quickly :-)
Attachment #65724 - Flags: approval+
(Assignee)

Comment 5

17 years ago
Checked in. It got further now, at least.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.