Prefix remaining OS defines with FML_ namespace (#31172)

Author: bdero

flow/frame_timings_recorder_unittests.cc

@@ -122,7 +122,7 @@ TEST(FrameTimingsRecorderTest, RecordRasterTimesWithCache) {
 }
 
 // Windows and Fuchsia don't allow testing with killed by signal.
-#if !defined(OS_FUCHSIA) && !defined(OS_WIN) && \
+#if !defined(OS_FUCHSIA) && !defined(FML_OS_WIN) && \
     (FLUTTER_RUNTIME_MODE == FLUTTER_RUNTIME_MODE_DEBUG)
 
 TEST(FrameTimingsRecorderTest, ThrowWhenRecordBuildBeforeVsync) {

fml/backtrace.cc

@@ -11,7 +11,7 @@
 #include <csignal>
 #include <sstream>
 
-#if OS_WIN
+#if FML_OS_WIN
 #include <crtdbg.h>
 #include <debugapi.h>
 #endif
@@ -132,7 +132,7 @@ static void ToggleSignalHandlers(bool set) {
 }
 
 void InstallCrashHandler() {
-#if OS_WIN
+#if FML_OS_WIN
   if (!IsDebuggerPresent()) {
     _CrtSetReportMode(_CRT_ASSERT, _CRTDBG_MODE_FILE | _CRTDBG_MODE_DEBUG);
     _CrtSetReportFile(_CRT_ASSERT, _CRTDBG_FILE_STDERR);

fml/build_config.h

@@ -4,8 +4,8 @@
 
 // This file adds defines about the platform we're currently building on.
 //  Operating System:
-//    OS_WIN / FML_OS_MACOSX / FML_OS_LINUX / FML_OS_POSIX (MACOSX or LINUX) /
-//    FML_OS_NACL (NACL_SFI or NACL_NONSFI) / FML_OS_NACL_SFI /
+//    FML_OS_WIN / FML_OS_MACOSX / FML_OS_LINUX / FML_OS_POSIX (MACOSX or LINUX)
+//    / FML_OS_NACL (NACL_SFI or NACL_NONSFI) / FML_OS_NACL_SFI /
 //    FML_OS_NACL_NONSFI
 //  Compiler:
 //    COMPILER_MSVC / COMPILER_GCC
@@ -27,7 +27,7 @@
 #include <TargetConditionals.h>
 #define FML_OS_MACOSX 1
 #if defined(TARGET_OS_IPHONE) && TARGET_OS_IPHONE
-#define OS_IOS 1
+#define FML_OS_IOS 1
 #endif  // defined(TARGET_OS_IPHONE) && TARGET_OS_IPHONE
 #elif defined(__linux__)
 #define FML_OS_LINUX 1
@@ -38,7 +38,7 @@
 #define LIBC_GLIBC 1
 #endif
 #elif defined(_WIN32)
-#define OS_WIN 1
+#define FML_OS_WIN 1
 #elif defined(__FreeBSD__)
 #define FML_OS_FREEBSD 1
 #elif defined(__OpenBSD__)
@@ -51,10 +51,10 @@
 #error Please add support for your platform in flutter/fml/build_config.h
 #endif
 
-// For access to standard BSD features, use OS_BSD instead of a
+// For access to standard BSD features, use FML_OS_BSD instead of a
 // more specific macro.
 #if defined(FML_OS_FREEBSD) || defined(FML_OS_OPENBSD)
-#define OS_BSD 1
+#define FML_OS_BSD 1
 #endif
 
 // For access to standard POSIXish features, use FML_OS_POSIX instead of a

fml/eintr_wrapper.h

@@ -9,7 +9,7 @@
 
 #include "flutter/fml/build_config.h"
 
-#if defined(OS_WIN)
+#if defined(FML_OS_WIN)
 
 // Windows has no concept of EINTR.
 #define FML_HANDLE_EINTR(x) (x)
@@ -38,6 +38,6 @@
     eintr_wrapper_result;                                 \
   })
 
-#endif  // defined(OS_WIN)
+#endif  // defined(FML_OS_WIN)
 
 #endif  // FLUTTER_FML_EINTR_WRAPPER_H_

fml/file_unittest.cc

@@ -228,7 +228,7 @@ TEST(FileTest, CanStopVisitEarly) {
   ASSERT_TRUE(fml::UnlinkDirectory(dir.fd(), "a"));
 }
 
-#if OS_WIN
+#if FML_OS_WIN
 #define AtomicWriteTest DISABLED_AtomicWriteTest
 #else
 #define AtomicWriteTest AtomicWriteTest

fml/logging.cc

@@ -11,7 +11,7 @@
 
 #if defined(FML_OS_ANDROID)
 #include <android/log.h>
-#elif defined(OS_IOS)
+#elif defined(FML_OS_IOS)
 #include <syslog.h>
 #elif defined(OS_FUCHSIA)
 #include <lib/syslog/global.h>
@@ -94,7 +94,7 @@ LogMessage::~LogMessage() {
       break;
   }
   __android_log_write(priority, "flutter", stream_.str().c_str());
-#elif defined(OS_IOS)
+#elif defined(FML_OS_IOS)
   syslog(LOG_ALERT, "%s", stream_.str().c_str());
 #elif defined(OS_FUCHSIA)
   fx_log_severity_t fx_severity;

fml/mapping.h

@@ -82,7 +82,7 @@ class FileMapping final : public Mapping {
   uint8_t* mapping_ = nullptr;
   uint8_t* mutable_mapping_ = nullptr;
 
-#if OS_WIN
+#if FML_OS_WIN
   fml::UniqueFD mapping_handle_;
 #endif
 

fml/memory/thread_checker.h

@@ -12,7 +12,7 @@
 #include "flutter/fml/logging.h"
 #include "flutter/fml/macros.h"
 
-#if defined(OS_WIN)
+#if defined(FML_OS_WIN)
 #include <windows.h>
 #else
 #include <pthread.h>
@@ -30,7 +30,7 @@ namespace fml {
 // there's a small space cost to having even an empty class. )
 class ThreadChecker final {
  public:
-#if defined(OS_WIN)
+#if defined(FML_OS_WIN)
   ThreadChecker() : self_(GetCurrentThreadId()) {}
   ~ThreadChecker() {}
 

fml/message_loop_impl.cc

@@ -21,7 +21,7 @@
 #include "flutter/fml/platform/fuchsia/message_loop_fuchsia.h"
 #elif FML_OS_LINUX
 #include "flutter/fml/platform/linux/message_loop_linux.h"
-#elif OS_WIN
+#elif FML_OS_WIN
 #include "flutter/fml/platform/win/message_loop_win.h"
 #endif
 
@@ -36,7 +36,7 @@ fml::RefPtr<MessageLoopImpl> MessageLoopImpl::Create() {
   return fml::MakeRefCounted<MessageLoopFuchsia>();
 #elif FML_OS_LINUX
   return fml::MakeRefCounted<MessageLoopLinux>();
-#elif OS_WIN
+#elif FML_OS_WIN
   return fml::MakeRefCounted<MessageLoopWin>();
 #else
   return nullptr;

fml/message_loop_unittests.cc

@@ -18,7 +18,7 @@
 #include "gtest/gtest.h"
 
 #define TIMESENSITIVE(x) TimeSensitiveTest_##x
-#if OS_WIN
+#if FML_OS_WIN
 #define PLATFORM_SPECIFIC_CAPTURE(...) [ __VA_ARGS__, count ]
 #else
 #define PLATFORM_SPECIFIC_CAPTURE(...) [__VA_ARGS__]

fml/native_library.h

@@ -12,20 +12,20 @@
 #include "flutter/fml/memory/ref_counted.h"
 #include "flutter/fml/memory/ref_ptr.h"
 
-#if defined(OS_WIN)
+#if defined(FML_OS_WIN)
 #include <windows.h>
-#endif  // defined(OS_WIN)
+#endif  // defined(FML_OS_WIN)
 
 namespace fml {
 class NativeLibrary : public fml::RefCountedThreadSafe<NativeLibrary> {
  public:
-#if OS_WIN
+#if FML_OS_WIN
   using Handle = HMODULE;
   using SymbolHandle = FARPROC;
-#else   // OS_WIN
+#else   // FML_OS_WIN
   using Handle = void*;
   using SymbolHandle = void*;
-#endif  // OS_WIN
+#endif  // FML_OS_WIN
 
   static fml::RefPtr<NativeLibrary> Create(const char* path);
 

fml/paths.cc

@@ -19,11 +19,11 @@ std::string JoinPaths(std::initializer_list<std::string> components) {
     i++;
     stream << component;
     if (i != size) {
-#if OS_WIN
+#if FML_OS_WIN
       stream << "\\";
-#else   // OS_WIN
+#else   // FML_OS_WIN
       stream << "/";
-#endif  // OS_WIN
+#endif  // FML_OS_WIN
     }
   }
   return stream.str();

fml/synchronization/semaphore.cc

@@ -59,7 +59,7 @@ class PlatformSemaphore {
 
 }  // namespace fml
 
-#elif OS_WIN
+#elif FML_OS_WIN
 #include <windows.h>
 
 namespace fml {

fml/thread.cc

@@ -13,7 +13,7 @@
 #include "flutter/fml/message_loop.h"
 #include "flutter/fml/synchronization/waitable_event.h"
 
-#if defined(OS_WIN)
+#if defined(FML_OS_WIN)
 #include <windows.h>
 #elif defined(OS_FUCHSIA)
 #include <lib/zx/thread.h>
@@ -55,7 +55,7 @@ void Thread::Join() {
   thread_->join();
 }
 
-#if defined(OS_WIN)
+#if defined(FML_OS_WIN)
 // The information on how to set the thread name comes from
 // a MSDN article: http://msdn2.microsoft.com/en-us/library/xcb2z8hs.aspx
 const DWORD kVCThreadNameException = 0x406D1388;
@@ -75,7 +75,7 @@ void Thread::SetCurrentThreadName(const std::string& name) {
   pthread_setname_np(name.c_str());
 #elif defined(FML_OS_LINUX) || defined(FML_OS_ANDROID)
   pthread_setname_np(pthread_self(), name.c_str());
-#elif defined(OS_WIN)
+#elif defined(FML_OS_WIN)
   THREADNAME_INFO info;
   info.dwType = 0x1000;
   info.szName = name.c_str();

fml/unique_fd.cc

@@ -9,7 +9,7 @@
 namespace fml {
 namespace internal {
 
-#if OS_WIN
+#if FML_OS_WIN
 
 namespace os_win {
 
@@ -22,7 +22,7 @@ void UniqueFDTraits::Free_Handle(HANDLE fd) {
 
 }  // namespace os_win
 
-#else  // OS_WIN
+#else  // FML_OS_WIN
 
 namespace os_unix {
 
@@ -36,7 +36,7 @@ void UniqueDirTraits::Free(DIR* dir) {
 
 }  // namespace os_unix
 
-#endif  // OS_WIN
+#endif  // FML_OS_WIN
 
 }  // namespace internal
 }  // namespace fml

fml/unique_fd.h

@@ -8,20 +8,20 @@
 #include "flutter/fml/build_config.h"
 #include "flutter/fml/unique_object.h"
 
-#if OS_WIN
+#if FML_OS_WIN
 #include <windows.h>
 #include <map>
 #include <mutex>
 #include <optional>
-#else  // OS_WIN
+#else  // FML_OS_WIN
 #include <dirent.h>
 #include <unistd.h>
-#endif  // OS_WIN
+#endif  // FML_OS_WIN
 
 namespace fml {
 namespace internal {
 
-#if OS_WIN
+#if FML_OS_WIN
 
 namespace os_win {
 
@@ -69,7 +69,7 @@ struct UniqueFDTraits {
 
 }  // namespace os_win
 
-#else  // OS_WIN
+#else  // FML_OS_WIN
 
 namespace os_unix {
 
@@ -87,20 +87,20 @@ struct UniqueDirTraits {
 
 }  // namespace os_unix
 
-#endif  // OS_WIN
+#endif  // FML_OS_WIN
 
 }  // namespace internal
 
-#if OS_WIN
+#if FML_OS_WIN
 
 using UniqueFD = UniqueObject<HANDLE, internal::os_win::UniqueFDTraits>;
 
-#else  // OS_WIN
+#else  // FML_OS_WIN
 
 using UniqueFD = UniqueObject<int, internal::os_unix::UniqueFDTraits>;
 using UniqueDir = UniqueObject<DIR*, internal::os_unix::UniqueDirTraits>;
 
-#endif  // OS_WIN
+#endif  // FML_OS_WIN
 
 }  // namespace fml
 

lib/ui/painting/image_generator_registry.cc

@@ -10,7 +10,7 @@
 #include "third_party/skia/src/codec/SkCodecImageGenerator.h"
 #ifdef FML_OS_MACOSX
 #include "third_party/skia/include/ports/SkImageGeneratorCG.h"
-#elif OS_WIN
+#elif FML_OS_WIN
 #include "third_party/skia/include/ports/SkImageGeneratorWIC.h"
 #endif
 
@@ -32,7 +32,7 @@ ImageGeneratorRegistry::ImageGeneratorRegistry() : weak_factory_(this) {
             std::move(generator));
       },
       0);
-#elif OS_WIN
+#elif FML_OS_WIN
   AddFactory(
       [](sk_sp<SkData> buffer) {
         auto generator = SkImageGeneratorWIC::MakeFromEncodedWIC(buffer);

lib/ui/ui_dart_state.cc

@@ -13,7 +13,7 @@
 
 #if defined(FML_OS_ANDROID)
 #include <android/log.h>
-#elif defined(OS_IOS)
+#elif defined(FML_OS_IOS)
 extern "C" {
 // Cannot import the syslog.h header directly because of macro collision.
 extern void syslog(int, const char*, ...);
@@ -219,7 +219,7 @@ void UIDartState::LogMessage(const std::string& tag,
 #if defined(FML_OS_ANDROID)
     __android_log_print(ANDROID_LOG_INFO, tag.c_str(), "%.*s",
                         (int)message.size(), message.c_str());
-#elif defined(OS_IOS)
+#elif defined(FML_OS_IOS)
     std::stringstream stream;
     if (tag.size() > 0) {
       stream << tag << ": ";

runtime/dart_snapshot.cc

@@ -25,7 +25,7 @@ const char* DartSnapshot::kIsolateInstructionsSymbol =
 // data through symbols that are statically linked into the executable.
 // On other platforms this data is obtained by a dynamic symbol lookup.
 #define DART_SNAPSHOT_STATIC_LINK \
-  ((OS_WIN || FML_OS_ANDROID) && FLUTTER_JIT_RUNTIME)
+  ((FML_OS_WIN || FML_OS_ANDROID) && FLUTTER_JIT_RUNTIME)
 
 #if !DART_SNAPSHOT_STATIC_LINK