Created attachment 532758 [details] [diff] [review]
I do not know why those functions are not static but I guess those were copy-and-paste from static method of nsWindow class.
Comment on attachment 532758 [details] [diff] [review]
AFAIK all of these methods already have internal linkage due to the static keyword in their declaration, but it makes sense to me to explicitly specify in the definition also.
Hiroyuki, when adding "checkin-needed" on a bug, could you make sure your patch is properly formatted so the push is easier to do. See these links:
And thanks for all these patches :)