Closed Bug 228600 Opened 20 years ago Closed 20 years ago

[ActiveX] support view-source protocol

Categories

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

Product:
Core Graveyard
Component:
Embedding: ActiveX Wrapper
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: adamlock, Assigned: adamlock)

Details

Attachments

(1 file)

Patch
5.87 KB, patch
darin.moz
: review+
bzbarsky
: superreview+
Details | Diff | Splinter Review 
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 Patch — — Splinter 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.