make nsIContentViewer scriptable

VERIFIED FIXED

Status

P3
normal
VERIFIED FIXED
19 years ago
14 years ago

People

(Reporter: jag+mozbugs, Assigned: jag+mozbugs)

Tracking

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

19 years ago
To further our efforts to get rid of appCore, we'll need to have access to the
textZoom property of nsIMarkupDocumentViewer from both Editor and Browser.

This can be gotten at through the documentViewer property of a DocShell, except
right now it's not scriptable.

See here the bug with a request to make it scriptable.

Not sure if XP Apps is the right component, please re-assign if not.

I'll see if I can do this (more hands-on learning!) but if anyone else wants to
do this, please do :-)
(Assignee)

Updated

19 years ago
Blocks: 54964
(Assignee)

Comment 1

19 years ago
*cough* nsIContentViewer *cough* contentViewer property *cough*
Summary: make nsIDocumentViewer scriptable → make nsIContentViewer scriptable
(Assignee)

Updated

19 years ago
Blocks: 10893
(Assignee)

Comment 2

19 years ago
Oooh, and let's not forget about mail/news. HTML e-mails will also benefit from
this zoom fu (bug 10893) :-)
(Assignee)

Updated

19 years ago
Blocks: 46200
(Assignee)

Comment 3

19 years ago
According to bryner, the only thing for nsIContentViewer which can't be
scriptable is:

NS_IMETHOD Init(nsIWidget* aParentWidget,
                nsIDeviceContext* aDeviceContext,
                const nsRect& aBounds)=0;

because of the nsIDeviceContext.

Currently I'll only need to get the contentViewer from a docShell and most of
the time just QI it to another interface, so that shouldn't be a problem I
think.
(Assignee)

Comment 4

19 years ago
Taking this bug... It works here! I'll attach a patch tomorrow.
Assignee: don → disttsc
(Assignee)

Comment 5

19 years ago
Done! See bug 46200 for the patch to this and its little friends.
Status: NEW → ASSIGNED
(Assignee)

Comment 6

18 years ago
And checked in (a while back)
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
rubberstamp vrfy.
Status: RESOLVED → VERIFIED
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.