Closed Bug 228600 Opened 21 years ago Closed 20 years ago

[ActiveX] support view-source protocol

Categories

(Core Graveyard :: Embedding: ActiveX Wrapper, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: adamlock, Assigned: adamlock)

Details

Attachments

(1 file)

The activex control has a view source menu option which is greyed out. The
view-source functionality should be implemented. This requires the necko2.dll be
shipped with the control
Attached patch PatchSplinter Review
Patch adds necko2.dll and fixes the view-source: handler.
Comment on attachment 137483 [details] [diff] [review]
Patch

Requesting review on this patch to view-source: in the ActiveX control. The
patch fixes up the OnViewSource method and allows it to be called from the
popup menu and through the IOleCommandTarget interface.
Attachment #137483 - Flags: superreview?(blizzard)
Attachment #137483 - Flags: review?(darin)
Comment on attachment 137483 [details] [diff] [review]
Patch

sr=bzbarsky
Attachment #137483 - Flags: superreview?(blizzard) → superreview+
Comment on attachment 137483 [details] [diff] [review]
Patch

r=darin
Attachment #137483 - Flags: review?(darin) → review+
Fix is checked in
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
The View Source menu item is still greyed out for User-Agent: Mozilla/5.0 
(Windows; U; Windows NT 5.1; en-US; rv:1.7.2) Gecko/20040709.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: