Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1704)

Issue 4517133: Add EGL context for Windows. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 11 months ago by bungeman
Modified:
12 years, 11 months ago
Reviewers:
bsalomon, reed1
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Fix whitespace and line length. #

Total comments: 1

Patch Set 3 : Update field names. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+218 lines, -2 lines) Patch
M gyp/utils.gyp View 1 2 3 chunks +17 lines, -0 lines 0 comments Download
M include/utils/SkEGLContext.h View 1 2 2 chunks +6 lines, -2 lines 0 comments Download
A src/utils/win/SkEGLContext_Win.cpp View 1 2 1 chunk +195 lines, -0 lines 0 comments Download

Messages

Total messages: 6
bungeman
12 years, 11 months ago (2011-06-02 14:38:21 UTC) #1
reed1
http://codereview.appspot.com/4517133/diff/2001/include/utils/SkEGLContext.h File include/utils/SkEGLContext.h (right): http://codereview.appspot.com/4517133/diff/2001/include/utils/SkEGLContext.h#newcode39 include/utils/SkEGLContext.h:39: #elif defined(SK_BUILD_FOR_WIN32) I know I didn't follow this in ...
12 years, 11 months ago (2011-06-02 14:54:37 UTC) #2
bungeman
Changed new field names to use f-prefix convention. We may wish to update existing fields ...
12 years, 11 months ago (2011-06-02 15:26:12 UTC) #3
reed1
LGTM quick, check it in before brian can complain!
12 years, 11 months ago (2011-06-02 15:28:10 UTC) #4
bungeman
Committed revision 1481.
12 years, 11 months ago (2011-06-02 15:38:53 UTC) #5
bungeman
12 years, 11 months ago (2011-06-02 15:40:25 UTC) #6

          
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b