Add DACL and fix test for anonymous read-only memory

Add DACL and fix test for anonymous read-only memory

BUG=338538
R=rvargas@chromium.org,brettw@chromium.org
TBR=brettw@chromium.org

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

[jschuh, 2014-08-07 05:23:26]

Turns out I landed only half a fix last time. This change fixes the existing
test and adds a DACL to ensure the restrictions are enforced.

rvargas@ - for content
brettw@ - for an owners stamp once rvargas@ signs off

[rvargas (doing something else), 2014-08-07 18:24:02]

LGTM. Just a small suggestion for the test.

https://codereview.chromium.org/444323005/diff/1/base/memory/shared_memory_un...
File base/memory/shared_memory_unittest.cc (right):

https://codereview.chromium.org/444323005/diff/1/base/memory/shared_memory_un...
base/memory/shared_memory_unittest.cc:449: }
could we add a call to duplicate the handle with read access and verify that it
succeeds? (that would have prevented the failure of this test)

[jschuh, 2014-08-07 18:35:48]

https://codereview.chromium.org/444323005/diff/1/base/memory/shared_memory_un...
File base/memory/shared_memory_unittest.cc (right):

https://codereview.chromium.org/444323005/diff/1/base/memory/shared_memory_un...
base/memory/shared_memory_unittest.cc:449: }
On 2014/08/07 18:24:02, rvargas wrote:
> could we add a call to duplicate the handle with read access and verify that
it
> succeeds? (that would have prevented the failure of this test)

Not if it has the same bug. :P

I mean, certainly. ;)

[jschuh, 2014-08-07 18:48:13]

The CQ bit was checked by jschuh@chromium.org

[commit-bot: I haz the power, 2014-08-07 18:52:57]

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

[commit-bot: I haz the power, 2014-08-07 22:08:20]

Change committed as 288152