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

Issue 2794183002: ui: dmabuf: match linux header define names

Created:
3 years, 8 months ago by llandwerlin
Modified:
3 years, 8 months ago
Reviewers:
danakj
CC:
chromium-reviews, tonikitoo, msisov, fwang
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

ui: dmabuf: match linux header define names Let's use the same names as we only define them if they're missing them. BUG=707604

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -16 lines) Patch
M ui/gfx/linux/client_native_pixmap_dmabuf.cc View 2 chunks +14 lines, -16 lines 1 comment Download

Messages

Total messages: 17 (14 generated)
llandwerlin
danakj: PTAL, thanks!
3 years, 8 months ago (2017-04-04 00:41:52 UTC) #11
danakj
https://codereview.chromium.org/2794183002/diff/1/ui/gfx/linux/client_native_pixmap_dmabuf.cc File ui/gfx/linux/client_native_pixmap_dmabuf.cc (right): https://codereview.chromium.org/2794183002/diff/1/ui/gfx/linux/client_native_pixmap_dmabuf.cc#newcode45 ui/gfx/linux/client_native_pixmap_dmabuf.cc:45: struct dma_buf_sync sync_start = {0}; I'm confused, how did ...
3 years, 8 months ago (2017-04-04 15:46:22 UTC) #16
danakj
3 years, 8 months ago (2017-04-07 14:47:38 UTC) #17
It looks like https://codereview.chromium.org/2805503003/ is doing the same
thing.

Powered by Google App Engine
This is Rietveld 408576698