Closed Bug 51500 Opened 24 years ago Closed 23 years ago

The page draws incorrectly - maybe frame width problem

Categories

(Core :: Layout, defect, P3)

Product:
Core
Component:
Layout
Platform:
x86
Linux
Type:
defect

Tracking

()

Status:
RESOLVED WORKSFORME
Milestone:
mozilla1.0

People

(Reporter: ruairi.hickey, Assigned: karnaze)

References

(
URL
)

Details

(Keywords: polish, testcase)

Attachments

(1 file)

testcase
421 bytes, text/html
Details 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.2.16-9mdksecure i686; en-US; m18)
Gecko/20000821
BuildID:    Built from CVS Sept 06 2000

The left frame aligns with both sides of the top frame making it too wide... on
ie5 and netscape the frame is the correct width and the whole page aligns with
the sides of the top frame

Reproducible: Always
Steps to Reproduce:
1.load url
2.
3.

Actual Results:  The page draws incorrectly

Expected Results:
->layout
Assignee: asa → clayton
URL: www.soccernet.com → http://www.soccernet.com
Component: Browser-General → Layout
QA Contact: doronr → petersen
If you resize the browser window after loading the URL, does it straighten
itself out?  I'm noticing a bug right now where a newly loaded or reload page
will lay out slightly wider than the viewable area until the browser window is
resized, even if you go back to the original size or smaller.  I'm seeing this
on 2000090509 on Solaris 8/SPARC.
And, in fact, what I just described is bug 51198.  So this might be a dupe of
that as well.
Created a commented minimal testcase (or close to). This appears to be caused
by how we deal with invalid HTML in tables - text/etc appearing after <table>
but before <tr><td>. Different browsers vary slightly in how they deal with it;
however the best way I can think of would be to insert a virtual <tr><td> before
it, and then let the "cell/row wasn't closed" logic handle inserting </td></tr>
when we see the <tr> following it.

