Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(171)

Unified Diff: mojo/edk/embedder/platform_channel_pair_win.cc

Issue 1441853003: Make the new Mojo EDK work on XP. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test fixes Created 5 years, 1 month 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: mojo/edk/embedder/platform_channel_pair_win.cc
diff --git a/mojo/edk/embedder/platform_channel_pair_win.cc b/mojo/edk/embedder/platform_channel_pair_win.cc
index 8ba325ae95841575bbc7176f98b123a509d4682d..f7788ad3e2780022210cd6efb98f7efb4fb2102c 100644
--- a/mojo/edk/embedder/platform_channel_pair_win.cc
+++ b/mojo/edk/embedder/platform_channel_pair_win.cc
@@ -89,9 +89,8 @@ void PlatformChannelPair::PrepareToPassClientHandleToChildProcess(
DCHECK(handle_passing_info);
DCHECK(client_handle_.is_valid());
- CHECK_GE(base::win::GetVersion(), base::win::VERSION_VISTA);
-
- handle_passing_info->push_back(client_handle_.get().handle);
+ if (base::win::GetVersion() >= base::win::VERSION_VISTA)
+ handle_passing_info->push_back(client_handle_.get().handle);
// Log a warning if the command line already has the switch, but "clobber" it
// anyway, since it's reasonably likely that all the switches were just copied

Powered by Google App Engine
This is Rietveld 408576698