Merge "only use glReadPixels() when needed when taking screenshots" into jb-mr2-dev
This commit is contained in:
commit
0188eeff1f
@ -109,6 +109,9 @@ SurfaceFlinger::SurfaceFlinger()
|
|||||||
// debugging stuff...
|
// debugging stuff...
|
||||||
char value[PROPERTY_VALUE_MAX];
|
char value[PROPERTY_VALUE_MAX];
|
||||||
|
|
||||||
|
property_get("ro.bq.gpu_to_cpu_unsupported", value, "0");
|
||||||
|
mGpuToCpuSupported = !!atoi(value);
|
||||||
|
|
||||||
property_get("debug.sf.showupdates", value, "0");
|
property_get("debug.sf.showupdates", value, "0");
|
||||||
mDebugRegion = atoi(value);
|
mDebugRegion = atoi(value);
|
||||||
|
|
||||||
@ -2546,15 +2549,16 @@ status_t SurfaceFlinger::captureScreen(const sp<IBinder>& display,
|
|||||||
virtual bool handler() {
|
virtual bool handler() {
|
||||||
Mutex::Autolock _l(flinger->mStateLock);
|
Mutex::Autolock _l(flinger->mStateLock);
|
||||||
sp<const DisplayDevice> hw(flinger->getDisplayDevice(display));
|
sp<const DisplayDevice> hw(flinger->getDisplayDevice(display));
|
||||||
// TODO: if we know the GL->CPU path works, we can call
|
// When we know the GL->CPU path works, we can call
|
||||||
// captureScreenImplLocked() directly, instead of using the
|
// captureScreenImplLocked() directly, instead of using the
|
||||||
// "CpuConsumer" version, which is much less efficient -- it is
|
// "CpuConsumer" version, which is much less efficient -- it is
|
||||||
// however needed by some older drivers.
|
// however needed by some older drivers.
|
||||||
if (isCpuConsumer) {
|
|
||||||
result = flinger->captureScreenImplCpuConsumerLocked(hw,
|
if (flinger->mGpuToCpuSupported || !isCpuConsumer) {
|
||||||
|
result = flinger->captureScreenImplLocked(hw,
|
||||||
producer, reqWidth, reqHeight, minLayerZ, maxLayerZ);
|
producer, reqWidth, reqHeight, minLayerZ, maxLayerZ);
|
||||||
} else {
|
} else {
|
||||||
result = flinger->captureScreenImplLocked(hw,
|
result = flinger->captureScreenImplCpuConsumerLocked(hw,
|
||||||
producer, reqWidth, reqHeight, minLayerZ, maxLayerZ);
|
producer, reqWidth, reqHeight, minLayerZ, maxLayerZ);
|
||||||
}
|
}
|
||||||
return true;
|
return true;
|
||||||
|
@ -412,6 +412,7 @@ private:
|
|||||||
HWComposer* mHwc;
|
HWComposer* mHwc;
|
||||||
GLuint mProtectedTexName;
|
GLuint mProtectedTexName;
|
||||||
nsecs_t mBootTime;
|
nsecs_t mBootTime;
|
||||||
|
bool mGpuToCpuSupported;
|
||||||
sp<EventThread> mEventThread;
|
sp<EventThread> mEventThread;
|
||||||
GLint mMaxViewportDims[2];
|
GLint mMaxViewportDims[2];
|
||||||
GLint mMaxTextureSize;
|
GLint mMaxTextureSize;
|
||||||
|
Loading…
Reference in New Issue
Block a user