Closed Bug 301281 Opened 19 years ago Closed 19 years ago

Remove 10.1 code from nsInternetConfigService

Categories

(Core Graveyard :: File Handling, defect, P2)

PowerPC
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.8beta4

People

(Reporter: asaf, Assigned: asaf)

Details

Attachments

(2 files)

 
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla1.8beta4
Attached patch patchSplinter Review
Attachment #189750 - Flags: superreview?(sfraser_bugs)
Attachment #189750 - Flags: review?(jhpedemonte)
Attached patch diff -wSplinter Review
Comment on attachment 189750 [details] [diff] [review]
patch

-  // Since protocol comes in with _just_ the protocol we have to add a ':' to
the end of it or
-  // LaunchServices will be very unhappy with the CFURLRef created from it
(crashes trying to look
-  // up a handler for it with LSGetApplicationForURL, at least under 10.2.1)

Might want to still keep this comment (at least, the part not in parentheses).
Attachment #189750 - Flags: review?(jhpedemonte) → review+
Attachment #189750 - Flags: superreview?(sfraser_bugs) → superreview+
Attachment #189750 - Flags: approval1.8b4?
Attachment #189750 - Flags: approval1.8b4? → approval1.8b4+
Checking in exthandler/mac/nsInternetConfigService.cpp;
/cvsroot/mozilla/uriloader/exthandler/mac/nsInternetConfigService.cpp,v  <-- 
nsInternetConfigService.cpp
new revision: 1.39; previous revision: 1.38
done
Checking in exthandler/mac/nsInternetConfigService.h;
/cvsroot/mozilla/uriloader/exthandler/mac/nsInternetConfigService.h,v  <-- 
nsInternetConfigService.h
new revision: 1.11; previous revision: 1.10
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: