summaryrefslogtreecommitdiffstats
path: root/gpu
Commit message (Collapse)AuthorAgeFilesLines
...
* Expose GLES2Interface to blink, and delete isContextLost() from WGC3D.danakj2016-03-163-10/+12
| | | | | | | | | | | | | | This deletes the isContextLost() method from WebGraphicsContext3D, as an example, and replaces calls to that in blink by called to GetGraphicsResetStatusKHR() on the GLES2Interface directly. R=esprehn@chromium.org, kbr@chromium.org, zmo@chromium.org BUG=584497 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1790753002 Cr-Commit-Position: refs/heads/master@{#381392}
* Add kbr to gpu/blink/OWNERS.danakj2016-03-161-0/+1
| | | | | | | | R=kbr@chromium.org, piman@chromium.org Review URL: https://codereview.chromium.org/1804323002 Cr-Commit-Position: refs/heads/master@{#381376}
* Initialize the GLES3 builtin shader constantscwallez2016-03-155-3/+115
| | | | | | | | | | | | | Previously the default values were used which didn't match the same values queried from the API, causing a subset of the deqp/functional/gles3/shaderbuiltinvar tests to fail. BUG=587781 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1759883003 Cr-Commit-Position: refs/heads/master@{#381127}
* Implement unpack alignment driver bug workaround.zmo2016-03-145-39/+219
| | | | | | | | | | | | | | | | | | On some drivers (Linux NVIDIA, Mac AMD, etc), Tex uploading from unpack buffer generates an error if buffer size does not unclude padding for the last row. With this CL, conformance2/textures/misc/tex-unpack-params.html pass on the above mentioned drivers. BUG=429053 TEST=conformance2/textures/misc/tex-unpack-params.html R=piman@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1798193002 Cr-Commit-Position: refs/heads/master@{#381113}
* [gpu/config] Extend cubemap exception to 10.9cwallez2016-03-141-2/+2
| | | | | | | | | | | | | | | | This will enable the exception for the mac_release_optional trybots running NVIDIA GPUs. This is necessary because these bots currently use the workaround list for Intel GPU when running WebGL (fixing this is TODO). Extending the exception should be safe because the workaround is older than 2012 and MacOS 10.9 was released in 2014. BUG=587781 BUG=576219 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1781863003 Cr-Commit-Position: refs/heads/master@{#381082}
* content gpu refactor: get rid of gpu related content switches.penghuang2016-03-141-0/+18
| | | | | | | | | | | | | BUG=590825 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq Committed: https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb Cr-Commit-Position: refs/heads/master@{#380906} Review URL: https://codereview.chromium.org/1784193003 Cr-Commit-Position: refs/heads/master@{#380988}
* Revert of content gpu refactor: get rid of gpu related content switches. ↵rockot2016-03-131-18/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #8 id:100002 of https://codereview.chromium.org/1784193003/ ) Reason for revert: Closed the tree: https://build.chromium.org/p/chromium/builders/Win/builds/41257/steps/compile/logs/stdio Original issue's description: > content gpu refactor: get rid of gpu related content switches. > > BUG=590825 > CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel > CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq > > Committed: https://crrev.com/9288ad04012900a9656959f441f4519f4239dccb > Cr-Commit-Position: refs/heads/master@{#380906} TBR=dcheng@chromium.org,mcasas@chromium.org,piman@chromium.org,penghuang@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=590825 Review URL: https://codereview.chromium.org/1796633002 Cr-Commit-Position: refs/heads/master@{#380908}
* content gpu refactor: get rid of gpu related content switches.penghuang2016-03-131-0/+18
| | | | | | | | | | BUG=590825 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq;tryserver.chromium.perf:linux_perf_cq Review URL: https://codereview.chromium.org/1784193003 Cr-Commit-Position: refs/heads/master@{#380906}
* gpu: Rename GpuScheduler to CommandExecutor.sunnyps2016-03-1219-195/+169
| | | | | | | | | | | | | | GpuScheduler has been a misnomer for what the class does. This CL is a simple refactoring to rename gpu::GpuScheduler to gpu::CommandExecutor. This allows the GpuScheduler name to be used for the upcoming scheduler and at the very least avoid confusion. BUG=514813 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1794463004 Cr-Commit-Position: refs/heads/master@{#380851}
* Upgrade Tex{Sub}Image{2|3}D to handle ES3 unpack parameters.zmo2016-03-118-188/+714
| | | | | | | | | | | | | | | | | | | Also, this CL changed the SKIP_* unpack parameters to be fully handled on the client side and never get sent down to the service side. Also, this CL fixed a bug in ClearLevel3D. If an unpack buffer is bound, unpack parameters have been submitted to the driver already. So when we call TexSubImage3D to clear a 3D texture, we need to clear these parameters if set by client, and restore them afterwards. BUG=487841,429053 TEST=gpu_unittests,webgl_conformance,webgl2_conformance R=piman@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1750123002 Cr-Commit-Position: refs/heads/master@{#380752}
* Texture swizzling for GL_ALPHA and GL_LUMINANCE emulationbajones2016-03-1111-31/+433
| | | | | | | | | | | | | This allows partial use of GL_ALPHA and GL_LUMINANCE textures when running on OpenGL core profile. It does not allow CopyTexture to work as expected. That will come in a follow up patch. BUG=577144 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1662903004 Cr-Commit-Position: refs/heads/master@{#380669}
* Remove GLSurfaceHandle and simplify view command bufferspiman2016-03-112-0/+19
| | | | | | | | | | | | | | | | | | | - Introduce gpu::SurfaceHandle which is just gfx::PluginWindowHandle, but separates GPU uses from plugin uses (which will disappear) - remove GLSurfaceHandle, in practice they're either empty (offscreen) or NATIVE_DIRECT. NULL_TRANSPORT was only used in tests. replace by gpu::SurfaceHandle which is just the handle (without enums) - remove surface_ids from content/common/gpu/client, use the SurfaceHandle directly - remove GLSurfaceHandle from GpuSurfaceTracker - the SurfaceHandle is always statically known (either the surface_id on Android/Mac or the AcceleratedWidget on Linux/Win). BUG=None CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1782743002 Cr-Commit-Position: refs/heads/master@{#380656}
* Fix transform feedback bugsqiankun.miao2016-03-115-29/+94
| | | | | | | | | | | | | | | | | This CL fixes two bugs: 1. Deleting an active transform feedback object will generate INVALID_OPERATION error (page 86, ES spec 3.0.4). 2. bindBufferBase bind the buffer object to both the general binding point and the binding point in the array givien by index (page 35, ES spec 3.0.4). BUG=295792, 591258 TEST=deqp/functional/gles3/lifetime.html CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1752703003 Cr-Commit-Position: refs/heads/master@{#380610}
* [WebGL] a bunch of small fixes: unnecessary arguments, typos...yunchao.he2016-03-112-3/+3
| | | | | | | | | BUG=295792 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1783743002 Cr-Commit-Position: refs/heads/master@{#380601}
* Fix a few issues in Tex function handling.zmo2016-03-113-67/+94
| | | | | | | | | | | | | | | | | | This is part of https://codereview.chromium.org/1750123002/ 1) Simplify row copying 2) Rename a function (remove unpack because it's also used by ReadPixels); add extra validation. 3) Fix ClearLevel3D BUG=487841, 429053 TEST=gpu_unittests,webgl_conformance,webgl2_conformance TBR=piman@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1787503002 Cr-Commit-Position: refs/heads/master@{#380571}
* Disable DirectComposition on Windows <10jbauman2016-03-101-1/+16
| | | | | | | | | | | | | | | | | | On Windows 8 and 8.1 the DirectComposition content doesn't seem to appear until after the window does its initial show animation. However the GDI content behind it still updates, which causes the window to flash black for a few frames when it's first created. This seems to be fixed on Windows 10, but before that there doesn't seem to be a good workaround. So for now we'll disable DirectComposition on Windows 8 and 8.1. BUG=593274,588588 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1778593004 Cr-Commit-Position: refs/heads/master@{#380326}
* Get rid of gpu related switches by passing gpu::GpuPreferences via IPC.penghuang2016-03-103-28/+2
| | | | | | | | | | | | BUG=590825 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Committed: https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48 Cr-Commit-Position: refs/heads/master@{#379651} Review URL: https://codereview.chromium.org/1753383003 Cr-Commit-Position: refs/heads/master@{#380320}
* Handle UNPACK_SKIP_* and PACK_SKIP_* parameters on the client side.zmo2016-03-098-146/+171
| | | | | | | | | | | | | | | This is part of https://codereview.chromium.org/1750123002/ Land this separately because this might have an unexpected perf impact (see crbug.com/571885) BUG=487841, 429053 TEST=gpu_unittests,webgl_conformance,webgl2_conformance TBR=piman@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1785483002 Cr-Commit-Position: refs/heads/master@{#380260}
* mustash: SurfaceTexture* => gpu/ipc/commonfsamuel2016-03-096-0/+157
| | | | | | | | | | | | | | As part of the effort to move content/common/gpu => gpu/ipc to allow code reuse in Mus, I'm moving some android specific code: SurfaceTextureManager and SurfaceTexturePeer to gpu/ipc/common. BUG=586371 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1770073002 Cr-Commit-Position: refs/heads/master@{#380240}
* Don't detach SurfaceTexture from GL Context on Mali <= KitKat.liberato2016-03-092-0/+17
| | | | | | | | | | | | | | | | | Mali with JB doesn't support SurfaceTexture::DetachFromGLContext, so on those devices we don't defer attaching the SurfaceTexture in AVDA. This may cause additional GL context switches if virtual contexts aren't enabled. We also turn off copying the most recent frame to the PictureBuffer textures, since it also crashes on Mali <= KitKat. BUG=591600,591564 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1760313002 Cr-Commit-Position: refs/heads/master@{#380215}
* Added initial Vulkan build files as well as a basic implementation file.dyen2016-03-097-0/+328
| | | | | | | | | | | | | | | | | As a first step in experimenting with Vulkan in Chromium, this CL adds GN rules to build and link with Vulkan under the GN variable "enable_vulkan". Currently we require users to have the Vulkan SDK installed, so the changes will not link unless a user has properly installed the SDK. R=piman@chromium.org BUG=582558 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1726303002 Cr-Commit-Position: refs/heads/master@{#380210}
* Minor clean-up to gles2_cmd_copy_texture_chromium.cc.erikchen2016-03-091-2/+2
| | | | | | | | | | | | The GLSL attribute a_position was declared and used as a vec4, but being initialized as a vec2. BUG= CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1774803002 Cr-Commit-Position: refs/heads/master@{#379999}
* Revert of Get rid of gpu related switches by passing gpu::GpuPreferences via ↵simonhatch2016-03-093-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | IPC. (patchset #8 id:140001 of https://codereview.chromium.org/1753383003/ ) Reason for revert: Suspecting this is breaking a lot of perf bots. BUG=593045 Original issue's description: > Get rid of gpu related switches by passing gpu::GpuPreferences via IPC. > > BUG=590825 > CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel > > Committed: https://crrev.com/248b114c9d3034964c7259222712dfcccfad8f48 > Cr-Commit-Position: refs/heads/master@{#379651} TBR=piman@chromium.org,boliu@chromium.org,palmer@chromium.org,penghuang@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=590825 Review URL: https://codereview.chromium.org/1778693002 Cr-Commit-Position: refs/heads/master@{#379993}
* gpu: Allow passing waiting command buffer id to target command buffer.sunnyps2016-03-083-5/+95
| | | | | | | | | | | | | | | SyncPointClient can taken an optional callback that notifies its owner when a release callback is added. This happens when another client is waiting on a FenceSync. The callback is passed the namespace and id. This is to allow upgrading the priority of a stream when another stream is waiting on it and will be used by the upcoming GPU scheduler. BUG=514813 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1749093005 Cr-Commit-Position: refs/heads/master@{#379928}
* Update GL_IMPLEMENTATION_COLOR_READ_FORMAT for BGRAericrk2016-03-086-5/+25
| | | | | | | | | | | | | | | | GL_IMPLEMENTATION_COLOR_READ_FORMAT currently always returns RGBA for BGRA or RGBA buffers. To provide a more reasonable emulation of this ES feature on desktop, we should return BGRA readback for framebuffers whose internal format is BGRA. On GLES, this change has no impact, as the GL API is queried directly. BUG=581311,591152 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1759433002 Cr-Commit-Position: refs/heads/master@{#379890}
* Command buffer: generate correct GL error for invalid level for compressed ↵yunchao.he2016-03-081-2/+9
| | | | | | | | | | | | | | texture This change can fix some bugs in negativetextureapi.html in WebGL deqp. BUG=565347 TEST=deqp/functional/gles3/negativetextureapi.html CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1749183003 Cr-Commit-Position: refs/heads/master@{#379697}
* Get rid of gpu related switches by passing gpu::GpuPreferences via IPC.penghuang2016-03-073-28/+2
| | | | | | | | | BUG=590825 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1753383003 Cr-Commit-Position: refs/heads/master@{#379651}
* [gpu/config] Do not limit cubemaps to 1024 on OSX >= 10.10cwallez2016-03-071-2/+6
| | | | | | | | | | | | | | | This limitation was introduced for old hardware and drivers that had corruption issues with large cubemaps. We are removing it for newer drivers because OpenGL ES3 and WebGL2 require support for cubemaps at least up to 2048. BUG=587781 BUG=576219 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1766793002 Cr-Commit-Position: refs/heads/master@{#379571}
* Fix GL version strings in testsmartina.kollarova2016-03-074-5/+5
| | | | | | | | | | | | | | Some unittests set a version string like "opengl 3.2", which is incorrect since GL_VERSION should be major.minor[.release_number] on desktop GL. This caused the version parser to set the version to 0.0 without an error. Added DCHECK for this case. BUG=None CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1727603002 Cr-Commit-Position: refs/heads/master@{#379530}
* mustash: Move gpu/ipc to gpu/ipc/commonfsamuel2016-03-0515-126/+29
| | | | | | | | | | | Mus will use the gpu service and client code currently in content/common/gpu. BUG=586389 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1714903003 Cr-Commit-Position: refs/heads/master@{#379416}
* [gpu/config] on Linux AMD using ANGLE do not fallback to softwarecwallez2016-03-041-0/+3
| | | | | | | | | | | | Because of a missing exception ANGLE was considered as an old AMD drivers and was being blacklisted. Add an exception for ANGLE. BUG=585132 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1759343002 Cr-Commit-Position: refs/heads/master@{#379316}
* Conditionalize build of dEQP tests again.kbr2016-03-031-1/+2
| | | | | | | | | | | | | | Conditionalize both the build targets and their isolates. The last attempt to conditionalize just the bodies of the targets broke the official builders. BUG=586565 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel TBR=piman@chromium.org,thakis@chromium.org Review URL: https://codereview.chromium.org/1760023002 Cr-Commit-Position: refs/heads/master@{#378909}
* android: limit max texture size to 4096boliu2016-03-021-1/+11
| | | | | | | | | BUG=591239 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1761593002 Cr-Commit-Position: refs/heads/master@{#378882}
* Revert "Build dEQP tests only with build_angle_deqp_tests==1."kbr2016-03-021-74/+56
| | | | | | | | | | | | | | This reverts commits 69b3aebb6fcc494d201d428a7ae073a0f424471e and de3cd13d57602a4d17223145a996e69394486bee . BUG=591281, 586565 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel TBR=piman@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/1756673003 Cr-Commit-Position: refs/heads/master@{#378847}
* Refactor gles2_conform warning suppressions to apply to all platforms.kbr2016-03-021-25/+12
| | | | | | | | | | BUG=586565 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel TBR=piman@chromium.org Review URL: https://codereview.chromium.org/1757023002 Cr-Commit-Position: refs/heads/master@{#378805}
* Command buffer: buffer size is zero should be correct if width/height/depth ↵yunchao.he2016-03-022-4/+136
| | | | | | | | | | | | | | | | | is zero in CompressedTex{Sub}Image{2D|3D}. This change can fix some critical bugs (webgl context lost) in negativetextureapi.html in WebGL deqp. BUG=565347 TEST=deqp/functional/gles3/negativetextureapi.html CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1758493003 Cr-Commit-Position: refs/heads/master@{#378775}
* Fix angle_deqp_gles3_tests target dep.jmadill2016-03-021-1/+1
| | | | | | | | | | | | | | This target was pulling in ANGLE's GLES2 tests supporting deps, when it should be pulling in the GLES3 dep. This was causing the tests to fail on the GPU FYI bots and on the ANGLE trybots. BUG=586565 TBR=kbr@chromium.org,thakis@chromium.org,sievers@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1757943002 Cr-Commit-Position: refs/heads/master@{#378758}
* Move content:GPUVideo...Stats to gpu::Video...Stats. HolyJavaStyleNamingBatman!markdittmer2016-03-026-0/+172
| | | | | | | | | | | | | | | As a part of the GPU refactor for Mus, we are trying to eliminate spots where content/common/gpu depends on content/. One such dependency is content::GPUVideoMemoryUsageStats. BUG=586367 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Committed: https://crrev.com/6c1a70a5d0436c7c647c995202fbfa307f28983a Cr-Commit-Position: refs/heads/master@{#377036} Review URL: https://codereview.chromium.org/1708783002 Cr-Commit-Position: refs/heads/master@{#378757}
* Build dEQP tests only with build_angle_deqp_tests==1.kbr2016-03-021-56/+74
| | | | | | | | | | | | This allows the bots on which they're built to be selected more precisely. BUG=586565 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1737053003 Cr-Commit-Position: refs/heads/master@{#378680}
* gpu: Add out-of-line copy ctors for complex classes.vmpstr2016-03-0132-0/+72
| | | | | | | | | | | | | | This patch adds out of line copy constructors for classes that our clang-plugin considers heavy. This is an effort to enable copy constructor checks by default. BUG=436357 R=piman@chromium.org, dcheng@chromium.org, thakis@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1725113002 Cr-Commit-Position: refs/heads/master@{#378590}
* Correct the uniformbuffer name in commandbuffer service.xinghua.cao2016-03-013-81/+48
| | | | | | | | | | | BUG=577368 TEST=deqp/functional/gles3/uniformbuffers.html conformance2/buffers/uniform-buffers.html CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1634683005 Cr-Commit-Position: refs/heads/master@{#378489}
* Check that MediaCodec sends FORMAT_CHANGED before decoded buffers.liberato2016-03-011-1/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | It looks like MediaCodec erroneously returns decoded frames before sending an OUTPUT_FORMAT_CHANGED notification. This will cause us to SendDecodedFrameToClient, then crashes when it tries to get a picture buffer to use. MediaCodec is not supposed to do that, see EncodeDecodeTest.java line 617 in the android cts tests. Once it does, it's unclear what state it's in. We post an error and stop decoding. I do not have a local repro, so this is my best guess about what is causing some reported crashes. They seem to happen only on JB with Adreno 330 GPUs (Snapdragon 800 series), driver version 45. This combination has been added to the blacklist for accelerated video decoding (AVDA). This CL also adds a metric for how often this occurs, to help to identify other affected devices. BUG=585963 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1685303004 Cr-Commit-Position: refs/heads/master@{#378340}
* Use GpuPreferences to avoid directly accessing switches in gpu related codepenghuang2016-02-2930-124/+289
| | | | | | | | | BUG=586369 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1716813002 Cr-Commit-Position: refs/heads/master@{#378305}
* tracing: Make ConvertableToTraceFormat move-onlyprimiano2016-02-291-10/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Summary. This CL: - Makes TraceEvent ownership a move-only scoped_ptr. - Makes ConvertableToTraceFormat (CTTF) itself move-only scoped_ptr. - Updates all the codebase that uses CTTF in TRACE_EVENT macros to use move-only semantics. Background: Historically ConvertableToTraceFormat (CTTF) was RefCounted. The main reason seems to be supporting monitoring mode (now deprecated) where tracing needed to copy TraceEvents without flushing the TraceLog. Not what monitoring mode is gone, there is no reason why TraceEvent(s) should not be move-only. Unfortunately CTTF being RefCounted exposed that implementation detail to its public interface. Fortunately, most of the codebase doesn't care about the fact that CTTF is RefCounted. The only exceptions are: 1. Memory-infra heap profiler {StackFrame,TypeInfo}Deduplicator 2. cc::Layer DebugInfo 1) Is addressed creating a proxy class which delegates the CTTF methods to the duplicators inside MDSessionState. Essentially it makes the CTTF metadata events shared co-owners of the MDSessionState. 2) After an offline chat with danakj@, it seems OK to make DebugInfo(s) moved scoped_ptr (as opposite as copied), moving the ownership to the active layer and keeping a raw ptr into the pending layer. BUG=559117 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel TBR=thakis,jochen,tbarzic,mnaganov,skyostil Review URL: https://codereview.chromium.org/1717283003 Cr-Commit-Position: refs/heads/master@{#378263}
* command_buffer: Enable gl path rendering by defaultkkinnunen2016-02-299-48/+7
| | | | | | | | | | | | | | | | | Enable gl path rendering by default if it is supported by the hardware. Skia will use it when GPU rasterization MSAA codepath is used. BUG=344330 Committed: https://crrev.com/3f4b630297d744ce825a18f167550c10892e7e76 Cr-Commit-Position: refs/heads/master@{#371753} CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1626363002 Cr-Commit-Position: refs/heads/master@{#378207}
* Command buffer: we should check internalformat at first if it exists, then ↵yunchao.he2016-02-273-129/+146
| | | | | | | | | | | | | | check the combination. This change can fix some bugs in negativetextureapi.html in WebGL deqp BUG=565347 TEST=deqp/functional/gles3/negativetextureapi.html CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1731113004 Cr-Commit-Position: refs/heads/master@{#378119}
* Fix segfault in gpu_timingmartina.kollarova2016-02-261-1/+1
| | | | | | | | | | | | The extension GL_ARB_timer_query depends on GL 3.2, expecting that glGetInteger64v is available. BUG=583656 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1679163003 Cr-Commit-Position: refs/heads/master@{#377866}
* WebGL 2: Fix bugs for drawing when the type of vertex attrib is a packed type.yunchao.he2016-02-266-24/+32
| | | | | | | | | | | | | This small change fixed bugs in draw.html and vertexarrays.html in WebGL deqp for GL_INT_2_10_10_10_REV and GL_UNSIGNED_INT_2_10_10_10_REV. BUG=295792 TESTS=deqp/functional/gles3/draw.html, deqp/functional/gles3/vertexarrays.html CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1708983002 Cr-Commit-Position: refs/heads/master@{#377836}
* command buffer: fix the bug when check fbo status before reading.yunchao.he2016-02-262-15/+20
| | | | | | | | | | | | This change can fix some bugs in negativetextureapi.html in deqp BUG=565347 TEST=deqp/functional/gles3/negativetextureapi.html CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1726603002 Cr-Commit-Position: refs/heads/master@{#377808}
* gpu: Do not shrink level_infos vectorboliu2016-02-261-0/+2
| | | | | | | | | | | | | The vector is generated sized to the max levels. So TextureDefinitions should not be resizing the vector to 1. Instead, update level 0, and reset the other levels, if there are any. BUG=569959 CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel Review URL: https://codereview.chromium.org/1729153003 Cr-Commit-Position: refs/heads/master@{#377785}