Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(328)

Unified Diff: include/gpu/GrContextOptions.h

Issue 1161543003: Set GeometryBufferMapThreshold defaults (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: more Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « include/gpu/GrCaps.h ('k') | src/gpu/gl/GrGLCaps.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/gpu/GrContextOptions.h
diff --git a/include/gpu/GrContextOptions.h b/include/gpu/GrContextOptions.h
index d8d59fac3a89186a6705279f446ef5c2ca978271..dfdab81399089b1f1a96243dd666438a93c4984e 100644
--- a/include/gpu/GrContextOptions.h
+++ b/include/gpu/GrContextOptions.h
@@ -16,7 +16,7 @@ struct GrContextOptions {
, fSuppressPrints(false)
, fMaxTextureSizeOverride(SK_MaxS32)
, fSuppressDualSourceBlending(false)
- , fGeometryBufferMapThreshold(1 << 15) {}
+ , fGeometryBufferMapThreshold(-1) {}
// EXPERIMENTAL
// May be removed in the future, or may become standard depending
@@ -33,11 +33,10 @@ struct GrContextOptions {
int fMaxTextureSizeOverride;
bool fSuppressDualSourceBlending;
- /** fGeometryBufferMapThreshold gives a threshold (in bytes) for when Gr should
- map a GrGeometryBuffer to update its contents. It will use map() if the
- size of the updated region is greater than the threshold. Otherwise it will
- use updateData(). */
- size_t fGeometryBufferMapThreshold;
+ /** the threshold in bytes above which we will use a buffer mapping API to map vertex and index
+ buffers to CPU memory in order to update them. A value of -1 means the GrContext should
+ deduce the optimal value for this platform. */
+ int fGeometryBufferMapThreshold;
};
#endif
« no previous file with comments | « include/gpu/GrCaps.h ('k') | src/gpu/gl/GrGLCaps.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698