Closed Bug 612320 Opened 11 years ago Closed 10 years ago
remove useless viewport changes
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
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.