Fix OS_MACOS typos. Should be OS_MACOSX.

Fix OS_MACOS typos. Should be OS_MACOSX.

BUG=163208
TEST=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=189130

[DaleCurtis, 2013-03-13 23:03:04]

brettw: Looks like you have the honor of covering OWNERS for all these.

[Dan Beam, 2013-03-13 23:06:56]

A presubmit check for this might be handy.  I've done a CL or two previously to
clean these up as well.  (inb4 "less presubmits")

[brettw, 2013-03-14 04:25:30]

lgtm

[commit-bot: I haz the power, 2013-03-14 17:51:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/12829005/1

[DaleCurtis, 2013-03-14 17:56:18]

dbeam: I don't see a global presubmit class of error which means "Did you really
mean to do this instead?" I'm happy to add one if that's a path we want to go
down.  maruel: WDYT?

[Dan Beam, 2013-03-14 18:23:34]

I think the header sorting one is a pretty good example of something we know the
right-ish solution for and prompt a user at upload about (and probably stop a CQ
on).

[DaleCurtis, 2013-03-14 18:24:36]

Doh, compile is failing on Mac, looks like the SandboxInit wants some arguments.
+jeremy who added this code in http://crrev.com/173432 for
http://crbug.com/163208

https://code.google.com/p/chromium/codesearch#chromium/src/content/public/com...

jeremy can you point out the correct arguments here (or would prefer to take
over the fix in that file)?

[DaleCurtis, 2013-03-14 18:24:58]

Whoops, actually +jeremy this time.

[jeremy, 2013-03-19 10:06:39]

https://codereview.chromium.org/12829005/diff/1/content/worker/worker_main.cc
File content/worker/worker_main.cc (right):

https://codereview.chromium.org/12829005/diff/1/content/worker/worker_main.cc...
content/worker/worker_main.cc:46: #elif defined(OS_MACOSX)
Per separate email, this should be fixed on trunk.

[DaleCurtis, 2013-03-19 17:45:33]

Thanks, rebased!

[commit-bot: I haz the power, 2013-03-19 17:48:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/12829005/20001

[DaleCurtis, 2013-03-19 19:36:49]

https://codereview.chromium.org/12829005/diff/20001/chrome/browser/ui/browser...
File chrome/browser/ui/browser_command_controller_unittest.cc (right):

https://codereview.chromium.org/12829005/diff/20001/chrome/browser/ui/browser...
chrome/browser/ui/browser_command_controller_unittest.cc:258: #if
defined(OS_MACOSX)
+jamescook this test is failing on the trybots now. What is the correct
expectation for this test?

[James Cook, 2013-03-19 20:21:01]

https://codereview.chromium.org/12829005/diff/20001/chrome/browser/ui/browser...
File chrome/browser/ui/browser_command_controller_unittest.cc (right):

https://codereview.chromium.org/12829005/diff/20001/chrome/browser/ui/browser...
chrome/browser/ui/browser_command_controller_unittest.cc:258: #if
defined(OS_MACOSX)
On 2013/03/19 19:36:50, DaleCurtis wrote:
> +jamescook this test is failing on the trybots now. What is the correct
> expectation for this test?

My mistake, sorry. The test expectations should be exactly the same as the
Windows/GTK section below. This whole section can be deleted.

[DaleCurtis, 2013-03-19 20:26:37]

https://codereview.chromium.org/12829005/diff/20001/chrome/browser/ui/browser...
File chrome/browser/ui/browser_command_controller_unittest.cc (right):

https://codereview.chromium.org/12829005/diff/20001/chrome/browser/ui/browser...
chrome/browser/ui/browser_command_controller_unittest.cc:258: #if
defined(OS_MACOSX)
On 2013/03/19 20:21:01, James Cook (Chromium) wrote:
> On 2013/03/19 19:36:50, DaleCurtis wrote:
> > +jamescook this test is failing on the trybots now. What is the correct
> > expectation for this test?
> 
> My mistake, sorry. The test expectations should be exactly the same as the
> Windows/GTK section below. This whole section can be deleted.

Done.

[James Cook, 2013-03-19 20:28:40]

lgtm

[DaleCurtis, 2013-03-19 22:07:04]

Committed patchset #3 manually as r189130 (presubmit successful).