67518 - Page Info image sizes seem to be incorrect

Status: VERIFIED FIXED

(SeaMonkey :: Page Info, defect)

Description

[Warner Young]

Bringing up Page Info on the given URL (and probably others) shows what appears
to be wrong width and height for the images.  I suspect the reported width and
height are taking into account the "border=1" attribute, but I don't know if
this is the correct behavior or not.

Comment 1

[Warner Young]

Forgot to mention that the first image is 512x666, which you can verify by
looking at the page source.  But Page Info lists it as 514x668.

Comment 2

[Keyser Sose]

Confirmed
Platform: PC
OS: Linux 2.2.16
Mozilla Build: 2001020512

Marking NEW.

Comment 3

[Daniel Brooks [:db48x]]

Interesting. It may ultimatly be a c++ problem (still a simple one though) 
because as I recall the code is just using image.width and image.height. If 
these values include the border, I think that it should be changed. When I get 
back to my computer I'll check it out. OTOH, it may not be possible to change 
that without screwing up the layout code, if the layout code is relying on 
these figures to include the border. That would suck.

Comment 4

[hangas]

Sending to component owner

Comment 5

[sairuh (rarely reading bugmail)]

daniel, would you like to take this one?

Comment 6

[Daniel Brooks [:db48x]]

Yes, I'll take it. Also, since I've basically fixed every bug that 52730 depends
on, should I also assign them to myself? If all goes well I'll have a patch that
fixes them all in a few days.

Comment 7

[Stuart Parmenter]

reassign to dom...

Comment 8

[Stuart Parmenter]

reassign to me..

Comment 9

[Stuart Parmenter]

checked in fix for 65708 which fixes this bug.

Comment 10

[Warner Young]

Yup, fixed in 2001052104 build.

Comment 11

[Prashant Desale]

Updating QA contact to Shivakiran Tummala.

Comment 12

[Sivakiran Tummala]

verified