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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 8348001: include fcntl.h on OpenBSD too (Closed)

Created:
9 years, 2 months ago by Robert Nagy
Modified:
9 years, 2 months ago
CC:
chromium-reviews, darin-cc_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

include fcntl.h on OpenBSD too BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107095

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M webkit/plugins/npapi/plugin_lib_posix.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Robert Nagy
9 years, 2 months ago (2011-10-18 17:43:38 UTC) #1
Robert Nagy
9 years, 2 months ago (2011-10-18 17:45:08 UTC) #2
wtc
LGTM. <fcntl.h> is a standard Unix header. It is fine to include it on all ...
9 years, 2 months ago (2011-10-21 19:53:16 UTC) #3
Robert Nagy
On 2011/10/21 19:53:16, wtc wrote: > LGTM. <fcntl.h> is a standard Unix header. It is ...
9 years, 2 months ago (2011-10-25 07:33:34 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/robert.nagy@gmail.com/8348001/1
9 years, 2 months ago (2011-10-25 09:27:00 UTC) #5
commit-bot: I haz the power
9 years, 2 months ago (2011-10-25 10:44:57 UTC) #6 
Change committed as 107095

Powered by Google App Engine
This is Rietveld 408576698