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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: sandbox/mac/os_compatibility.cc

Issue 2288003002: Delete IsAtLeastOS10_9() and IsAtMostOS10_9() (Closed)
Patch Set: Clean up a duplicate check, delete a stray ! Created 4 years, 4 months 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
« base/sys_info_mac.mm ('K') | « chrome/browser/ui/cocoa/custom_frame_view_unittest.mm ('k') | ui/base/cocoa/base_view.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sandbox/mac/os_compatibility.cc
diff --git a/sandbox/mac/os_compatibility.cc b/sandbox/mac/os_compatibility.cc
index 1e0ba5be76faa7802182862b353526d63a29a9a5..41ffde2f69fccd8ce9edda9395fec870557c5ff7 100644
--- a/sandbox/mac/os_compatibility.cc
+++ b/sandbox/mac/os_compatibility.cc
@@ -181,10 +181,10 @@ class OSCompatibility_10_10 : public OSCompatibility {
// static
std::unique_ptr<OSCompatibility> OSCompatibility::CreateForPlatform() {
- if (base::mac::IsOS10_9())
- return base::WrapUnique(new OSCompatibility_10_7());
- else
+ if (base::mac::IsAtLeastOS10_10())
return base::WrapUnique(new OSCompatibility_10_10());
+ else
+ return base::WrapUnique(new OSCompatibility_10_7());
}
OSCompatibility::~OSCompatibility() {}
« base/sys_info_mac.mm ('K') | « chrome/browser/ui/cocoa/custom_frame_view_unittest.mm ('k') | ui/base/cocoa/base_view.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698