Changeset 76603 in webkit


Ignore:
Timestamp:
Jan 25, 2011 9:47:30 AM (13 years ago)
Author:
zmo@google.com
Message:

2011-01-24 Zhenyao Mo <​zmo@google.com>

Reviewed by Kenneth Russell.

Style cleanup for WebGLRenderingContext
​https://bugs.webkit.org/show_bug.cgi?id=52352

  • html/canvas/WebGLBuffer.cpp:
  • html/canvas/WebGLBuffer.h:
  • html/canvas/WebGLObject.cpp:
  • html/canvas/WebGLProgram.cpp:
  • html/canvas/WebGLRenderbuffer.cpp:
  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::create): (WebCore::WebGLRenderingContext::blendFunc): (WebCore::WebGLRenderingContext::createShader): (WebCore::WebGLRenderingContext::deleteBuffer): (WebCore::WebGLRenderingContext::deleteShader): (WebCore::WebGLRenderingContext::deleteTexture): (WebCore::WebGLRenderingContext::disableVertexAttribArray): (WebCore::WebGLRenderingContext::validateIndexArrayPrecise): (WebCore::WebGLRenderingContext::validateRenderingState): (WebCore::WebGLRenderingContext::enableVertexAttribArray): (WebCore::WebGLRenderingContext::getUniform): (WebCore::WebGLRenderingContext::detachAndRemoveAllObjects):
  • html/canvas/WebGLRenderingContext.h:
  • html/canvas/WebGLShader.cpp:
  • html/canvas/WebGLTexture.cpp:
  • html/canvas/WebGLUniformLocation.cpp:
  • html/canvas/WebGLUniformLocation.h:
