Initial work to get ETC1 data up to the GPU

src/gpu/GrGpu.h

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef GrGpu_DEFINED
 #define GrGpu_DEFINED
 
@@ skipping 68 matching lines @@
      * @param desc        describes the texture to be created.
      * @param srcData     texel data to load texture. Begins with full-size
      *                    palette data for paletted textures. Contains width*
      *                    height texels. If NULL texture data is uninitialized.
      *
      * @return    The texture object if successful, otherwise NULL.
      */
     GrTexture* createTexture(const GrTextureDesc& desc,
                              const void* srcData, size_t rowBytes);
 
     /**

[robertphillips, 2014/05/30 15:00:22]

Can we test these cases (on a Xoom say) ?

+     * Creates a compressed texture object. If desc width or height is not a power of
+     * two but underlying API requires a power of two texture then this function will
+     * return NULL. If the pixel configuration passed in desc is not a valid format
+     * supported by the device, then this function will return NULL.
+     *
+     * The kRenderTarget_TextureFlag should not be specified
+     *
+     * @param desc        describes the texture to be created. The width and height of
+     *                    the texture should be multiples of the block size for the
+     *                    requested compressed format. The origin of the texture is
+     *                    also expected to be in the top left.
+     * @param srcData     texel data to load texture. This is expected to be the
+     *                    proper size based on the width and height of the texture along
+     *                    with the associated block size specified in GrTextureFormatDesc
+     *
+     * @return    The texture object if successful, otherwise NULL.
+     */
+    GrTexture* createCompressedTexture(const GrTextureDesc& desc, const void* srcData);

[bsalomon, 2014/05/30 15:16:58]

I'm wondering if this makes sense as a public GrGpu function. I can see wanting
to split in the implementation (either as onCreateTexture and
onCreateCompressedTexture or in the GrGpuGL code), but isn't the distinction
between creating a compressed or uncompressed texture now already captured in
the desc's config? Why make the caller worry about this detail?

[krajcevski, 2014/05/30 16:21:50]

I agree to an extent... it's always better to be implicit rather than explicit.
That being said, the distinction between compressed versus uncompressed textures
probably doesn't need to be at this height of the API. I've changed the
description to reflect the new behavior of createTexture by moving
createComrpessedTexture logic into it.

+
+    /**
      * Implements GrContext::wrapBackendTexture
      */
     GrTexture* wrapBackendTexture(const GrBackendTextureDesc&);
 
     /**
      * Implements GrContext::wrapBackendTexture
      */
     GrRenderTarget* wrapBackendRenderTarget(const GrBackendRenderTargetDesc&);
 
     /**
@@ skipping 307 matching lines @@
 
 
     // called when the 3D context state is unknown. Subclass should emit any
     // assumed 3D context state and dirty any state cache.
     virtual void onResetContext(uint32_t resetBits) = 0;
 
     // overridden by backend-specific derived class to create objects.
     virtual GrTexture* onCreateTexture(const GrTextureDesc& desc,
                                        const void* srcData,
                                        size_t rowBytes) = 0;
+    virtual GrTexture* onCreateCompressedTexture(const GrTextureDesc& desc,
+                                                 const void* srcData) = 0;
     virtual GrTexture* onWrapBackendTexture(const GrBackendTextureDesc&) = 0;
     virtual GrRenderTarget* onWrapBackendRenderTarget(const GrBackendRenderTargetDesc&) = 0;
     virtual GrVertexBuffer* onCreateVertexBuffer(size_t size, bool dynamic) = 0;
     virtual GrIndexBuffer* onCreateIndexBuffer(size_t size, bool dynamic) = 0;
     virtual GrPath* onCreatePath(const SkPath& path, const SkStrokeRec&) = 0;
 
     // overridden by backend-specific derived class to perform the clear and
     // clearRect. NULL rect means clear whole target. If canIgnoreRect is
     // true, it is okay to perform a full clear instead of a partial clear
     virtual void onClear(const SkIRect* rect, GrColor color, bool canIgnoreRect) = 0;
@@ skipping 87 matching lines @@
     // these are mutable so they can be created on-demand
     mutable GrIndexBuffer*                                              fQuadIndexBuffer;
     // Used to abandon/release all resources created by this GrGpu. TODO: Move this
     // functionality to GrResourceCache.
     ObjectList                                                          fObjectList;
 
     typedef GrDrawTarget INHERITED;
 };
 
 #endif