cc: Don't use partial updates for scrollbars when they are not allowed.

cc/resource_update_controller.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/resource_update_controller.h"
 
 #include <limits>
 
 #include "base/debug/trace_event.h"
 #include "cc/prioritized_resource.h"
 #include "cc/resource_provider.h"
 #include "cc/texture_copier.h"
 #include "cc/thread.h"
 #include "skia/ext/refptr.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebGraphicsContext3D.h"
 #include "third_party/WebKit/Source/Platform/chromium/public/WebSharedGraphicsContext3D.h"
 #include "third_party/khronos/GLES2/gl2.h"
 #include "third_party/skia/include/gpu/SkGpuDevice.h"
 
 using WebKit::WebGraphicsContext3D;
 using WebKit::WebSharedGraphicsContext3D;
 
 namespace {
 
 // Number of partial updates we allow.
-#if defined(OS_ANDROID)
-const size_t partialTextureUpdatesMax = 0;

[enne (OOO), 2012/12/22 01:17:18]

I see that you moved this to settings, but where is the code that uses the value
from settings rather than from resource provider?

[danakj, 2012/12/22 01:22:38]

LayerTreeHost takes the min of settings and the value from the proxy, and puts
that back in the settings.

-#else
 const size_t partialTextureUpdatesMax = 12;
-#endif
 
 // Measured in seconds.
 const double textureUpdateTickRate = 0.004;
 
 // Measured in seconds.
 const double uploaderBusyTickRate = 0.001;
 
 // Number of blocking update intervals to allow.
 const size_t maxBlockingUpdateIntervals = 4;
 
@@ skipping 246 matching lines @@
     if (!uploads)
         return;
 
     while (m_queue->fullUploadSize() && uploads--)
         updateTexture(m_queue->takeFirstFullUpload());
 
     m_resourceProvider->flushUploads();
 }
 
 }  // namespace cc