Revert "Base cleanups"

base/BUILD.gn

@@ -2773,6 +2773,10 @@ component("i18n") {
     "i18n/utf8_validator_tables.h",
   ]
   defines = [ "BASE_I18N_IMPLEMENTATION" ]
+  defines += [
+    "U_COMMON_IMPLEMENTATION",
+    "UCONFIG_NO_CONVERSION=0",
+  ]
   configs += [ "//build/config/compiler:wexit_time_destructors" ]
   public_deps = [
     "//third_party/ced",
@@ -3640,6 +3644,8 @@ test("base_unittests") {
     "//third_party/modp_b64",
   ]
 
+  defines = [ "U_COMMON_IMPLEMENTATION" ]
+
   data_deps = [
   ]
 

base/allocator/partition_allocator/partition_alloc_base/rand_util_posix.cc

@@ -69,7 +69,8 @@ namespace partition_alloc::internal::base {
 // (https://chromium-review.googlesource.com/c/chromium/src/+/1545096) and land
 // it or some form of it.
 void RandBytes(void* output, size_t output_length) {
-#if BUILDFLAG(IS_LINUX) || BUILDFLAG(IS_CHROMEOS)
+// TODO(b/298237462): Clean this up for Cobalt or don't use PA.
+#if 0
   // We have to call `getrandom` via Linux Syscall Support, rather than through
   // the libc wrapper, because we might not have an up-to-date libc (e.g. on
   // some bots).

base/check_is_test_unittest.cc

@@ -5,11 +5,13 @@
 #include "base/check_is_test.h"
 
 #include "testing/gtest/include/gtest/gtest.h"
+#include "base/test/allow_check_is_test_for_testing.h"
 
 // Note: `base::AllowCheckIsTestForTesting` is being called in
 // `base/test/launcher/unit_test_launcher.cc` before this test is run.
 //
 // Thus, `CHECK_IS_TEST()` will succeed.
 TEST(CheckIsTest, Usage) {
+  base::test::AllowCheckIsTestForTesting();
   CHECK_IS_TEST();
 }

base/i18n/time_formatting.cc

@@ -260,11 +260,10 @@ std::u16string DateIntervalFormat(const Time& begin_time,
   UErrorCode status = U_ZERO_ERROR;
 
   std::unique_ptr<icu::DateIntervalFormat> formatter(
-#if defined(USE_HACKY_COBALT_CHANGES)
-      icu::DateIntervalFormat::createInstance(icu::UnicodeString(DateFormatToString(format)),
-#else
+/* Cobalt
       icu::DateIntervalFormat::createInstance(DateFormatToString(format),
-#endif
+Cobalt */
+      icu::DateIntervalFormat::createInstance(icu::UnicodeString(DateFormatToString(format)),
                                               status));
 
   icu::FieldPosition pos = 0;

base/test/run_all_unittests.cc

@@ -11,13 +11,11 @@
 
 
 #if defined(STARBOARD)
-#include "base/test/allow_check_is_test_for_testing.h"
 #include "starboard/client_porting/wrap_main/wrap_main.h"
 
 
 int TestSuiteRun(int argc, char** argv) {
   base::AtExitManager exit_manager;
-  base::test::AllowCheckIsTestForTesting();
   return base::TestSuite(argc, argv).Run();
 }
 

net/base/network_change_notifier_linux.cc

@@ -30,10 +30,16 @@ class NetworkChangeNotifierLinux::BlockingThreadObjects {
   // Plumbing for NetworkChangeNotifier::GetCurrentConnectionType.
   // Safe to call from any thread.
   NetworkChangeNotifier::ConnectionType GetCurrentConnectionType() {
+/* Cobalt
     return address_tracker_.GetCurrentConnectionType();
+Cobalt */
+    return NetworkChangeNotifier::ConnectionType::CONNECTION_UNKNOWN;
   }
 
+/* Cobalt
   internal::AddressTrackerLinux* address_tracker() { return &address_tracker_; }
+Cobalt */
+  internal::AddressTrackerLinux* address_tracker() { return nullptr; }
 
   // Begin watching for netlink changes.
   void Init();
@@ -44,14 +50,17 @@ class NetworkChangeNotifierLinux::BlockingThreadObjects {
   void OnIPAddressChanged();
   void OnLinkChanged();
   // Used to detect online/offline state and IP address changes.
+/* Cobalt
   internal::AddressTrackerLinux address_tracker_;
+Cobalt */
   NetworkChangeNotifier::ConnectionType last_type_ =
       NetworkChangeNotifier::CONNECTION_NONE;
 };
 
 NetworkChangeNotifierLinux::BlockingThreadObjects::BlockingThreadObjects(
     const std::unordered_set<std::string>& ignored_interfaces,
     scoped_refptr<base::SequencedTaskRunner> blocking_thread_runner)
+/* Cobalt
     : address_tracker_(
           base::BindRepeating(&NetworkChangeNotifierLinux::
                                   BlockingThreadObjects::OnIPAddressChanged,
@@ -62,15 +71,20 @@ NetworkChangeNotifierLinux::BlockingThreadObjects::BlockingThreadObjects(
           base::DoNothing(),
           ignored_interfaces,
           std::move(blocking_thread_runner)) {}
+Cobalt */  {}
 
 void NetworkChangeNotifierLinux::BlockingThreadObjects::Init() {
+/* Cobalt
   address_tracker_.Init();
+Cobalt */
   last_type_ = GetCurrentConnectionType();
 }
 
 void NetworkChangeNotifierLinux::BlockingThreadObjects::InitForTesting(
     base::ScopedFD netlink_fd) {
+/* Cobalt
   address_tracker_.InitWithFdForTesting(std::move(netlink_fd));  // IN-TEST
+Cobalt */
   last_type_ = GetCurrentConnectionType();
 }
 

starboard/build/install/install_target.gni

@@ -40,6 +40,9 @@ template("install_target") {
   copy("copy_" + target_name) {
     forward_variables_from(invoker, [ "testonly" ])
 
+    # deps = []
+    # not_needed(invoker, [ "deps" ])
+
     deps = invoker.deps
     deps += [ installable_target_dep ]
     sources = [ "$root_out_dir/$source_name" ]

starboard/client_porting/eztime/BUILD.gn

@@ -43,7 +43,7 @@ target(gtest_target_type, "eztime_test") {
   ]
 
   data_deps = [
-    "//cobalt/network:copy_ssl_certificates",
+    # "//cobalt/network:copy_ssl_certificates",
     "//third_party/icu:icudata",
   ]
 }

starboard/elf_loader/BUILD.gn

@@ -119,7 +119,7 @@ if (current_toolchain == starboard_toolchain && !is_host_win) {
       deps += [ "//third_party/crashpad/crashpad/wrapper:wrapper_stub" ]
     }
     if (sb_is_evergreen_compatible && sb_evergreen_compatible_package) {
-      deps += [ "//starboard/loader_app:copy_crashpad_handler_named_as_so" ]
+      # deps += [ "//starboard/loader_app:copy_crashpad_handler_named_as_so" ]
     }
   }
 }

starboard/extension/BUILD.gn

@@ -14,7 +14,8 @@
 
 target(gtest_target_type, "extension_test") {
   testonly = true
-  has_pedantic_warnings = true
+
+  # has_pedantic_warnings = true
   sources = [
     "enhanced_audio_test.cc",
     "extension_test.cc",

starboard/linux/shared/BUILD.gn

@@ -446,7 +446,7 @@ static_library("starboard_platform_sources") {
 
 if (current_toolchain == starboard_toolchain) {
   target(starboard_level_gtest_target_type, "starboard_platform_tests") {
-    build_loader = false
+    # build_loader = false
     testonly = true
 
     sources = media_tests_sources + player_tests_sources + [

starboard/linux/x64x11/platform_configuration/BUILD.gn

@@ -13,6 +13,17 @@
 # limitations under the License.
 
 config("platform_configuration") {
+  libs = [
+    "X11",
+    "Xcomposite",
+    "Xrender",
+  ]
+  libs += [
+    "asound",
+    "dl",
+    "pthread",
+    "rt",
+  ]
   configs = [
     "//starboard/build/config/sabi",
     "//starboard/linux/x64x11/shared/platform_configuration",

starboard/nplb/BUILD.gn

@@ -320,7 +320,7 @@ target(gtest_target_type, "nplb") {
   }
 
   data_deps = [
-    "//cobalt/network:copy_ssl_certificates",
+    # "//cobalt/network:copy_ssl_certificates",
     "//starboard/nplb/testdata/file_tests:nplb_file_tests_data",
     "//starboard/shared/starboard/player:player_download_test_data",
     "//third_party/icu:icudata",

starboard/nplb/nplb_evergreen_compat_tests/BUILD.gn

@@ -34,7 +34,7 @@ target(gtest_target_type, "nplb_evergreen_compat_tests") {
   ]
 
   data_deps = [
-    "//cobalt/network:copy_ssl_certificates",
+    # "//cobalt/network:copy_ssl_certificates",
     "//third_party/icu:icudata",
   ]
 }

starboard/raspi/shared/BUILD.gn

@@ -22,7 +22,7 @@ group("starboard_platform") {
 static_library("starboard_platform_sources") {
   check_includes = false
 
-  has_pedantic_warnings = true
+  # has_pedantic_warnings = true
 
   sources = [
     "//starboard/linux/shared/atomic_public.h",

starboard/shared/starboard/player/filter/tools/BUILD.gn

@@ -14,7 +14,7 @@
 
 if (current_toolchain == starboard_toolchain) {
   target(starboard_level_final_executable_type, "audio_dmp_player") {
-    build_loader = false
+    # build_loader = false
     sources = [ "audio_dmp_player.cc" ]
     configs += [ "//starboard/build/config:starboard_implementation" ]
     public_deps = [
@@ -24,7 +24,7 @@ if (current_toolchain == starboard_toolchain) {
       "//starboard/shared/starboard/player:video_dmp",
     ]
     data_deps = [
-      "//cobalt/network:copy_ssl_certificates",
+      # "//cobalt/network:copy_ssl_certificates",
       "//starboard/shared/starboard/player:player_download_test_data",
       "//third_party/icu:icudata",
     ]

starboard/win/shared/BUILD.gn

@@ -45,7 +45,7 @@ config("starboard_platform_config") {
 }
 
 static_library("starboard_platform") {
-  has_pedantic_warnings = true
+  # has_pedantic_warnings = true
 
   sources = [
     "//starboard/common/thread.cc",

testing/perf/BUILD.gn

@@ -1 +1 @@
-group("perf") {}
+group("perf") {}

third_party/angle/BUILD.gn

@@ -254,7 +254,7 @@ angle_static_library("preprocessor") {
   sources = angle_preprocessor_sources
 
   if (is_starboard) {
-    suppressed_configs += [ "//starboard/build/config:warnings_as_errors" ]
+    # suppressed_configs += [ "//starboard/build/config:warnings_as_errors" ]
   }
 
   public_deps = [
@@ -518,7 +518,7 @@ angle_static_library("translator") {
   public_configs += [ ":external_config" ]
 
   if (is_starboard) {
-    suppressed_configs += [ "//starboard/build/config:warnings_as_errors" ]
+    # suppressed_configs += [ "//starboard/build/config:warnings_as_errors" ]
   }
 
   deps = [

third_party/boringssl/BUILD.gn

@@ -251,6 +251,7 @@ if (!use_cobalt_customizations) {
     configs -= [ "//starboard/build/config:size" ]
     configs += [ "//starboard/build/config:speed" ]
 
+    # sources += boringssl_linux_x86_64_files
     if (sb_is_modular) {
       calling_convention = sabi_variables.calling_convention
       if ((calling_convention != "aarch64" && calling_convention != "eabi" &&
@@ -500,3 +501,10 @@ if (build_with_chromium) {
     seed_corpus = "src/fuzz/server_corpus_no_fuzzer_mode"
   }
 }
+
+# group("boringssl") {
+#   all_dependent_configs = [ ":external_config" ]
+#   public_deps = [
+#   ":crypto"
+#   ]
+# }

third_party/crashpad/crashpad/client/BUILD.gn

@@ -93,7 +93,7 @@ static_library("client") {
   deps = []
 
   if (crashpad_is_in_starboard) {
-    public_deps += [ "//starboard/elf_loader:evergreen_info" ]
+    # public_deps += [ "//starboard/elf_loader:evergreen_info" ]
   }
   if (crashpad_is_in_starboard || crashpad_is_in_native_target_build) {
     deps += [ "../wrapper/proto:crashpad_annotations_proto" ]

third_party/icu/source/common/ubrk.cpp

@@ -93,6 +93,7 @@ ubrk_open(UBreakIteratorType type,
 //                       Invokes the rule builder.
 //
 //------------------------------------------------------------------------------
+// static_assert(sizeof(UChar) == 0, "The type T must be an integral type!");
 U_CAPI UBreakIterator* U_EXPORT2
 ubrk_openRules(  const UChar        *rules,
                        int32_t       rulesLength,

third_party/icu/source/common/unicode/unistr.h

@@ -3153,7 +3153,7 @@ class U_COMMON_API UnicodeString : public Replaceable
    */
   inline UnicodeString(std::nullptr_t buffer, int32_t buffLength, int32_t buffCapacity);
 
-#if U_CHARSET_IS_UTF8 || !UCONFIG_NO_CONVERSION
+#if U_CHARSET_IS_UTF8 || !UCONFIG_NO_CONVERSION || 1
 
   /**
    * char* constructor.

third_party/libxml/BUILD.gn

@@ -349,4 +349,4 @@ if (!use_cobalt_customizations) {
   include_dirs = [ "$os_include" ]
 }
 
-group("xml_reader") {}
+group("xml_reader") {}

third_party/llvm-project/libunwind/BUILD.gn

@@ -142,6 +142,6 @@ static_library("unwind_starboard") {
 
   public_deps = [
     "//starboard/common",
-    "//starboard/elf_loader:evergreen_info",
+    # "//starboard/elf_loader:evergreen_info",
   ]
 }

third_party/zlib/BUILD.gn

@@ -389,7 +389,7 @@ component("zlib_hidden") {
   } else {
     sources += [
       "arm_stub.c",
-      "inflate.c",
+      # "inflate.c",
     ]
   }
 
@@ -541,8 +541,8 @@ target(gtest_target_type, "zip_unittests") {
   testonly = true
 
   sources = [
-    "google/compression_utils_unittest.cc",
-    "google/zip_reader_unittest.cc",
+    # "google/compression_utils_unittest.cc",
+    # "google/zip_reader_unittest.cc",
     "google/zip_unittest.cc",
     "google/compression_utils.cc",
     "google/compression_utils.h",