chrome/common/multi_process_lock.h - Issue 8341052: share all the needed linux code with OpenBSD in chrome and content

Unified Diff: chrome/common/multi_process_lock.h

Issue 8341052: share all the needed linux code with OpenBSD in chrome and content (Closed) Base URL: http://git.chromium.org/chromium/src.git@master

Patch Set: Created 9 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: chrome/common/multi_process_lock.h

diff --git a/chrome/common/multi_process_lock.h b/chrome/common/multi_process_lock.h

index 24754f54bdc2de9c70d3a118d2ce7621b9e4a799..e9ccd18f353aa8328297ffe22c31a8258ab44849 100644

--- a/chrome/common/multi_process_lock.h

+++ b/chrome/common/multi_process_lock.h

@@ -6,6 +6,8 @@

#define CHROME_COMMON_MULTI_PROCESS_LOCK_H_

#pragma once

+#include "build/build_config.h"

#include <sys/types.h>

#include <string>

@@ -18,7 +20,11 @@ class MultiProcessLock {

// The length of a multi-process lock name is limited on Linux, so

Mark Mentovai 2011/10/26 23:24:28 This comment indicates that MULTI_PROCESS_LOCK_NAM

Robert Nagy 2011/10/31 15:38:38 I've done the modifications for the linux and the

Mark Mentovai 2011/10/31 15:47:40 Robert Nagy wrote:

// it is limited it on all platforms for consistency. This length does

// not include a terminator.

+#if defined(OS_OPENBSD)

+ static const size_t MULTI_PROCESS_LOCK_NAME_MAX_LEN = 102;

+#else

static const size_t MULTI_PROCESS_LOCK_NAME_MAX_LEN = 106;

+#endif

// Factory method for creating a multi-process lock.

// |name| is the name of the lock. The name has special meaning on Windows

« no previous file with comments | « chrome/common/chrome_paths.cc ('k') | chrome/common/url_constants.h » ('j') | chrome/test/automation/proxy_launcher.cc » ('J')