New patch that should alleviate crash. This supercedes the previous patch.

Possibly if there is no current window/document, we want to throw an exception rather

than return L"" - I think this would be more meaningful.

Maybe name pageCount (line 117) verticalPageCount to be explicit and contrast with

horizontalPageCount

How does it behave with a non-integer number of pages? It looks like to will probably

replicate some of the pixels...

(I have not tested the patch - I'm assuming you have)

Re: No current window or document: My only question is "what does Firefox do"? I'll

research that further.

Yeah, the variable name should be verticalPageCount. I was just lazy.

If the page size is not an exact multiple of the viewable area, we will grab some of

the same pixels multiple times. However, when we write those pixels to the full page

image, we position them in the right place, so we are overwriting what is already 
there with an exact copy. I'm not too worried about this for the first revision; if

someone wants to come along later and fix it up, that's okay with me.

Is there a way to capture the full buffer at once rather than scrolling?  Scrolling
ends up not working well with fixed position elements.  I suspect it's a problem for
sites that iteratively add content as you reach the bottom of the viewport as well.

I have also looked into screenshots for IE, and I'd like to merge our efforts. But,

when I use your code on IE8, the screenshot is not right (attached converted to PNG).

It looks like the width isn't properly calculated and it's not scrolling. I will try
to 
figure out what the issue is.

Funny, I use IE8 to develop against and I didn't see any issues. Could you post the
PNG?

Sorry, thought I attached it, here it is.

dup and fixed with item #5876