Revert "Delete mac_utils.h and mac_utils.cc"
This reverts commit ada8e17125d2124f5bcdc1558182ce95d6311d93. Reason for revert: Breaks chromium, due to undeclared dependency on SystemConfiguration.framework Original change's description: > Delete mac_utils.h and mac_utils.cc > > They defined two functions: ToUtf16 and ToUtf8. The former was unused, > and the latter is moved to > modules/desktop_capture/mac/window_list_utils.cc, the only user. > > Tbr: sergeyu@chromium.org > Bug: None > Change-Id: Ib8a513da42e43ba8d41a2de4c1645b3f48448dc9 > Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/148531 > Commit-Queue: Niels Moller <nisse@webrtc.org> > Reviewed-by: Kári Helgason <kthelgason@webrtc.org> > Reviewed-by: Sergey Ulanov <sergeyu@google.com> > Cr-Commit-Position: refs/heads/master@{#28913} TBR=zijiehe@chromium.org,nisse@webrtc.org,kthelgason@webrtc.org,sergeyu@google.com,sergeyu@chromium.org Change-Id: I9d6a2f63b3acde0eefab92d034529b800d6adcab No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: None Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/149811 Reviewed-by: Niels Moller <nisse@webrtc.org> Commit-Queue: Niels Moller <nisse@webrtc.org> Cr-Commit-Position: refs/heads/master@{#28915}
This commit is contained in:
@ -16,6 +16,7 @@
|
||||
|
||||
#include "modules/desktop_capture/mac/window_list_utils.h"
|
||||
#include "rtc_base/logging.h"
|
||||
#include "rtc_base/mac_utils.h"
|
||||
#include "rtc_base/time_utils.h"
|
||||
|
||||
namespace webrtc {
|
||||
|
@ -19,6 +19,7 @@
|
||||
#include <map>
|
||||
|
||||
#include "rtc_base/checks.h"
|
||||
#include "rtc_base/mac_utils.h"
|
||||
|
||||
static_assert(static_cast<webrtc::WindowId>(kCGNullWindowID) ==
|
||||
webrtc::kNullWindowId,
|
||||
@ -28,19 +29,6 @@ namespace webrtc {
|
||||
|
||||
namespace {
|
||||
|
||||
bool ToUtf8(const CFStringRef str16, std::string* str8) {
|
||||
size_t maxlen = CFStringGetMaximumSizeForEncoding(CFStringGetLength(str16),
|
||||
kCFStringEncodingUTF8) +
|
||||
1;
|
||||
std::unique_ptr<char[]> buffer(new char[maxlen]);
|
||||
if (!buffer ||
|
||||
!CFStringGetCString(str16, buffer.get(), maxlen, kCFStringEncodingUTF8)) {
|
||||
return false;
|
||||
}
|
||||
str8->assign(buffer.get());
|
||||
return true;
|
||||
}
|
||||
|
||||
// Get CFDictionaryRef from |id| and call |on_window| against it. This function
|
||||
// returns false if native APIs fail, typically it indicates that the |id| does
|
||||
// not represent a window. |on_window| will not be called if false is returned
|
||||
@ -259,7 +247,7 @@ std::string GetWindowTitle(CFDictionaryRef window) {
|
||||
CFStringRef title = reinterpret_cast<CFStringRef>(
|
||||
CFDictionaryGetValue(window, kCGWindowName));
|
||||
std::string result;
|
||||
if (title && ToUtf8(title, &result)) {
|
||||
if (title && rtc::ToUtf8(title, &result)) {
|
||||
return result;
|
||||
}
|
||||
|
||||
@ -280,7 +268,7 @@ std::string GetWindowOwnerName(CFDictionaryRef window) {
|
||||
CFStringRef owner_name = reinterpret_cast<CFStringRef>(
|
||||
CFDictionaryGetValue(window, kCGWindowOwnerName));
|
||||
std::string result;
|
||||
if (owner_name && ToUtf8(owner_name, &result)) {
|
||||
if (owner_name && rtc::ToUtf8(owner_name, &result)) {
|
||||
return result;
|
||||
}
|
||||
return std::string();
|
||||
|
@ -27,6 +27,7 @@
|
||||
#include "modules/desktop_capture/mac/full_screen_chrome_window_detector.h"
|
||||
#include "modules/desktop_capture/mac/window_list_utils.h"
|
||||
#include "modules/desktop_capture/mouse_cursor.h"
|
||||
#include "rtc_base/mac_utils.h"
|
||||
|
||||
namespace webrtc {
|
||||
|
||||
|
@ -27,6 +27,7 @@
|
||||
#include "modules/desktop_capture/window_finder_mac.h"
|
||||
#include "rtc_base/constructor_magic.h"
|
||||
#include "rtc_base/logging.h"
|
||||
#include "rtc_base/mac_utils.h"
|
||||
#include "rtc_base/trace_event.h"
|
||||
|
||||
namespace webrtc {
|
||||
|
@ -926,6 +926,19 @@ rtc_static_library("rtc_base") {
|
||||
]
|
||||
}
|
||||
|
||||
if (is_mac) {
|
||||
sources += [
|
||||
"mac_utils.cc",
|
||||
"mac_utils.h",
|
||||
]
|
||||
libs += [
|
||||
"Cocoa.framework",
|
||||
"Foundation.framework",
|
||||
"IOKit.framework",
|
||||
"Security.framework",
|
||||
"SystemConfiguration.framework",
|
||||
]
|
||||
}
|
||||
if (is_ios) {
|
||||
libs += [
|
||||
"CFNetwork.framework",
|
||||
|
48
rtc_base/mac_utils.cc
Normal file
48
rtc_base/mac_utils.cc
Normal file
@ -0,0 +1,48 @@
|
||||
/*
|
||||
* Copyright 2007 The WebRTC Project Authors. All rights reserved.
|
||||
*
|
||||
* Use of this source code is governed by a BSD-style license
|
||||
* that can be found in the LICENSE file in the root of the source
|
||||
* tree. An additional intellectual property rights grant can be found
|
||||
* in the file PATENTS. All contributing project authors may
|
||||
* be found in the AUTHORS file in the root of the source tree.
|
||||
*/
|
||||
|
||||
#include "rtc_base/mac_utils.h"
|
||||
|
||||
#include <sys/utsname.h>
|
||||
|
||||
#include <cstring>
|
||||
#include <memory>
|
||||
|
||||
#include "rtc_base/checks.h"
|
||||
#include "rtc_base/logging.h"
|
||||
|
||||
namespace rtc {
|
||||
|
||||
bool ToUtf8(const CFStringRef str16, std::string* str8) {
|
||||
if ((nullptr == str16) || (nullptr == str8)) {
|
||||
return false;
|
||||
}
|
||||
size_t maxlen = CFStringGetMaximumSizeForEncoding(CFStringGetLength(str16),
|
||||
kCFStringEncodingUTF8) +
|
||||
1;
|
||||
std::unique_ptr<char[]> buffer(new char[maxlen]);
|
||||
if (!buffer ||
|
||||
!CFStringGetCString(str16, buffer.get(), maxlen, kCFStringEncodingUTF8)) {
|
||||
return false;
|
||||
}
|
||||
str8->assign(buffer.get());
|
||||
return true;
|
||||
}
|
||||
|
||||
bool ToUtf16(const std::string& str8, CFStringRef* str16) {
|
||||
if (nullptr == str16) {
|
||||
return false;
|
||||
}
|
||||
*str16 = CFStringCreateWithBytes(kCFAllocatorDefault,
|
||||
reinterpret_cast<const UInt8*>(str8.data()),
|
||||
str8.length(), kCFStringEncodingUTF8, false);
|
||||
return nullptr != *str16;
|
||||
}
|
||||
} // namespace rtc
|
23
rtc_base/mac_utils.h
Normal file
23
rtc_base/mac_utils.h
Normal file
@ -0,0 +1,23 @@
|
||||
/*
|
||||
* Copyright 2007 The WebRTC Project Authors. All rights reserved.
|
||||
*
|
||||
* Use of this source code is governed by a BSD-style license
|
||||
* that can be found in the LICENSE file in the root of the source
|
||||
* tree. An additional intellectual property rights grant can be found
|
||||
* in the file PATENTS. All contributing project authors may
|
||||
* be found in the AUTHORS file in the root of the source tree.
|
||||
*/
|
||||
|
||||
#ifndef RTC_BASE_MAC_UTILS_H_
|
||||
#define RTC_BASE_MAC_UTILS_H_
|
||||
|
||||
#include <CoreFoundation/CoreFoundation.h>
|
||||
|
||||
#include <string>
|
||||
|
||||
namespace rtc {
|
||||
bool ToUtf8(const CFStringRef str16, std::string* str8);
|
||||
bool ToUtf16(const std::string& str8, CFStringRef* str16);
|
||||
} // namespace rtc
|
||||
|
||||
#endif // RTC_BASE_MAC_UTILS_H_
|
Reference in New Issue
Block a user