Add CC software renderer.

cc/CCResourceProvider.cpp

 // 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 "config.h"
 
 #include "CCResourceProvider.h"
 #ifdef LOG
 #undef LOG
 #endif
@@ skipping 323 matching lines @@
     , m_useShallowFlush(false)
     , m_maxTextureSize(0)
 {
 }
 
 bool CCResourceProvider::initialize()
 {
     ASSERT(CCProxy::isImplThread());
     WebGraphicsContext3D* context3d = m_context->context3D();
     if (!context3d) {
         m_maxTextureSize = INT_MAX;

[danakj, 2012/09/19 18:35:14]

If you do something like

if (textureSize > m_maxTextureSize)

and they are both ints, and m_maxTextureSize == INT_MAX, isn't that going to
just going to be always true?

If this is going to be INT_MAX should m_maxTextureSize be an unsigned (this
would force other places to be unsigned in order to compare)?

It reminds me of all the overflow issues caused by INT_MAX stuff in the subpixel
layout code. The checks we have in place for max texture size would be no-ops in
this case.

[aelias_OOO_until_Jul13, 2012/09/19 19:58:27]

(textureSize > m_maxTextureSize) would always be false, as desired.  That said
you have a point that INT_MAX has some insane properties, for example if we ever
add it to any positive number.  I mainly wanted to avoid a magic number.  I
guess we could go with INT_MAX / 2 or 100000000?

[danakj, 2012/09/19 20:06:18]

er.. false. yes. I'd prefer a magic number, yeh. Maybe something based on
INT_MAX more than an independent constant, so INT_MAX/2.

-
-        // FIXME: Implement this path for software compositing.
-        return false;
+        return true;
     }
     if (!context3d->makeContextCurrent())
         return false;
 
     std::string extensionsString = UTF16ToASCII(context3d->getString(GraphicsContext3D::EXTENSIONS));
     std::vector<std::string> extensions;
     base::SplitString(extensionsString, ' ', &extensions);
     bool useMapSub = false;
     for (size_t i = 0; i < extensions.size(); ++i) {
         if (extensions[i] == "GL_EXT_texture_storage")
@@ skipping 181 matching lines @@
         for (ResourceMap::iterator it = m_resources.begin(); it != m_resources.end(); ++it) {
             if (childPoolSet.contains(it->second.pool))
                 ++maxMailboxCount;
         }
     }
     while (m_mailboxes.size() > maxMailboxCount)
         m_mailboxes.removeFirst();
 }
 
 }