add support for using system jpeg headers here for OpenBSD

add support for using system jpeg headers here for OpenBSD


BUG=
TEST=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=106127

[Robert Nagy, 2011-10-18 18:00:29]

Please review the following OpenBSD patches.

Thanks

[Mark Mentovai, 2011-10-18 18:10:57]

http://codereview.chromium.org/8348003/diff/1/third_party/iccjpeg/iccjpeg.h
File third_party/iccjpeg/iccjpeg.h (right):

http://codereview.chromium.org/8348003/diff/1/third_party/iccjpeg/iccjpeg.h#n...
third_party/iccjpeg/iccjpeg.h:21: #if defined(USE_SYSTEM_LIBJPEG)
You need to edit README.chromium if you make a change like this to source that’s
imported from elsewhere.

[Robert Nagy, 2011-10-18 18:27:12]

http://codereview.chromium.org/8348003/diff/1/third_party/iccjpeg/iccjpeg.h
File third_party/iccjpeg/iccjpeg.h (right):

http://codereview.chromium.org/8348003/diff/1/third_party/iccjpeg/iccjpeg.h#n...
third_party/iccjpeg/iccjpeg.h:21: #if defined(USE_SYSTEM_LIBJPEG)
On 2011/10/18 18:10:58, Mark Mentovai wrote:
> You need to edit README.chromium if you make a change like this to source
that’s
> imported from elsewhere.

Done.

[Robert Nagy, 2011-10-18 18:27:48]

On 2011/10/18 18:27:12, Robert Nagy wrote:
> http://codereview.chromium.org/8348003/diff/1/third_party/iccjpeg/iccjpeg.h
> File third_party/iccjpeg/iccjpeg.h (right):
> 
>
http://codereview.chromium.org/8348003/diff/1/third_party/iccjpeg/iccjpeg.h#n...
> third_party/iccjpeg/iccjpeg.h:21: #if defined(USE_SYSTEM_LIBJPEG)
> On 2011/10/18 18:10:58, Mark Mentovai wrote:
> > You need to edit README.chromium if you make a change like this to source
> that’s
> > imported from elsewhere.
> 
> Done.

I've also had to add a security critical section to the readme because the
presubmit check errored out.

[Mark Mentovai, 2011-10-18 18:29:39]

http://codereview.chromium.org/8348003/diff/1003/third_party/iccjpeg/README.c...
File third_party/iccjpeg/README.chromium (right):

http://codereview.chromium.org/8348003/diff/1003/third_party/iccjpeg/README.c...
third_party/iccjpeg/README.chromium:17: the system, so a USE_SYSTEM_LIBJPEG
ifdef is added to decide which one
You said how you modified it, but you didn’t say what file you modified.

[Robert Nagy, 2011-10-18 18:32:22]

http://codereview.chromium.org/8348003/diff/1003/third_party/iccjpeg/README.c...
File third_party/iccjpeg/README.chromium (right):

http://codereview.chromium.org/8348003/diff/1003/third_party/iccjpeg/README.c...
third_party/iccjpeg/README.chromium:17: the system, so a USE_SYSTEM_LIBJPEG
ifdef is added to decide which one
On 2011/10/18 18:29:39, Mark Mentovai wrote:
> You said how you modified it, but you didn’t say what file you modified.

Done.

[Mark Mentovai, 2011-10-18 18:33:47]

LGTM

[commit-bot: I haz the power, 2011-10-18 18:34:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robert.nagy@gmail.com/8348003/6001

[commit-bot: I haz the power, 2011-10-18 20:57:39]

Change committed as 106127