Martijn Wargers (dead) Assignee
	Description • 19 years ago

See upcoming testcase.
The showAutoscrollMarker relies on documentElement.offsetHeight and -offsetWidth
to get the document height.
This is not the case with the testcase.
The result of this is that the user gets to see the wrong autoscrollmarker image
(autoscroll_all.png).

	Martijn Wargers (dead) Assignee
	Comment 2 • 19 years ago

This patch fixes it for me. I've also checked this works in xul and xml
environments.
I've also simplified some other code in that function (which I think is an
improvement).

	Martijn Wargers (dead) Assignee
	Comment 3 • 19 years ago

Uhm, Gavin, why have you cancelled my review? Did I do something wrong?

	:Gavin Sharp [email: gavin@gavinsharp.com]
	Comment 4 • 19 years ago

Comment on attachment 184934 [details] [diff] [review]
patch

No, the change of component canceled the request. Sorry Martijn.

	Martijn Wargers (dead) Assignee
	Comment 5 • 19 years ago

Ok, no problem, Gavin, but this sounds like a bugzilla bug to me (and I can't
find any bug that is similar to this).

	:Gavin Sharp [email: gavin@gavinsharp.com]
	Comment 6 • 19 years ago

Yes, it's bugzilla bug 261995.

	Martijn Wargers (dead) Assignee
	Comment 7 • 19 years ago

*** Bug 216035 has been marked as a duplicate of this bug. ***

	Martijn Wargers (dead) Assignee
	Comment 8 • 19 years ago

Comment on attachment 184934 [details] [diff] [review]
patch

Anyway, this is good enough. The overflow:hidden case can be solved, but I'll
do that in an eventuel other patch.

	Martijn Wargers (dead) Assignee
	Comment 9 • 19 years ago

Oops! Comment 8 is about the wrong bug (browser tabs can sometimes be very
confusing).

	Mike Connor [:mconnor]
	Comment 10 • 19 years ago

Comment on attachment 184934 [details] [diff] [review]
patch

I'm not in wifi range as I'm writing this, but I swear I've reviewed this patch
before.

	Martijn Wargers (dead) Assignee
	Comment 11 • 19 years ago

Checked in by bz.

	:Gavin Sharp [email: gavin@gavinsharp.com]
	Comment 12 • 19 years ago

Checked in on the 1.8 branch.
mozilla/toolkit/content/widgets/browser.xml; new revision: 1.70.2.6;