Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(124)

Issue 2282733003: Update ANGLE to latest as of August 25, 2016 (Closed)

Created:
4 years, 3 months ago by Brian Osman
Modified:
4 years, 3 months ago
CC:
reviews_skia.org
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Update ANGLE to latest as of August 25, 2016 With this version, it's now exposing GL_EXT_color_buffer_half_float, but it's (incorrectly) using GL_HALF_FLOAT rather than GL_HALF_FLOAT_OES. In addition, release builds were crashing in EGLImageTest due to an apparent disagreement about calling convention on function-pointer to exported symbol. There is a proper typedef for that in one of their headers, but I can't seem to include it without creating more problems, so I just fixed the decoration on our local typedef. With those changes, all tests pass, and (on my Windows machine), all GMs and SKPs produce identical results vs. master in angle and angle-gl. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2282733003 Committed: https://skia.googlesource.com/skia/+/e3031b607c7816980d13a3805382afb403fdfa04

Patch Set 1 #

Patch Set 2 : Fix crash. #

Total comments: 3

Patch Set 3 : Add comment with link to bug #

Patch Set 4 : Fix tab vs spaces #

Patch Set 5 : Rebase? #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -4 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M gyp/common_variables.gypi View 1 2 3 1 chunk +1 line, -0 lines 0 comments Download
M src/gpu/gl/GrGLCaps.cpp View 1 2 2 chunks +6 lines, -2 lines 0 comments Download
M tools/gpu/gl/angle/GLTestContext_angle.cpp View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 19 (13 generated)
Brian Osman
Took a look at latest ANGLE. After a few issues, it's all working (and it ...
4 years, 3 months ago (2016-08-26 15:34:24 UTC) #5
robertphillips
lgtm https://codereview.chromium.org/2282733003/diff/20001/gyp/common_variables.gypi File gyp/common_variables.gypi (right): https://codereview.chromium.org/2282733003/diff/20001/gyp/common_variables.gypi#newcode77 gyp/common_variables.gypi:77: 'skia_egl%': '<(skia_egl)', over too far ? https://codereview.chromium.org/2282733003/diff/20001/src/gpu/gl/GrGLCaps.cpp File ...
4 years, 3 months ago (2016-08-26 15:43:58 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2282733003/60001
4 years, 3 months ago (2016-08-26 15:58:03 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: Build-Mac-Clang-x86_64-Release-GN-Trybot on master.client.skia.compile (JOB_FAILED, http://build.chromium.org/p/client.skia.compile/builders/Build-Mac-Clang-x86_64-Release-GN-Trybot/builds/923)
4 years, 3 months ago (2016-08-26 16:00:53 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2282733003/80001
4 years, 3 months ago (2016-08-26 17:29:05 UTC) #17
commit-bot: I haz the power
4 years, 3 months ago (2016-08-26 17:38:01 UTC) #19
Message was sent while issue was closed.
Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/e3031b607c7816980d13a3805382afb403fdfa04

Powered by Google App Engine
This is Rietveld 408576698