"bad" tables happen all the time on the web, especially at the start. Even
Raggett's old paper on the topic suggests that this case be handled.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: 4xp, polish
This is different than 51198 (which I've also seen and reported).
Keywords: testcase
Looks like a parser problem, content model looks out of wack
CCing Rick
Assignee: clayton → rickg
Component: Layout → Parser
QA Contact: petersen → janc
Back to peter for more triage. Thanks.
Assignee: rickg → peterl
Rick - did you mean to assign this to Peter Linss?  Or is his email address
being reused as an alias for peterlubczynski@netscape.com ?
I meant to resend this to peterlubczynski@netscape.com; he said he had peterl. 
But to be sure, let's do it the long way.
Assignee: peterl → peterlubczynski
oops...content model looks fine, but there is a problem with tables. In the 
testcase, the left and right cells of the outer table should be pretty close to 
balanced but the left cell is bigger. Reassigning to Chris. Here's the table 
dump of the testcase:

  TO::Rfl en 022CA414 rea=0 av=(16170,UC) comp=(0,0) count=0
     T::Rfl en 022CA468 rea=0 av=(16170,UC) comp=(11580,UC) count=1
       TRG::Rfl 022CA4D0 rea=0 av=(UC,UC) comp=(UC,UC) count=2
         TR::Rfl en 022CA514 rea=0 av=(UC,UC) comp=(UC,UC) count=3
           TC::Rfl 022CA564 rea=0 av=(UC,UC) comp=(UC,UC) count=4
             Area::Rfl en 022CA5C4 rea=0 av=(UC,UC) comp=(UC,UC) count=5
             TO::Rfl en 022CA64C rea=0 av=(UC,UC) comp=(UC,0) count=6
               T::Rfl en 022CA6A0 rea=0 av=(UC,UC) comp=(UC,UC) count=7
               T::Rfl ex 022CA6A0 des=(0,0) maxElem=(0,0)
            TO::Rfl ex 022CA64C des=(0,0) maxElem=(0,0)
            Area::Rfl ex 022CA5C4 des=(0,0) maxElem=(0,0)
          TC::Rfl ex 022CA564 des=(45,60) maxElem=(45,60)
        TR::Rfl ex 022CA514 des=(105,60) maxElem=(45,60)
      TRG::Rfl ex 022CA4D0 des=(UC,90) maxElem=(45,60)
      TRG::Rfl 022CA4D0 rea=2 av=(11580,UC) comp=(11580,UC) count=8
         TR::Rfl en 022CA514 rea=2 av=(11580,UC) comp=(11580,UC) count=9
           TC::Rfl 022CA564 rea=2 av=(11520,UC) comp=(11490,UC) count=10
             Area::Rfl en 022CA5C4 rea=2 av=(11490,UC) comp=(11490,UC) count=11
             TO::Rfl en 022CA64C rea=2 av=(11490,UC) comp=(0,0) count=12
               T::Rfl en 022CA6A0 rea=2 av=(11490,UC) comp=(UC,UC) count=13
               T::Rfl ex 022CA6A0 des=(0,30)
            TO::Rfl ex 022CA64C des=(0,30)
            Area::Rfl ex 022CA5C4 des=(11490,30)
          TC::Rfl ex 022CA564 des=(11520,60)
        TR::Rfl ex 022CA514 des=(11550,60)
      TRG::Rfl ex 022CA4D0 des=(11580,60)
    T::Rfl ex 022CA468 des=(11580,120)
  TO::Rfl ex 022CA414 des=(11580,120)
  TO::Rfl en 022CA414 rea=1 av=(16170,UC) comp=(0,0) count=14
     T::Rfl en 022CA468 rea=1 av=(16170,UC) comp=(11580,UC) count=15
       TRG::Rfl 022CA4D0 rea=1 av=(11580,UC) comp=(11580,UC) count=16
         TR::Rfl en 022CA514 rea=1 av=(11580,UC) comp=(11580,UC) count=17
           TC::Rfl 022CA564 rea=1 av=(11520,UC) comp=(11490,UC) count=18
             Area::Rfl en 022CA5C4 rea=1 av=(11490,UC) comp=(11490,UC) count=19
             Area::Rfl ex 022CA5C4 des=(11490,315) maxElem=(315,285)max=540
          TC::Rfl ex 022CA564 des=(11520,345) maxElem=(345,315)max=570
        TR::Rfl ex 022CA514 des=(11580,345) max=0
      TRG::Rfl ex 022CA4D0 des=(11580,0) max=0
      TRG::Rfl 022CA4D0 rea=2 av=(11580,UC) comp=(11580,UC) count=20
         TR::Rfl en 022CA514 rea=2 av=(11580,UC) comp=(11580,UC) count=21
         TR::Rfl ex 022CA514 des=(11550,345)
      TRG::Rfl ex 022CA4D0 des=(11580,345)
    T::Rfl ex 022CA468 des=(11580,405) max=11580
  TO::Rfl ex 022CA414 des=(11580,405)
  TO::Rfl en 022CA414 rea=1 av=(16170,UC) comp=(0,0) count=22
     T::Rfl en 022CA468 rea=1 av=(16170,UC) comp=(11580,UC) count=23
       TRG::Rfl 022CA4D0 rea=1 av=(11580,UC) comp=(11580,UC) count=24
         TR::Rfl en 022CA514 rea=1 av=(11580,UC) comp=(11580,UC) count=25
           TC::Rfl 022CA564 rea=1 av=(11520,UC) comp=(11490,UC) count=26
             Area::Rfl en 022CA5C4 rea=1 av=(11490,UC) comp=(11490,UC) count=27
             TO::Rfl en 022CA64C rea=1 av=(11490,UC) comp=(0,0) count=28
               T::Rfl en 022CA6A0 rea=1 av=(11490,UC) comp=(UC,UC) count=29
                 TRG::Rfl 022CA890 rea=2 av=(UC,UC) comp=(UC,UC) count=30
                   TR::Rfl en 022CA8D4 rea=2 av=(UC,UC) comp=(UC,UC) count=31
                     TC::Rfl 022CA924 rea=0 av=(UC,UC) comp=(UC,UC) count=32
                       Area::Rfl en 022CA984 rea=0 av=(UC,UC) comp=(UC,UC) 
count=33
                       Area::Rfl ex 022CA984 des=(375,285) maxElem=(375,285)
                    TC::Rfl ex 022CA924 des=(405,315) maxElem=(405,315)
                  TR::Rfl ex 022CA8D4 des=(465,315) maxElem=(405,315)
                TRG::Rfl ex 022CA890 des=(UC,315) maxElem=(405,315)
                TRG::Rfl 022CA890 rea=2 av=(465,UC) comp=(465,UC) count=34
                   TR::Rfl en 022CA8D4 rea=2 av=(465,UC) comp=(465,UC) count=35
                     TC::Rfl 022CA924 rea=2 av=(405,UC) comp=(375,UC) count=36
                       Area::Rfl en 022CA984 rea=2 av=(375,UC) comp=(375,UC) 
count=37
                       Area::Rfl ex 022CA984 des=(375,285)
                    TC::Rfl ex 022CA924 des=(405,315)
                  TR::Rfl ex 022CA8D4 des=(435,315)
                TRG::Rfl ex 022CA890 des=(465,315)
              T::Rfl ex 022CA6A0 des=(465,375) maxElem=(465,0)max=465
            TO::Rfl ex 022CA64C des=(465,375) maxElem=(465,0)max=465
            Area::Rfl ex 022CA5C4 des=(11490,660) maxElem=(465,285)max=540
          TC::Rfl ex 022CA564 des=(11520,690) maxElem=(495,315)max=570
        TR::Rfl ex 022CA514 des=(11580,690) max=0
      TRG::Rfl ex 022CA4D0 des=(11580,0) max=0
      TRG::Rfl 022CA4D0 rea=2 av=(11580,UC) comp=(11580,UC) count=38
         TR::Rfl en 022CA514 rea=2 av=(11580,UC) comp=(11580,UC) count=39
         TR::Rfl ex 022CA514 des=(11550,690)
      TRG::Rfl ex 022CA4D0 des=(11580,690)
    T::Rfl ex 022CA468 des=(11580,750) max=11580
  TO::Rfl ex 022CA414 des=(11580,750)
  TO::Rfl en 022CA414 rea=1 av=(16170,UC) comp=(0,0) count=40
     T::Rfl en 022CA468 rea=1 av=(16170,UC) comp=(11580,UC) count=41
       TRG::Rfl 022CA4D0 rea=1 av=(11580,UC) comp=(11580,UC) count=42
         TR::Rfl en 022CA514 rea=1 av=(11580,UC) comp=(11580,UC) count=43
           TC::Rfl 022CA564 rea=1 av=(11520,UC) comp=(11490,UC) count=44
             Area::Rfl en 022CA5C4 rea=1 av=(11490,UC) comp=(11490,UC) count=45
             TO::Rfl en 022CAB1C rea=0 av=(UC,UC) comp=(UC,0) count=46
               T::Rfl en 022CAB70 rea=0 av=(UC,UC) comp=(UC,UC) count=47
                 TRG::Rfl 022CABD8 rea=0 av=(UC,UC) comp=(UC,UC) count=48
                   TR::Rfl en 022CAC1C rea=0 av=(UC,UC) comp=(UC,UC) count=49
                     TC::Rfl 022CAC6C rea=0 av=(UC,UC) comp=(UC,UC) count=50
                       Area::Rfl en 022CACCC rea=0 av=(UC,UC) comp=(UC,UC) 
count=51
                       Area::Rfl ex 022CACCC des=(630,285) maxElem=(630,285)
                    TC::Rfl ex 022CAC6C des=(660,315) maxElem=(660,315)
                  TR::Rfl ex 022CAC1C des=(720,315) maxElem=(660,315)
                TRG::Rfl ex 022CABD8 des=(UC,315) maxElem=(660,315)
              T::Rfl ex 022CAB70 des=(720,345) maxElem=(720,0)
            TO::Rfl ex 022CAB1C des=(720,345) maxElem=(720,0)max=720
            TO::Rfl en 022CAB1C rea=0 av=(11490,UC) comp=(0,0) count=52
               T::Rfl en 022CAB70 rea=0 av=(11490,UC) comp=(UC,UC) count=53
                 TRG::Rfl 022CABD8 rea=2 av=(720,UC) comp=(720,UC) count=54
                   TR::Rfl en 022CAC1C rea=2 av=(720,UC) comp=(720,UC) count=55
                     TC::Rfl 022CAC6C rea=2 av=(660,UC) comp=(630,UC) count=56
                       Area::Rfl en 022CACCC rea=2 av=(630,UC) comp=(630,UC) 
count=57
                       Area::Rfl ex 022CACCC des=(630,285)
                    TC::Rfl ex 022CAC6C des=(660,315)
                  TR::Rfl ex 022CAC1C des=(690,315)
                TRG::Rfl ex 022CABD8 des=(720,315)
              T::Rfl ex 022CAB70 des=(720,375) maxElem=(0,0)
            TO::Rfl ex 022CAB1C des=(11490,375) maxElem=(11490,0)max=11490
            Area::Rfl ex 022CA5C4 des=(11490,1035) maxElem=(11490,375)max=11490
          TC::Rfl ex 022CA564 des=(11520,1065) maxElem=(11520,405)max=11520
        TR::Rfl ex 022CA514 des=(11580,1065) max=0
      TRG::Rfl ex 022CA4D0 des=(11580,0) max=0
      TRG::Rfl 022CA4D0 rea=2 av=(11580,UC) comp=(11580,UC) count=58
         TR::Rfl en 022CA514 rea=2 av=(11580,UC) comp=(11580,UC) count=59
         TR::Rfl ex 022CA514 des=(11550,1065)
      TRG::Rfl ex 022CA4D0 des=(11580,1065)
    T::Rfl ex 022CA468 des=(11580,1125) max=11580
  TO::Rfl ex 022CA414 des=(11580,1125)
  TO::Rfl en 022CA414 rea=1 av=(16170,UC) comp=(0,0) count=60
     T::Rfl en 022CA468 rea=1 av=(16170,UC) comp=(11580,UC) count=61
       TRG::Rfl 022CA4D0 rea=1 av=(11580,UC) comp=(11580,UC) count=62
         TR::Rfl en 022CA514 rea=1 av=(11580,UC) comp=(11580,UC) count=63
           TC::Rfl 022CAEDC rea=0 av=(UC,UC) comp=(6000,UC) count=64
             Area::Rfl en 022CAF3C rea=0 av=(UC,UC) comp=(UC,UC) count=65
             Area::Rfl ex 022CAF3C des=(390,285) maxElem=(390,285)
          TC::Rfl ex 022CAEDC des=(420,315) maxElem=(420,315)
        TR::Rfl ex 022CA514 des=(12000,315) max=0
      TRG::Rfl ex 022CA4D0 des=(11580,0) max=0
      TRG::Rfl 022CA4D0 rea=2 av=(12030,UC) comp=(12030,UC) count=66
         TR::Rfl en 022CA514 rea=2 av=(12030,UC) comp=(12030,UC) count=67
           TC::Rfl 022CAEDC rea=2 av=(420,UC) comp=(390,UC) count=68
             Area::Rfl en 022CAF3C rea=2 av=(390,UC) comp=(390,UC) count=69
             Area::Rfl ex 022CAF3C des=(390,285)
          TC::Rfl ex 022CAEDC des=(420,315)
        TR::Rfl ex 022CA514 des=(12000,1065)
      TRG::Rfl ex 022CA4D0 des=(12030,1065)
    T::Rfl ex 022CA468 des=(12030,1125) max=12030
  TO::Rfl ex 022CA414 des=(12030,1125)
Assignee: peterlubczynski → karnaze
Component: Parser → Layout
Moving to m1.0
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0
QA Contact to gerardok
QA Contact: janc → gerardok
wfm, win2k 10/1/1.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → WORKSFORME
QA Contact: gerardok → amar
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: