Commit cde5521b authored by Palak Agarwal's avatar Palak Agarwal Committed by Commit Bot

Fixing OS Macros definitions for ChromeOS

Bug: 1117534
Change-Id: I34823045b6e3364edbab94fd3eb74b8b1ef02d6f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2397619Reviewed-by: default avatarGuido Urdaneta <guidou@chromium.org>
Commit-Queue: Palak Agarwal <agpalak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#805411}
parent 7ba35877
...@@ -50,7 +50,7 @@ class WebRtcDisableEncryptionFlagBrowserTest : public WebRtcTestBase { ...@@ -50,7 +50,7 @@ class WebRtcDisableEncryptionFlagBrowserTest : public WebRtcTestBase {
// TODO(crbug.com/910216): De-flake this for ChromeOs. // TODO(crbug.com/910216): De-flake this for ChromeOs.
// TODO(crbug.com/984879): De-flake this for ASAN/MSAN Linux. // TODO(crbug.com/984879): De-flake this for ASAN/MSAN Linux.
#if defined(OS_CHROMEOS) || \ #if defined(OS_CHROMEOS) || \
((defined(OS_LINUX) || defined(OS_CHROMEOS)) && \ (defined(OS_LINUX) && \
(defined(MEMORY_SANITIZER) || defined(ADDRESS_SANITIZER))) || \ (defined(MEMORY_SANITIZER) || defined(ADDRESS_SANITIZER))) || \
(defined(OS_WIN) && defined(ADDRESS_SANITIZER)) (defined(OS_WIN) && defined(ADDRESS_SANITIZER))
#define MAYBE_VerifyEncryption DISABLED_VerifyEncryption #define MAYBE_VerifyEncryption DISABLED_VerifyEncryption
......
...@@ -90,11 +90,10 @@ IN_PROC_BROWSER_TEST_F(WebRtcGetDisplayMediaBrowserTestWithPicker, ...@@ -90,11 +90,10 @@ IN_PROC_BROWSER_TEST_F(WebRtcGetDisplayMediaBrowserTestWithPicker,
#if defined(OS_CHROMEOS) || defined(OS_WIN) #if defined(OS_CHROMEOS) || defined(OS_WIN)
#define MAYBE_GetDisplayMediaVideoAndAudio DISABLED_GetDisplayMediaVideoAndAudio #define MAYBE_GetDisplayMediaVideoAndAudio DISABLED_GetDisplayMediaVideoAndAudio
// On linux debug bots, it's flaky as well. // On linux debug bots, it's flaky as well.
#elif ((defined(OS_LINUX) || defined(OS_CHROMEOS)) && !defined(NDEBUG)) #elif (defined(OS_LINUX) && !defined(NDEBUG))
#define MAYBE_GetDisplayMediaVideoAndAudio DISABLED_GetDisplayMediaVideoAndAudio #define MAYBE_GetDisplayMediaVideoAndAudio DISABLED_GetDisplayMediaVideoAndAudio
// On linux asan bots, it's flaky as well - msan and other rel bot are fine. // On linux asan bots, it's flaky as well - msan and other rel bot are fine.
#elif ((defined(OS_LINUX) || defined(OS_CHROMEOS)) && \ #elif (defined(OS_LINUX) && defined(ADDRESS_SANITIZER))
defined(ADDRESS_SANITIZER))
#define MAYBE_GetDisplayMediaVideoAndAudio DISABLED_GetDisplayMediaVideoAndAudio #define MAYBE_GetDisplayMediaVideoAndAudio DISABLED_GetDisplayMediaVideoAndAudio
#else #else
#define MAYBE_GetDisplayMediaVideoAndAudio GetDisplayMediaVideoAndAudio #define MAYBE_GetDisplayMediaVideoAndAudio GetDisplayMediaVideoAndAudio
......
...@@ -347,7 +347,7 @@ void WebRtcLogUploader::SetupMultipart( ...@@ -347,7 +347,7 @@ void WebRtcLogUploader::SetupMultipart(
const char product[] = "Chrome"; const char product[] = "Chrome";
#elif defined(OS_MAC) #elif defined(OS_MAC)
const char product[] = "Chrome_Mac"; const char product[] = "Chrome_Mac";
#elif defined(OS_LINUX) || defined(OS_CHROMEOS) #elif defined(OS_LINUX)
#if !defined(ADDRESS_SANITIZER) #if !defined(ADDRESS_SANITIZER)
const char product[] = "Chrome_Linux"; const char product[] = "Chrome_Linux";
#else #else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment