Created attachment 490610 [details] [diff] [review] remove useless viewport change In bug 607774 we added at the last minute viewport changes before doing the glClear() call, but it seems that that's useless: glClear() ignores the viewport.
Attachment #490610 - Flags: review?(vladimir)
mm, you're right, glClear does -- but it does not ignore the scissor rectangle. So I think this needs to be a PushScissorRect etc. instead.
We're already disabling the scissor test here. realGLboolean wasScissorTestEnabled = mContext->gl->fIsEnabled(LOCAL_GL_SCISSOR_TEST); mContext->gl->fDisable(LOCAL_GL_SCISSOR_TEST);
Attachment #490610 - Flags: review?(vladimir) → review+
Looks like this can be landed, but wasn't.
Comment on attachment 490610 [details] [diff] [review] remove useless viewport change Forgot to land this... putting r?me so I can't forget it again
Assignee: nobody → bjacob
Target Milestone: --- → mozilla12
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.