diff options
author | bsalomon@google.com <bsalomon@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-05-22 15:44:03 +0000 |
---|---|---|
committer | bsalomon@google.com <bsalomon@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> | 2012-05-22 15:44:03 +0000 |
commit | bcc8486ad421350ecd17c07c3ce76c016894abb3 (patch) | |
tree | b2aabf071039290ffc928e0a3cf3394a0bf457a2 /src/gpu | |
parent | 46256ea736868e6df1a6102672fc4bb41f99d29d (diff) |
Commit comment change that was supposed to go into r4024 but wasn't because I forgot to save.
git-svn-id: http://skia.googlecode.com/svn/trunk@4028 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src/gpu')
-rw-r--r-- | src/gpu/gl/GrGpuGL.h | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/src/gpu/gl/GrGpuGL.h b/src/gpu/gl/GrGpuGL.h index 0be9f9cd64..d140de3477 100644 --- a/src/gpu/gl/GrGpuGL.h +++ b/src/gpu/gl/GrGpuGL.h @@ -69,14 +69,17 @@ protected: kDownOnWrite_UpOnRead_UnpremulConversion } fUnpremulConversion; - // As flush of GL state proceeds the tracking variables are updated to - // reflect the new state. Later parts of the flush may have to perform - // cascaded changes but now the tracking vars have been updated. These flags - // track when the RT or texture has been changed so that the subclass can - // trigger any cascaded changes. Subclass should call resetDirtyFlags after - // its flush is complete. + // The current render target and textures are bound by GrGpuGL when it + // flushes state to GL. After the bindings occur the variables that track + // the current GL state are updated to reflect the new bindings. However, + // the GrGpuGL subclass may have subsequent GL state manipulation it must + // perform whenever RT or textures change. So the GrGpuGL will set these + // dirty flags when it changes the RT or texture bindings. The subclass can + // use them to trigger its dependent state flushing. The subclass should + // call resetDirtyFlags to zero these out after it has consumed them. + // // TODO: Merge GrGpuGLShaders into GrGpuGL and remove the need for these - // flags + // flags. struct { bool fRenderTargetChanged : 1; int fTextureChangedMask; |