Closed Bug 120891 Opened 24 years ago Closed 24 years ago

TestProtocols includes unneeded nsICookieService.h header

Categories

(Core :: Networking, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bbaetz, Assigned: bbaetz)

Details

Attachments

(1 file)

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.
Attached patch patchSplinter Review
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+
Checked in. It got further now, at least.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: