summaryrefslogtreecommitdiffstats
path: root/gpu
Commit message (Collapse)AuthorAgeFilesLines
* Expose the sandbox related code through the content API. I did a bit of ↵jam@chromium.org2011-10-295-5/+30
| | | | | | | | | | | | | cleanup while I was doing this. -got rid of SandboxInitWrapper, since I didn't see a need to expose given that we can just expose sandbox::SandboxInterfaceInfo -got rid of the duplicated code to initialize the broker -since I made MainFunctionParams only have the sandbox struct on Windows, I also made the mac specific auto release pool behind an ifdef as well. It seemed odd to make something so mac specific compile on all platforms to save some #ifdefs. BUG=98716 Review URL: http://codereview.chromium.org/8414020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107863 0039d316-1c4b-4281-b951-d872f2087c98
* Strip comments from shader before checking for invalid charactersgman@chromium.org2011-10-275-1/+223
| | | | | | | | | | BUG=101870 TEST=unit tests Review URL: http://codereview.chromium.org/8404029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107631 0039d316-1c4b-4281-b951-d872f2087c98
* Split GL binding init into core and extension partjoth@chromium.org2011-10-274-43/+700
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change splits the GL binding initialization into to phases: core functions and extension functions. This is motivated by two peculiarities of the EGL window binding layer: 1. Pointers to core GL entry points may not be queried through eglGetProcAddress(). 2. The return value of eglGetProcAddress() is not specified to be NULL for unsupported functions. In light of these limitations, the current strategy of blindly querying all GL entry points from eglGetProcAddress() and falling back to dlsym() results in bogus function pointers on some EGL implementations. This patch fixes the problem as follows: 1. Add GetGLCoreProcAddress() that only queries dlsym() and use it to initialize pointers to all entry points. On EGL based platforms pointers to extension functions will most likely be NULL at this point. 2. When a context is first made current, look up the set of supported extensions and use GetGLProcAddress() to populate the remaining extension function pointers. The reworked GL bindings generator uses the standard Khronos headers to recognize the extension functions and produce appropriate initialization code. Note that the patch also updates gl2ext.h to match the upstream Khronos version and corrects some minor errors in the downstream changes. BUG=5427391 TEST= Review URL: http://codereview.chromium.org/8381001 Patch from Sami Kyostila <skyostil@chromium.org>. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107602 0039d316-1c4b-4281-b951-d872f2087c98
* Make GLES2Implementation use a larger chunk size for mapping functions.gman@chromium.org2011-10-275-2/+49
| | | | | | | | | | | | | This will make it allocate less shared memory blocks resulting in less file descriptors. BUG=none TEST=unit tests Review URL: http://codereview.chromium.org/8340003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107544 0039d316-1c4b-4281-b951-d872f2087c98
* GpuScheduler can unschedule a command buffer until the GPU has made progress ↵apatrick@chromium.org2011-10-252-1/+91
| | | | | | | | | | | | up to a fence. Other command buffers on the channel can still be processed but if any more calls are issued on the unscheduled command buffer, the entire channel is unscheduled. This prevents reordering of GL commands wrt flushes. It'll be used by this: http://codereview.chromium.org/8060045/ Review URL: http://codereview.chromium.org/8387008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107154 0039d316-1c4b-4281-b951-d872f2087c98
* Rename setVisibility to setVisibilityCHROMIUM and expose as ↵nduca@chromium.org2011-10-251-0/+1
| | | | | | | | GL_CHROIMIUM_set_visibility extension. Review URL: http://codereview.chromium.org/8373006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107034 0039d316-1c4b-4281-b951-d872f2087c98
* Add GLES2Implemention::FreeUnusedSharedMemorygman@chromium.org2011-10-249-5/+102
| | | | | | | | | | | | | | This is to help get the compositor free memory when appropritate BUG=none TEST=unit tests R=vangelis@chromium.org Review URL: http://codereview.chromium.org/8369010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106987 0039d316-1c4b-4281-b951-d872f2087c98
* Adjust workaround for mipmap-related OpenGL implementation bug on Mac OS X.kbr@chromium.org2011-10-212-3/+6
| | | | | | | | BUG=101105 TEST=GPU unit tests; manually ran WebGL conformance test textures/texture-mips.html on NVIDIA and AMD GPUs on Mac OS Review URL: http://codereview.chromium.org/8363016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106747 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 106634 - Adjust workaround for mipmap-related OpenGL implementation ↵kbr@chromium.org2011-10-211-5/+2
| | | | | | | | | | | | | bug on Mac OS X. BUG=101105 TEST=manually ran WebGL conformance test textures/texture-mips.html on NVIDIA and AMD GPUs on Mac OS Review URL: http://codereview.chromium.org/8365018 TBR=kbr@chromium.org Review URL: http://codereview.chromium.org/8362019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106638 0039d316-1c4b-4281-b951-d872f2087c98
* Adjust workaround for mipmap-related OpenGL implementation bug on Mac OS X.kbr@chromium.org2011-10-201-2/+5
| | | | | | | | BUG=101105 TEST=manually ran WebGL conformance test textures/texture-mips.html on NVIDIA and AMD GPUs on Mac OS Review URL: http://codereview.chromium.org/8365018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106634 0039d316-1c4b-4281-b951-d872f2087c98
* Roll skia to r2506.bsalomon@google.com2011-10-201-0/+23
| | | | | | Review URL: http://codereview.chromium.org/8360002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106524 0039d316-1c4b-4281-b951-d872f2087c98
* Make command buffer validate glRenderbufferStorage width and heightgman@chromium.org2011-10-199-26/+127
| | | | | | | | | | TEST=unit tests BUG=86815 Review URL: http://codereview.chromium.org/8342023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106388 0039d316-1c4b-4281-b951-d872f2087c98
* Make OpenGL string entry points validate the stringgman@chromium.org2011-10-182-18/+96
| | | | | | | | | | | | BUG=69808 TEST=unit tests R=apatrick@chromium.org Review URL: http://codereview.chromium.org/8345003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106111 0039d316-1c4b-4281-b951-d872f2087c98
* Last straw for translated_shader_source extension.zmo@google.com2011-10-171-0/+2
| | | | | | | | | | While hooking up with webkit side of this extension, I realized something is missing. So this CL is the fix. BUG=95531 TEST=tested with the webkit side CL Review URL: http://codereview.chromium.org/8294002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105969 0039d316-1c4b-4281-b951-d872f2087c98
* Check for NULL from GetBucketgman@chromium.org2011-10-172-1/+61
| | | | | | | | | | | | There were a few places in the code that needed to check for NULL TEST=unit tests BUG=58468 Review URL: http://codereview.chromium.org/8320007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105920 0039d316-1c4b-4281-b951-d872f2087c98
* Make GPU_DCHECK* keep reference to passed variables pph34r@gmail.com2011-10-141-37/+37
| | | | | | | | | | | | | to avoid unused variable warnings in release build. BUG=87490 TEST=None Review URL: http://codereview.chromium.org/8271002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105432 0039d316-1c4b-4281-b951-d872f2087c98
* Support dynamic switching between integrated and discrete GPUs on Mac OS X.kbr@chromium.org2011-10-132-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change Chrome to allocate most OpenGL contexts with the kCGLPFAAllowOfflineRenderers flag, and specify NSSupportsAutomaticGraphicsSwitching in the Info.plist for the main executable and helper apps. This keeps Chrome on the integrated GPU except when using WebGL, accelerated 2D Canvas, Pepper 3D, and Core Animation-based plugins (except Flash). Chrome shares resources between OpenGL contexts in order to display WebGL and other content in the compositor, and resource sharing doesn't work between contexts allocated on different GPUs. Therefore, when the first context for a given renderer requests the discrete GPU, the channel is dropped and all contexts are reallocated on the discrete GPU. Similarly, when the last context requesting the discrete GPU for a given renderer is shut down, all contexts are dropped and reallocated on the integrated GPU. Currently dynamic GPU switching is only supported on the latest Mac OS X 10.7 update and MacBook Pros with dual AMD / Intel GPUs, though this will improve in future OS updates. Tested with WebGL, CSS 3D, Flash and Unity3D content and observed desired GPU switching behavior. Also added a layout test to WebKit under https://bugs.webkit.org/show_bug.cgi?id=69776 which when run in Chrome catches an assertion failure related to the destruction of contexts. The intent is to add it as a UI layout test on the GPU bots. BUG=88788 TEST=none Review URL: http://codereview.chromium.org/8233027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105399 0039d316-1c4b-4281-b951-d872f2087c98
* Change the enum GL_TRANSLATED_SHADER_SOURCE_LENGTH_ANGLE to the assigned value.zmo@google.com2011-10-133-3/+3
| | | | | | | | | | The ANGLE side change is in r785, thus, roll ANGLE from r784 to r785. BUG=none TEST=tree green Review URL: http://codereview.chromium.org/8252012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105329 0039d316-1c4b-4281-b951-d872f2087c98
* Make command buffer limit cubemaps to 512x512 on Intel on OSXgman@chromium.org2011-10-134-3/+74
| | | | | | | | | | | | | This is to work around bug where cubemaps larger than 512x512 have errors on all Intel macs up to and including OSX 10.7.2 TEST=none BUG=99424 Review URL: http://codereview.chromium.org/8227030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105234 0039d316-1c4b-4281-b951-d872f2087c98
* GPU service understands size of GL_UNSIGNED_INT pixel data type.apatrick@chromium.org2011-10-122-0/+4
| | | | | | | | | It is needed for the GL_GOOGLE_depth_texture extension. BUG=99860 Review URL: http://codereview.chromium.org/8241020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@105164 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure that a ContextGroup is not destroyed if it was not correctly initialized.apatrick@chromium.org2011-10-101-2/+6
| | | | | | | | | The counting of contexts in the group was out of sync, causing an assertion in release builds. BUG=97775 Review URL: http://codereview.chromium.org/8205021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104791 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure that ContextGroup is destroyed when an appropriate GL context is current.apatrick@chromium.org2011-10-055-35/+66
| | | | | | | | | | | Before it relied on the destructor being invoked and this was not always true. I think the subsequent invalid GL commands might have caused memory corruption that lead to the bug referenced below. Now the ContextGroup is destroyed explicitly when the context is known to be correct. BUG=97775 Review URL: http://codereview.chromium.org/8135014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104157 0039d316-1c4b-4281-b951-d872f2087c98
* Implement 'ANGLE_translated_shader_source' extension in command-buffer.zmo@google.com2011-10-0518-5/+192
| | | | | | | | | | | | This works with ANGLE r780. So if it's on top of ANGLE, query the translated HLSL using ANGLE's GetTranslatedShaderSource extension; otherwise, return the cached translated shader source from ANGLE shader translator. BUG=95531 TEST=unittest Review URL: http://codereview.chromium.org/8120020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104036 0039d316-1c4b-4281-b951-d872f2087c98
* From comment:apatrick@chromium.org2011-10-031-0/+9
| | | | | | | | | | | | | | | | | | // TODO(apatrick): Attempt to diagnose crbug.com/97775. If SwapBuffers is // never called on an offscreen context, no data will ever be uploaded to the // saved offscreen color texture (it is deferred until to when SwapBuffers // is called). My idea is that some nvidia drivers might have a bug where // deleting a texture that has never has not been populated might cause a // crash. In the context of bug 97775, the uptime is very short. It is possible that SwapBuffers is never being called on the view context, which on mac is actually an offscreen context. This is a bit a shot in the dark but I haven't reproduced it. BUG=97775 Review URL: http://codereview.chromium.org/8089006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103746 0039d316-1c4b-4281-b951-d872f2087c98
* Add an additional feature to GLES2DecoderImpl that is used to force the ↵twiz@chromium.org2011-10-031-9/+63
| | | | | | | | | | | | | | | context to obey WebGL feature semantics. The 'force_webgl_glsl_validation' flag is used to override the FeatureInfo parameters of the context group. This change enables all extensions for WebGL contexts, in preparation for allowing resource sharing with the compositor. BUG=96605 TEST=https://cvs.khronos.org/svn/repos/registry/trunk/public/webgl/sdk/tests/webgl-conformance-tests.html Review URL: http://codereview.chromium.org/7782038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103724 0039d316-1c4b-4281-b951-d872f2087c98
* External stream texture support.sievers@chromium.org2011-09-2832-78/+774
| | | | | | | | | | | | | | | | | | | | | | This adds a GL_CHROMIUM_stream_texture extension with an abstract interface that can be used to allow a consumer to render from a stream of textures being produced externally. It adds the following functionality: * GLuint glCreateStreamTextureCHROMIUM(GLuint texture) This API takes a texture id and creates a StreamTexture object associated with the texture in the GPU process that can be implemented in a platform-specific way. The returned handle is supposed to identify the stream endpoint. Connecting this to a producer is outside the scope of this change. * GLvoid glDestroyStreamTextureCHROMIUM(GLuint texture) Deletes the StreamTexture object associated with the texture and marks the texture as not special anymore. * BindTexture() will invoke method StreamTexture::Update() if the target is GL_TEXTURE_EXTERNAL_OES and the texture is marked as 'stream texture'. TEST=added unit tests Review URL: http://codereview.chromium.org/7754033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103191 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 102978 - Change X11 error handler override to allow easy X11 error ↵dominich@chromium.org2011-09-2711-61/+27
| | | | | | | | | | | | | | | checking. BUG=None TEST=None Review URL: http://codereview.chromium.org/7889040 TBR=dominich@chromium.org Review URL: http://codereview.chromium.org/8052015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102985 0039d316-1c4b-4281-b951-d872f2087c98
* Change X11 error handler override to allow easy X11 error checking.dominich@chromium.org2011-09-2711-27/+61
| | | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7889040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102978 0039d316-1c4b-4281-b951-d872f2087c98
* Fix issue with unused mips.gman@chromium.org2011-09-272-9/+36
| | | | | | | | | | | | | | | | | | | | The bug was 1) Create a texture with N mips (4x4, 2x2, 1x1) 2) Then update the texture to use less than N mips (2x2, 1x1) etc. The code would incorrectly look at the unused mips in determining whether or not the texture was renderable. TEST=unit test BUG=97207 R=zmo@chromium.org Review URL: http://codereview.chromium.org/8041048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102967 0039d316-1c4b-4281-b951-d872f2087c98
* Roll skia to 2303bsalomon@google.com2011-09-221-0/+3
| | | | | | Review URL: http://codereview.chromium.org/7977031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102366 0039d316-1c4b-4281-b951-d872f2087c98
* Code cleanup: rename DisallowedExtensions to DisallowedFeature.zmo@google.com2011-09-2213-31/+31
| | | | | | | | BUG=none TEST=bots green Review URL: http://codereview.chromium.org/7979031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102213 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes issues where WGC3D based on OSMesa GL implementation isn't allowing ↵bsalomon@google.com2011-09-211-5/+3
| | | | | | | | | | OES extensions that are supported. TEST=none BUG=none Review URL: http://codereview.chromium.org/7980034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102135 0039d316-1c4b-4281-b951-d872f2087c98
* Turn on glsl built-in function emulation in ANGLE shader translator.zmo@google.com2011-09-206-23/+89
| | | | | | | | | | | | At the moment we need to turn it on for ATI/Mac and also Windows/D3D. Also, added the switch to turn of driver bug workaround, although this CL doesn't hook it up with all workarounds yet. BUG=96293 TEST=failing webgl glsl feature tests passing, with the switch, these tests fail. Review URL: http://codereview.chromium.org/7923002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101880 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 101545 - Moved code not relating to GPU scheduling out of ↵apatrick@chromium.org2011-09-2021-1353/+149
| | | | | | | | | | | | | | | | | | GpuScheduler and into GpuCommandBufferStub. Reason for revert: Compile failure on Arm. This was mostly a refactor because the code was awkward. I also deleted the original gles2_demo since we have the gles2 book demos now. THings still to do are a common way of setting up the few objects involved in initializing a command buffer that is now more-or-less duplicated in the gles2 conformance tests, the gles2 demos, the command buffer stub and the in-process webgl context. I also want to completely remove the reference to the decoder from the scheduler. I tested WebGL on both windows and mac and saw no regressions. I checked the conformance tests, the gpu tests and am running by the try bots now. The gles2 demos still work. Review URL: http://codereview.chromium.org/7782041 Review URL: http://codereview.chromium.org/7951008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101873 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101545 - Moved code not relating to GPU scheduling out of ↵rsimha@chromium.org2011-09-1621-149/+1353
| | | | | | | | | | | | | | | | | | | | GpuScheduler and into GpuCommandBufferStub. Reason for revert: Compile failure on Arm. This was mostly a refactor because the code was awkward. I also deleted the original gles2_demo since we have the gles2 book demos now. THings still to do are a common way of setting up the few objects involved in initializing a command buffer that is now more-or-less duplicated in the gles2 conformance tests, the gles2 demos, the command buffer stub and the in-process webgl context. I also want to completely remove the reference to the decoder from the scheduler. I tested WebGL on both windows and mac and saw no regressions. I checked the conformance tests, the gpu tests and am running by the try bots now. The gles2 demos still work. Review URL: http://codereview.chromium.org/7782041 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7920015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101547 0039d316-1c4b-4281-b951-d872f2087c98
* Moved code not relating to GPU scheduling out of GpuScheduler and into ↵apatrick@chromium.org2011-09-1621-1353/+149
| | | | | | | | | | | | | | | GpuCommandBufferStub. This was mostly a refactor because the code was awkward. I also deleted the original gles2_demo since we have the gles2 book demos now. THings still to do are a common way of setting up the few objects involved in initializing a command buffer that is now more-or-less duplicated in the gles2 conformance tests, the gles2 demos, the command buffer stub and the in-process webgl context. I also want to completely remove the reference to the decoder from the scheduler. I tested WebGL on both windows and mac and saw no regressions. I checked the conformance tests, the gpu tests and am running by the try bots now. The gles2 demos still work. Review URL: http://codereview.chromium.org/7782041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101545 0039d316-1c4b-4281-b951-d872f2087c98
* Renamed ppb_opengles.h to ppb_opengles2.h.alokp@chromium.org2011-09-151-3/+11
| | | | | | Review URL: http://codereview.chromium.org/7888051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101308 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 100801 - Ensure that compositor does not block the renderer process ↵apatrick@chromium.org2011-09-131-2/+3
| | | | | | | | | by issuing too many SwapBuffers. This might have regressed with r84372. It caused there to be one additional token queued in the swap buffers token queue than before. Review URL: http://codereview.chromium.org/7876010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100978 0039d316-1c4b-4281-b951-d872f2087c98
* Use Instance* in the Graphics3D constructor rather than Instance&, forbrettw@chromium.org2011-09-131-1/+1
| | | | | | | | | | | | | | | | consistency. I also created a new 2-arg constructor since the share_context will normally be empty, and it's less clear what to use there if you don't have a shared context. I copied comments from the C interface to the C++ one, and converted the comments in the C header file to C-style comments from C++ ones. The documentation still needs some Doxygen love, but we can worry about that later. Review URL: http://codereview.chromium.org/7867048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100865 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100801 - Ensure that compositor does not block the renderer process ↵apatrick@chromium.org2011-09-121-3/+2
| | | | | | | | | | | | | | by issuing too many SwapBuffers. This might have regressed with r84372. It caused there to be one additional token queued in the swap buffers token queue than before. BUG=95649 Review URL: http://codereview.chromium.org/7867053 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7880009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100803 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure that compositor does not block the renderer process by issuing too ↵apatrick@chromium.org2011-09-121-2/+3
| | | | | | | | | | | many SwapBuffers. This might have regressed with r84372. It caused there to be one additional token queued in the swap buffers token queue than before. BUG=95649 Review URL: http://codereview.chromium.org/7867053 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100801 0039d316-1c4b-4281-b951-d872f2087c98
* Keep track of the current per-thread context and surface. Addkbr@google.com2011-09-091-6/+2
| | | | | | | | | | | | | | | | ScopedMakeCurrent to make it easier to briefly make another context current. Use it in AcceleratedSurface implementation and verify in the caller that it is working. This infrastructure would have made it possible to catch http://crbug.com/95492 and similar bugs where the wrong context was current. BUG=95962 TEST=ran CSS 3D example poster-circle; no assertion failures Review URL: http://codereview.chromium.org/7787022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100528 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Revert 100494 - Fix bug in SimulateAttrib0."""gman@chromium.org2011-09-092-42/+137
| | | | | | | | | | TEST=none BUG=95625 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7796016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100507 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100494 - Revert "Revert 100453 - Revert "Revert 100384 - Fix bug in ↵dmichael@chromium.org2011-09-092-137/+42
| | | | | | | | | | | | | | | | | SimulateAttrib0."" The bug was a unsigned->signed conversion issue. Clang truncates, gcc/vc overflow TEST=unit tests BUG=95625 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7863007 TBR=gman@chromium.org Review URL: http://codereview.chromium.org/7796015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100498 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Revert 100453 - Revert "Revert 100384 - Fix bug in SimulateAttrib0.""gman@chromium.org2011-09-092-42/+137
| | | | | | | | | | | | The bug was a unsigned->signed conversion issue. Clang truncates, gcc/vc overflow TEST=unit tests BUG=95625 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7863007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100494 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100453 - Revert "Revert 100384 - Fix bug in SimulateAttrib0 that did ↵dpapad@chromium.org2011-09-092-125/+32
| | | | | | | | | | | | | | | not check for out of memory." TEST=unit tests and run on linux touch (where bug was) BUG=95625 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7841067 TBR=gman@chromium.org Review URL: http://codereview.chromium.org/7866006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100473 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Revert 100384 - Fix bug in SimulateAttrib0 that did not check for ↵gman@chromium.org2011-09-092-32/+125
| | | | | | | | | | | | out of memory." TEST=unit tests and run on linux touch (where bug was) BUG=95625 TBR=apatrick@chromium.org Review URL: http://codereview.chromium.org/7841067 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100453 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 100384 - Fix bug in SimulateAttrib0 that did not check for out of memory.nsylvain@chromium.org2011-09-092-121/+32
| | | | | | | | | | | | | | | | | | It also did not correctly check for math overflow. Also fixed similar bugs in SimulateFixedAttribs TEST=unit tests BUG=95625 R=apatrick@chromium.org Review URL: http://codereview.chromium.org/7845017 TBR=gman@chromium.org Review URL: http://codereview.chromium.org/7857037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100411 0039d316-1c4b-4281-b951-d872f2087c98
* Fix bug in SimulateAttrib0 that did not check for out of memory.gman@chromium.org2011-09-092-32/+121
| | | | | | | | | | | | | | | It also did not correctly check for math overflow. Also fixed similar bugs in SimulateFixedAttribs TEST=unit tests BUG=95625 R=apatrick@chromium.org Review URL: http://codereview.chromium.org/7845017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100384 0039d316-1c4b-4281-b951-d872f2087c98
* A less-mysterious fix for b95492: Restore the GL context after resize.nduca@chromium.org2011-09-092-4/+9
| | | | | | | | | BUG=95492 TEST=crash reports do not reappear in canary Review URL: http://codereview.chromium.org/7847016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100350 0039d316-1c4b-4281-b951-d872f2087c98