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

Side by Side Diff: components/variations/client_filterable_state.cc

Issue 2924983003: [Variations] Refactor all state used for study filtering into a container struct. (Closed)
Patch Set: A bit more cleanup Created 3 years, 6 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "components/variations/client_filterable_state.h"
6
7 #include "build/build_config.h"
8
9 namespace variations {
10
11 // static
12 Study::Platform ClientFilterableState::GetCurrentPlatform() {
13 #if defined(OS_WIN)
14 return Study::PLATFORM_WINDOWS;
15 #elif defined(OS_IOS)
16 return Study::PLATFORM_IOS;
17 #elif defined(OS_MACOSX)
18 return Study::PLATFORM_MAC;
19 #elif defined(OS_CHROMEOS)
20 return Study::PLATFORM_CHROMEOS;
21 #elif defined(OS_ANDROID)
22 return Study::PLATFORM_ANDROID;
23 #elif defined(OS_LINUX) || defined(OS_BSD) || defined(OS_SOLARIS)
24 // Default BSD and SOLARIS to Linux to not break those builds, although these
25 // platforms are not officially supported by Chrome.
26 return Study::PLATFORM_LINUX;
27 #else
28 #error Unknown platform
29 #endif
30 }
31
32 ClientFilterableState::ClientFilterableState() {}
33 ClientFilterableState::~ClientFilterableState() {}
34
35 } // namespace variations
OLDNEW
« no previous file with comments | « components/variations/client_filterable_state.h ('k') | components/variations/service/variations_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698