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

Issue 271303002: ozone: Create a separate small gfx_ozone target. (Closed)

Created:
6 years, 7 months ago by sadrul
Modified:
6 years, 7 months ago
Reviewers:
rjkroege, sky
CC:
chromium-reviews, rjkroege, kalyank, ozone-reviews_chromium.org
Visibility:
Public.

Description

ozone: Create a separate small gfx_ozone target. Instead of bundling up the ozone stuff into the larger gfx.gyp:gfx target, split up the ozone-related code into a smaller gfx_ozone target with a small dependency (base, skia). This allows ozone-platforms to implement the ozone interface without bringing along the rest of gfx. BUG=361137

Patch Set 1 #

Patch Set 2 : . #

Patch Set 3 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -8 lines) Patch
M ui/gfx/gfx.gyp View 1 1 chunk +0 lines, -8 lines 0 comments Download
A ui/gfx/ozone/gfx_ozone.gyp View 1 2 1 chunk +31 lines, -0 lines 0 comments Download
M ui/ozone/ozone.gyp View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
sadrul
sky@ for ui/gfx rjkroege@ for ui/gfx/ozone and ui/ozone/
6 years, 7 months ago (2014-05-09 21:46:16 UTC) #1
sky
LGTM
6 years, 7 months ago (2014-05-09 22:06:29 UTC) #2
rjkroege
lgtm
6 years, 7 months ago (2014-05-09 22:08:10 UTC) #3
sadrul
6 years, 7 months ago (2014-05-10 12:02:57 UTC) #4
Committed in r269570

Powered by Google App Engine
This is Rietveld 408576698