Location:
trunk/Source/WebCore
Files:
12 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r76601 r76603  
     12011-01-24  Zhenyao Mo  <zmo@google.com>
     2
     3        Reviewed by Kenneth Russell.
     4
     5        Style cleanup for WebGLRenderingContext
     6        https://bugs.webkit.org/show_bug.cgi?id=52352
     7
     8        * html/canvas/WebGLBuffer.cpp:
     9        * html/canvas/WebGLBuffer.h:
     10        * html/canvas/WebGLObject.cpp:
     11        * html/canvas/WebGLProgram.cpp:
     12        * html/canvas/WebGLRenderbuffer.cpp:
     13        * html/canvas/WebGLRenderingContext.cpp:
     14        (WebCore::WebGLRenderingContext::create):
     15        (WebCore::WebGLRenderingContext::blendFunc):
     16        (WebCore::WebGLRenderingContext::createShader):
     17        (WebCore::WebGLRenderingContext::deleteBuffer):
     18        (WebCore::WebGLRenderingContext::deleteShader):
     19        (WebCore::WebGLRenderingContext::deleteTexture):
     20        (WebCore::WebGLRenderingContext::disableVertexAttribArray):
     21        (WebCore::WebGLRenderingContext::validateIndexArrayPrecise):
     22        (WebCore::WebGLRenderingContext::validateRenderingState):
     23        (WebCore::WebGLRenderingContext::enableVertexAttribArray):
     24        (WebCore::WebGLRenderingContext::getUniform):
     25        (WebCore::WebGLRenderingContext::detachAndRemoveAllObjects):
     26        * html/canvas/WebGLRenderingContext.h:
     27        * html/canvas/WebGLShader.cpp:
     28        * html/canvas/WebGLTexture.cpp:
     29        * html/canvas/WebGLUniformLocation.cpp:
     30        * html/canvas/WebGLUniformLocation.h:
     31
    1322011-01-24  Chris Marrin  <cmarrin@apple.com>
    233

  • trunk/Source/WebCore/html/canvas/WebGLBuffer.cpp

    r76600 r76603  
    3535
    3636namespace WebCore {
    37    
     37
    3838PassRefPtr<WebGLBuffer> WebGLBuffer::create(WebGLRenderingContext* ctx)
    3939{

  • trunk/Source/WebCore/html/canvas/WebGLBuffer.h

    r75741 r76603  
    4343
    4444    bool associateBufferData(GC3Dsizeiptr size);
    45     bool associateBufferData(ArrayBuffer* array);
    46     bool associateBufferData(ArrayBufferView* array);
    47     bool associateBufferSubData(GC3Dintptr offset, ArrayBuffer* array);
    48     bool associateBufferSubData(GC3Dintptr offset, ArrayBufferView* array);
     45    bool associateBufferData(ArrayBuffer*);
     46    bool associateBufferData(ArrayBufferView*);
     47    bool associateBufferSubData(GC3Dintptr offset, ArrayBuffer*);
     48    bool associateBufferSubData(GC3Dintptr offset, ArrayBufferView*);
    4949
    5050    GC3Dsizeiptr byteLength() const;

  • trunk/Source/WebCore/html/canvas/WebGLObject.cpp

    r76600 r76603  
    3333
    3434namespace WebCore {
    35    
     35
    3636WebGLObject::WebGLObject(WebGLRenderingContext* context)
    3737    : m_object(0)

  • trunk/Source/WebCore/html/canvas/WebGLProgram.cpp

    r76600 r76603  
    3333
    3434namespace WebCore {
    35    
     35
    3636PassRefPtr<WebGLProgram> WebGLProgram::create(WebGLRenderingContext* ctx)
    3737{

  • trunk/Source/WebCore/html/canvas/WebGLRenderbuffer.cpp

    r76600 r76603  
    3333
    3434namespace WebCore {
    35    
     35
    3636PassRefPtr<WebGLRenderbuffer> WebGLRenderbuffer::create(WebGLRenderingContext* ctx)
    3737{

  • trunk/Source/WebCore/html/canvas/WebGLRenderingContext.cpp

    r76601 r76603  
    352352        return 0;
    353353    }
    354        
     354
    355355    return new WebGLRenderingContext(canvas, context, attributes);
    356356}
    … …  
    576576}
    577577
    578 
    579578void WebGLRenderingContext::bindFramebuffer(GC3Denum target, WebGLFramebuffer* buffer, ExceptionCode& ec)
    580579{
    … …  
    616615    cleanupAfterGraphicsCall(false);
    617616}
    618 
    619617
    620618void WebGLRenderingContext::bindTexture(GC3Denum target, WebGLTexture* texture, ExceptionCode& ec)
    … …  
    685683    m_context->blendFunc(sfactor, dfactor);
    686684    cleanupAfterGraphicsCall(false);
    687 }       
     685}
    688686
    689687void WebGLRenderingContext::blendFuncSeparate(GC3Denum srcRGB, GC3Denum dstRGB, GC3Denum srcAlpha, GC3Denum dstAlpha)
    … …  
    990988    return o;
    991989}
    992        
     990
    993991PassRefPtr<WebGLFramebuffer> WebGLRenderingContext::createFramebuffer()
    994992{
    … …  
    10361034        return 0;
    10371035    }
    1038    
     1036
    10391037    RefPtr<WebGLShader> o = WebGLShader::create(this, type);
    10401038    addObject(o.get());
    … …  
    10541052    if (isContextLost() || !buffer)
    10551053        return;
    1056    
     1054
    10571055    buffer->deleteObject();
    10581056
    … …  
    11121110    if (isContextLost() || !shader)
    11131111        return;
    1114    
     1112
    11151113    shader->deleteObject();
    11161114}
    … …  
    11201118    if (isContextLost() || !texture)
    11211119        return;
    1122    
     1120
    11231121    texture->deleteObject();
    11241122    if (m_framebufferBinding)
    … …  
    11681166}
    11691167
    1170 
    11711168void WebGLRenderingContext::disable(GC3Denum cap)
    11721169{
    … …  
    11861183        return;
    11871184    }
    1188    
     1185
    11891186    if (index < m_vertexAttribState.size())
    11901187        m_vertexAttribState[index].enabled = false;
    … …  
    13091306            ++p;
    13101307        }
    1311     }   
    1312        
     1308    }
     1309
    13131310    // Then set the last index in the index array and make sure it is valid.
    13141311    numElementsRequired = lastIndex + 1;
    … …  
    13541351        }
    13551352    }
    1356    
     1353
    13571354    if (smallestNumElements == INT_MAX)
    13581355        smallestNumElements = 0;
    1359    
     1356
    13601357    return numElementsRequired <= smallestNumElements;
    13611358}
    … …  
    15131510        return;
    15141511    }
    1515    
     1512
    15161513    if (index >= m_vertexAttribState.size())
    15171514        m_vertexAttribState.resize(index + 1);
    1518        
     1515
    15191516    m_vertexAttribState[index].enabled = true;
    1520    
     1517
    15211518    m_context->enableVertexAttribArray(index);
    15221519    cleanupAfterGraphicsCall(false);
    … …  
    15301527    cleanupAfterGraphicsCall(true);
    15311528}
    1532 
    15331529
    15341530void WebGLRenderingContext::flush()
    … …  
    22442240    }
    22452241    GC3Dint location = uniformLocation->location();
    2246    
     2242
    22472243    WebGLStateRestorer(this, false);
    22482244    // FIXME: make this more efficient using WebGLUniformLocation and caching types in it
    … …  
    37493745    for (HashSet<RefPtr<WebGLObject> >::iterator it = m_canvasObjects.begin(); it != pend; ++it)
    37503746        (*it)->detachContext();
    3751        
     3747
    37523748    m_canvasObjects.clear();
    37533749}

  • trunk/Source/WebCore/html/canvas/WebGLRenderingContext.h

    r76601 r76603  
    7070    virtual bool paintsIntoCanvasBuffer() const;
    7171
    72     void activeTexture(GC3Denum texture, ExceptionCode& ec);
    73     void attachShader(WebGLProgram*, WebGLShader*, ExceptionCode& ec);
    74     void bindAttribLocation(WebGLProgram*, GC3Duint index, const String& name, ExceptionCode& ec);
    75     void bindBuffer(GC3Denum target, WebGLBuffer*, ExceptionCode& ec);
    76     void bindFramebuffer(GC3Denum target, WebGLFramebuffer*, ExceptionCode& ec);
    77     void bindRenderbuffer(GC3Denum target, WebGLRenderbuffer*, ExceptionCode& ec);
    78     void bindTexture(GC3Denum target, WebGLTexture*, ExceptionCode& ec);
     72    void activeTexture(GC3Denum texture, ExceptionCode&);
     73    void attachShader(WebGLProgram*, WebGLShader*, ExceptionCode&);
     74    void bindAttribLocation(WebGLProgram*, GC3Duint index, const String& name, ExceptionCode&);
     75    void bindBuffer(GC3Denum target, WebGLBuffer*, ExceptionCode&);
     76    void bindFramebuffer(GC3Denum target, WebGLFramebuffer*, ExceptionCode&);
     77    void bindRenderbuffer(GC3Denum target, WebGLRenderbuffer*, ExceptionCode&);
     78    void bindTexture(GC3Denum target, WebGLTexture*, ExceptionCode&);
    7979    void blendColor(GC3Dfloat red, GC3Dfloat green, GC3Dfloat blue, GC3Dfloat alpha);
    8080    void blendEquation(GC3Denum mode);
    … …  
    9595    void clearStencil(GC3Dint);
    9696    void colorMask(GC3Dboolean red, GC3Dboolean green, GC3Dboolean blue, GC3Dboolean alpha);
    97     void compileShader(WebGLShader*, ExceptionCode& ec);
     97    void compileShader(WebGLShader*, ExceptionCode&);
    9898
    9999    // void compressedTexImage2D(GC3Denum target, GC3Dint level, GC3Denum internalformat, GC3Dsizei width, GC3Dsizei height, GC3Dint border, GC3Dsizei imageSize, const void* data);
    … …  
    132132    void finish();
    133133    void flush();
    134     void framebufferRenderbuffer(GC3Denum target, GC3Denum attachment, GC3Denum renderbuffertarget, WebGLRenderbuffer*, ExceptionCode& ec);
    135     void framebufferTexture2D(GC3Denum target, GC3Denum attachment, GC3Denum textarget, WebGLTexture*, GC3Dint level, ExceptionCode& ec);
     134    void framebufferRenderbuffer(GC3Denum target, GC3Denum attachment, GC3Denum renderbuffertarget, WebGLRenderbuffer*, ExceptionCode&);
     135    void framebufferTexture2D(GC3Denum target, GC3Denum attachment, GC3Denum textarget, WebGLTexture*, GC3Dint level, ExceptionCode&);
    136136    void frontFace(GC3Denum mode);
    137137    void generateMipmap(GC3Denum target);
    … …  
    148148    WebGLGetInfo getParameter(GC3Denum pname, ExceptionCode&);
    149149    WebGLGetInfo getProgramParameter(WebGLProgram*, GC3Denum pname, ExceptionCode&);
    150     String getProgramInfoLog(WebGLProgram*, ExceptionCode& ec);
     150    String getProgramInfoLog(WebGLProgram*, ExceptionCode&);
    151151    WebGLGetInfo getRenderbufferParameter(GC3Denum target, GC3Denum pname, ExceptionCode&);
    152     WebGLGetInfo getShaderParameter(WebGLShader*, GC3Denum pname, ExceptionCode& ec);
    153     String getShaderInfoLog(WebGLShader*, ExceptionCode& ec);
     152    WebGLGetInfo getShaderParameter(WebGLShader*, GC3Denum pname, ExceptionCode&);
     153    String getShaderInfoLog(WebGLShader*, ExceptionCode&);
    154154
    155155    // TBD
    … …  
    193193    void texImage2D(GC3Denum target, GC3Dint level, GC3Denum internalformat,
    194194                    GC3Dsizei width, GC3Dsizei height, GC3Dint border,
    195                     GC3Denum format, GC3Denum type, ArrayBufferView* pixels, ExceptionCode&);
     195                    GC3Denum format, GC3Denum type, ArrayBufferView*, ExceptionCode&);
    196196    void texImage2D(GC3Denum target, GC3Dint level, GC3Denum internalformat,
    197                     GC3Denum format, GC3Denum type, ImageData* pixels, ExceptionCode&);
     197                    GC3Denum format, GC3Denum type, ImageData*, ExceptionCode&);
    198198    void texImage2D(GC3Denum target, GC3Dint level, GC3Denum internalformat,
    199                     GC3Denum format, GC3Denum type, HTMLImageElement* image, ExceptionCode&);
     199                    GC3Denum format, GC3Denum type, HTMLImageElement*, ExceptionCode&);
    200200    void texImage2D(GC3Denum target, GC3Dint level, GC3Denum internalformat,
    201                     GC3Denum format, GC3Denum type, HTMLCanvasElement* canvas, ExceptionCode&);
     201                    GC3Denum format, GC3Denum type, HTMLCanvasElement*, ExceptionCode&);
    202202    void texImage2D(GC3Denum target, GC3Dint level, GC3Denum internalformat,
    203                     GC3Denum format, GC3Denum type, HTMLVideoElement* video, ExceptionCode&);
     203                    GC3Denum format, GC3Denum type, HTMLVideoElement*, ExceptionCode&);
    204204
    205205    void texParameterf(GC3Denum target, GC3Denum pname, GC3Dfloat param);
    … …  
    208208    void texSubImage2D(GC3Denum target, GC3Dint level, GC3Dint xoffset, GC3Dint yoffset,
    209209                       GC3Dsizei width, GC3Dsizei height,
    210                        GC3Denum format, GC3Denum type, ArrayBufferView* pixels, ExceptionCode&);
     210                       GC3Denum format, GC3Denum type, ArrayBufferView*, ExceptionCode&);
    211211    void texSubImage2D(GC3Denum target, GC3Dint level, GC3Dint xoffset, GC3Dint yoffset,
    212                        GC3Denum format, GC3Denum type, ImageData* pixels, ExceptionCode&);
     212                       GC3Denum format, GC3Denum type, ImageData*, ExceptionCode&);
    213213    void texSubImage2D(GC3Denum target, GC3Dint level, GC3Dint xoffset, GC3Dint yoffset,
    214                        GC3Denum format, GC3Denum type, HTMLImageElement* image, ExceptionCode&);
     214                       GC3Denum format, GC3Denum type, HTMLImageElement*, ExceptionCode&);
    215215    void texSubImage2D(GC3Denum target, GC3Dint level, GC3Dint xoffset, GC3Dint yoffset,
    216                        GC3Denum format, GC3Denum type, HTMLCanvasElement* canvas, ExceptionCode&);
     216                       GC3Denum format, GC3Denum type, HTMLCanvasElement*, ExceptionCode&);
    217217    void texSubImage2D(GC3Denum target, GC3Dint level, GC3Dint xoffset, GC3Dint yoffset,
    218                        GC3Denum format, GC3Denum type, HTMLVideoElement* video, ExceptionCode&);
     218                       GC3Denum format, GC3Denum type, HTMLVideoElement*, ExceptionCode&);
    219219
    220220    void uniform1f(const WebGLUniformLocation* location, GC3Dfloat x, ExceptionCode&);
    … …  
    335335    bool validateRenderingState(int numElements);
    336336
    337     bool validateWebGLObject(WebGLObject* object);
    338 
    339     PassRefPtr<Image> videoFrameToImage(HTMLVideoElement* video);
     337    bool validateWebGLObject(WebGLObject*);
     338
     339    PassRefPtr<Image> videoFrameToImage(HTMLVideoElement*);
    340340
    341341    RefPtr<GraphicsContext3D> m_context;
    … …  
    458458
    459459    // Helpers for getParameter and others
    460     WebGLGetInfo getBooleanParameter(GC3Denum pname);
    461     WebGLGetInfo getBooleanArrayParameter(GC3Denum pname);
    462     WebGLGetInfo getFloatParameter(GC3Denum pname);
    463     WebGLGetInfo getIntParameter(GC3Denum pname);
    464     WebGLGetInfo getUnsignedIntParameter(GC3Denum pname);
    465     WebGLGetInfo getWebGLFloatArrayParameter(GC3Denum pname);
    466     WebGLGetInfo getWebGLIntArrayParameter(GC3Denum pname);
     460    WebGLGetInfo getBooleanParameter(GC3Denum);
     461    WebGLGetInfo getBooleanArrayParameter(GC3Denum);
     462    WebGLGetInfo getFloatParameter(GC3Denum);
     463    WebGLGetInfo getIntParameter(GC3Denum);
     464    WebGLGetInfo getUnsignedIntParameter(GC3Denum);
     465    WebGLGetInfo getWebGLFloatArrayParameter(GC3Denum);
     466    WebGLGetInfo getWebGLIntArrayParameter(GC3Denum);
    467467
    468468    void texImage2DBase(GC3Denum target, GC3Dint level, GC3Denum internalformat,
    … …  
    470470                        GC3Denum format, GC3Denum type, void* pixels, ExceptionCode&);
    471471    void texImage2DImpl(GC3Denum target, GC3Dint level, GC3Denum internalformat,
    472                         GC3Denum format, GC3Denum type, Image* image,
     472                        GC3Denum format, GC3Denum type, Image*,
    473473                        bool flipY, bool premultiplyAlpha, ExceptionCode&);
    474474    void texSubImage2DBase(GC3Denum target, GC3Dint level, GC3Dint xoffset, GC3Dint yoffset,
    … …  
    547547    // Helper function to print warnings to console. Currently
    548548    // used only to warn about use of obsolete functions.
    549     void printWarningToConsole(const String& message);
     549    void printWarningToConsole(const String&);
    550550
    551551    // Helper function to validate input parameters for framebuffer functions.
    … …  
    563563
    564564    // Helper function to validate input parameters for uniform functions.
    565     bool validateUniformParameters(const WebGLUniformLocation* location, Float32Array* v, GC3Dsizei mod);
    566     bool validateUniformParameters(const WebGLUniformLocation* location, Int32Array* v, GC3Dsizei mod);
    567     bool validateUniformParameters(const WebGLUniformLocation* location, void* v, GC3Dsizei size, GC3Dsizei mod);
    568     bool validateUniformMatrixParameters(const WebGLUniformLocation* location, GC3Dboolean transpose, Float32Array* v, GC3Dsizei mod);
    569     bool validateUniformMatrixParameters(const WebGLUniformLocation* location, GC3Dboolean transpose, void* v, GC3Dsizei size, GC3Dsizei mod);
     565    bool validateUniformParameters(const WebGLUniformLocation*, Float32Array*, GC3Dsizei mod);
     566    bool validateUniformParameters(const WebGLUniformLocation*, Int32Array*, GC3Dsizei mod);
     567    bool validateUniformParameters(const WebGLUniformLocation*, void*, GC3Dsizei size, GC3Dsizei mod);
     568    bool validateUniformMatrixParameters(const WebGLUniformLocation*, GC3Dboolean transpose, Float32Array*, GC3Dsizei mod);
     569    bool validateUniformMatrixParameters(const WebGLUniformLocation*, GC3Dboolean transpose, void*, GC3Dsizei size, GC3Dsizei mod);
    570570
    571571    // Helper function to validate parameters for bufferData.
    … …  
    574574
    575575    // Helper functions for vertexAttribNf{v}.
    576     void vertexAttribfImpl(GC3Duint index, GC3Dsizei expectedSize, GC3Dfloat v0, GC3Dfloat v1, GC3Dfloat v2, GC3Dfloat v3);
    577     void vertexAttribfvImpl(GC3Duint index, Float32Array* v, GC3Dsizei expectedSize);
    578     void vertexAttribfvImpl(GC3Duint index, GC3Dfloat* v, GC3Dsizei size, GC3Dsizei expectedSize);
     576    void vertexAttribfImpl(GC3Duint index, GC3Dsizei expectedSize, GC3Dfloat, GC3Dfloat, GC3Dfloat, GC3Dfloat);
     577    void vertexAttribfvImpl(GC3Duint index, Float32Array*, GC3Dsizei expectedSize);
     578    void vertexAttribfvImpl(GC3Duint index, GC3Dfloat*, GC3Dsizei size, GC3Dsizei expectedSize);
    579579
    580580    // Helpers for simulating vertexAttrib0

  • trunk/Source/WebCore/html/canvas/WebGLShader.cpp

    r76600 r76603  
    3333
    3434namespace WebCore {
    35    
     35
    3636PassRefPtr<WebGLShader> WebGLShader::create(WebGLRenderingContext* ctx, GC3Denum type)
    3737{

  • trunk/Source/WebCore/html/canvas/WebGLTexture.cpp

    r76600 r76603  
    3434
    3535namespace WebCore {
    36    
     36
    3737PassRefPtr<WebGLTexture> WebGLTexture::create(WebGLRenderingContext* ctx)
    3838{

  • trunk/Source/WebCore/html/canvas/WebGLUniformLocation.cpp

    r76600 r76603  
    3232
    3333namespace WebCore {
    34    
     34
    3535PassRefPtr<WebGLUniformLocation> WebGLUniformLocation::create(WebGLProgram* program, GC3Dint location)
    3636{

  • trunk/Source/WebCore/html/canvas/WebGLUniformLocation.h

    r75741 r76603  
    4040    virtual ~WebGLUniformLocation() { }
    4141
    42     static PassRefPtr<WebGLUniformLocation> create(WebGLProgram* program, GC3Dint location);
     42    static PassRefPtr<WebGLUniformLocation> create(WebGLProgram*, GC3Dint location);
    4343
    4444    WebGLProgram* program() const;
    … …  
    4747
    4848protected:
    49     WebGLUniformLocation(WebGLProgram* program, GC3Dint location);
     49    WebGLUniformLocation(WebGLProgram*, GC3Dint location);
    5050
    5151private:
Note: See TracChangeset for help on using the changeset viewer.