Cleanup: Remove unused SharedMemory ctor.

Cleanup: Remove unused SharedMemory ctor.

BUG=345734
Committed: https://crrev.com/d3ede883000c85863bbbf7265bc4175d57ea3ec1
Cr-Commit-Position: refs/heads/master@{#365776}

[Lei Zhang, 2015-12-16 23:26:47]

thestig@chromium.org changed reviewers:
+ wfh@chromium.org

[Lei Zhang, 2015-12-16 23:26:48]

Since we started talking about dead code...

[Will Harris, 2015-12-17 05:18:56]

On 2015/12/16 23:26:48, Lei Zhang wrote:
> Since we started talking about dead code...

lgtm; I suppose compiler warnings are better than runtime warnings. Seems
DesktopSessionProxy::IpcSharedBufferCore could maybe use ShareReadOnlyToProcess
instead of what it currently does...?

[Lei Zhang, 2015-12-17 07:16:30]

On 2015/12/17 05:18:56, Will Harris wrote:
> On 2015/12/16 23:26:48, Lei Zhang wrote:
> > Since we started talking about dead code...
> 
> lgtm; I suppose compiler warnings are better than runtime warnings. Seems
> DesktopSessionProxy::IpcSharedBufferCore could maybe use
ShareReadOnlyToProcess
> instead of what it currently does...?

Isn't DesktopSessionProxy::IpcSharedBufferCore() trying to take a shared memory
handle, whereas ShareReadOnlyToProcess() is for creating a handle to hand out?

[Lei Zhang, 2015-12-17 07:16:34]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-12-17 07:17:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1529283002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1529283002/40001

[commit-bot: I haz the power, 2015-12-17 07:45:48]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-17 07:46:31]

Description was changed from

==========
Cleanup: Remove unused SharedMemory ctor.

BUG=345734
==========

to

==========
Cleanup: Remove unused SharedMemory ctor.

BUG=345734

Committed: https://crrev.com/d3ede883000c85863bbbf7265bc4175d57ea3ec1
Cr-Commit-Position: refs/heads/master@{#365776}
==========

[commit-bot: I haz the power, 2015-12-17 07:46:32]

Patchset 3 (id:??) landed as
https://crrev.com/d3ede883000c85863bbbf7265bc4175d57ea3ec1
Cr-Commit-Position: refs/heads/master@{#365776}