379564 - Viewing large image does not trigger horizontal scrollbar after un-zooming

Status: VERIFIED DUPLICATE of bug 400887

(Camino Graveyard :: Page Layout, defect)

Description

[philippe (part-time)]

Attachment: test image

Tested with: 2007050205 (2.0a1pre) + default profile (Troubleshoot Camino)
Trunk only - works fine on Camino Branch.

When viewing large images in a window, no horizontal scrollbar is triggered after scaling up the image.
The edges of the image can only be viewed using the arrow keys.
STR:
Make sure your window is less than 1100px wide for the test.
View attached image
The image opens scaled. Zoom in.
No horizontal scrollbar.

Camino only. No problems with Minefield.

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

We have another bug like this that I think Torben filed a long time ago.

Comment 2

[philippe (part-time)]

(In reply to comment #1)
> We have another bug like this that I think Torben filed a long time ago.
> 
Are you sure ?

I found a regression range for this: 
works
2007041002 (1.2+)
fails
2007041018 (1.2+) (which is a tinderbox build)

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2007-04-10+00%3A00%3A00&maxdate=2007-04-10+18%3A00%3A00&cvsroot=%2Fcvsroot

--> The scrollbar rewrite, bug 370439 is a likely candidate.
or else, bug 372020 (project patch)?

cc'ing Colin

Comment 3

[Colin Barrett [:cbarrett]]

Does Camino generate its own scrollbars for the content area, or does it use Gecko's?

Comment 4

[Stuart Morgan]

It's all Gecko. Has anyone tried this in a Cocoa-widget-enabled Minefield build?

Comment 5

[Colin Barrett [:cbarrett]]

Removing the flag I didn't mean to set when replying originally. Sorry that.

smorgan, you have the right patch to turn on Cocoa widgets? The one I have that Josh gave me in January isn't working.

Comment 6

[Paul Borokhov (lensovet)]

can we set http://upload.wikimedia.org/wikipedia/commons/8/8f/Parthenon_from_south.jpg as the URL? and how is this minor?
fwiw, Minefield Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a5pre) Gecko/20070511 Minefield/3.0a5pre does not suffer from this problem but does suffer from bug 379199.

Comment 7

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

(In reply to comment #1)
> We have another bug like this that I think Torben filed a long time ago.

FWIW, it was Ulrik and bug 308445.

Comment 8

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

> It's all Gecko. Has anyone tried this in a Cocoa-widget-enabled Minefield
> build?
 
Today's Minefield definitely triggers a scrollbar--even on 10.3 ;)

Comment 10

[Kurt Weinschenker]

After noticing this issue, Bugzilla didn't pop this bug up, despite a reasonably thorough search.  Ugh.

Also, I have to agree with Comment #6 -- how is this "minor"?  It seems to me that something like this should be at "normal."  I suppose that developers have higher priorities, but IMO a bug like this one just leaps right out at the end user.  Thanks for listening, guys.

Comment 11

[Stuart Morgan]

(In reply to comment #10)
> I suppose that developers have higher priorities

It was filed that way by the reporter, so it has nothing to do with developer priorities.

Comment 12

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

From the console in a debug build when loading the page (zoom-to-fit enabled), may or may be relevant:

++DOMWINDOW == 4
WARNING: NS_ENSURE_TRUE(NS_SUCCEEDED(rv) && if_info) failed: file /Users/smokey/Camino/dev/trunk/mozilla/dom/src/base/nsDOMClassInfo.cpp, line 4779
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /Users/smokey/Camino/dev/trunk/mozilla/dom/src/base/nsDOMClassInfo.cpp, line 5282
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /Users/smokey/Camino/dev/trunk/mozilla/dom/src/base/nsDOMClassInfo.cpp, line 5856
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x8000FFFF: file /Users/smokey/Camino/dev/trunk/mozilla/dom/src/base/nsDOMClassInfo.cpp, line 7514
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x8000FFFF: file /Users/smokey/Camino/dev/trunk/mozilla/dom/src/base/nsDOMClassInfo.cpp, line 1557
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x8000FFFF: file /Users/smokey/Camino/dev/trunk/mozilla/dom/src/base/nsDOMClassInfo.cpp, line 1557
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x8000FFFF: file /Users/smokey/Camino/dev/trunk/mozilla/content/xbl/src/nsXBLProtoImpl.cpp, line 134
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x8000FFFF: file /Users/smokey/Camino/dev/trunk/mozilla/content/xbl/src/nsXBLProtoImpl.cpp, line 81
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x8000FFFF: file /Users/smokey/Camino/dev/trunk/mozilla/content/xbl/src/nsXBLService.cpp, line 587
JavaScript error: , line 0: uncaught exception: [Exception... "Failure"  nsresult: "0x80004005 (NS_ERROR_FAILURE)"  location: "<unknown>"  data: no]

I suppose there's a chance this could be caused by a missing xpt, so we should check after bug 400887.

Comment 13

[Stuart Morgan]

Adding the xpts from bug 371860's patch fixes this, so duping to bug 400877 where we will be fixing this for trunk.