Closed Bug 121881 Opened 23 years ago Closed 18 years ago

remove viewer

Categories

(Core Graveyard :: Viewer App, defect, P2)

x86
All
defect

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: blizzard, Assigned: benjamin)

References

Details

(Whiteboard: [patch])

Attachments

(1 file)

It would be really nice to get rid of viewer if possible.  We could get rid of a
lot of the native widgets, remove a lot of glue code that iirc is only required
for viewer.  We need to figure out how to move the tests and whatnot into the
embedding tests, though, maybe through some common library?

Anyway, here's what waterson said in bug 91490:

Seriously tho, if you want kill viewer, then we'll need to:

  - rewrite the layout regression test harness
  - add the `dump [content|frames|views|style-contexts]' functionality
    to some other app.

Not rocket science, but probably takes some doin'. Oh, and this stuff would need
to work on Mac and Win32, too.
Blocks: 93467
Blocks: 108273
QA Contact: asa
Running the regression tests in the layout debugger should work now, so I don't
think we have any need for viewer anymore.  (The embedding demo app for your
platform should handle debugging needs, and the layout debugger, i.e.,
extensions/layout-debug and "mozilla -layoutdebug", should handle the stuff in
viewer's menus.)

Instructions on running regression tests in the layout debugger:
 1. Have a debug build with --enable-extensions=default,layout-debug or just:
     cd mozilla/extensions
     make makefiles DIRS=layout-debug
     cd layout-debug
     make
 2. Run ./mozilla -layoutdebug
 3. [You only need to do this once; it's then saved in your profile.]
    Go to Regression Test -> Add New List..., and pick
    mozilla/layout/html/tests/block/rtest.lst
 4. Choose that list in the Regression Test -> Run Baseline or Regression Test
    -> Run Verify menus as appropriate.

When removing viewer, I'd like to remove:
  webshell/tests/*
  layout/html/tests/.../file_list.txt (all such files, after replacing with
    rtest.lst where that's not done already)
  layout/html/tests/.../{rtest.sh,rtest.bat,runtests.sh} (all such files)

Assigning to myself and targetting to Mozilla 1.7b.  Hopefully anyone who wants
to object will do so in the next few weeks.
Assignee: blizzard → viewer-app
Summary: get rid of viewer → remove viewer
Whiteboard: [patch]
Target Milestone: --- → mozilla1.7beta
Assignee: viewer-app → dbaron
dbaron:
Can I still do print regression tests with the "new" viewer app., esp. using the
crawler mode ?
Death of viewer?  I can't wait!
We can also remove nsIDocumentViewer::CreateDocumentViewerUsing
And subsequently remove all of webshell/tests
Assignee: dbaron → benjamin
Status: NEW → ASSIGNED
Attachment #210398 - Flags: review?(dbaron)
Priority: -- → P2
Target Milestone: mozilla1.7beta → mozilla1.9alpha
r=dbaron.  Please make sure there's a followup bug on removing the now-unneeded native widget code from widget on platforms where it hasn't been migrated to the viewer directory.
Blocks: 326417
Fixed on trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
*** Bug 323451 has been marked as a duplicate of this bug. ***
Blocks: 204304
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: