Componentize internal class FlagsState in flags_ui component.

chrome/browser/chromeos/settings/owner_flags_storage.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/settings/owner_flags_storage.h"
 
 #include "base/command_line.h"
 #include "base/prefs/pref_service.h"
 #include "base/values.h"
 #include "chrome/browser/about_flags.h"
@@ skipping 39 matching lines @@
 
 OwnerFlagsStorage::~OwnerFlagsStorage() {}
 
 bool OwnerFlagsStorage::SetFlags(const std::set<std::string>& flags) {
   PrefServiceFlagsStorage::SetFlags(flags);
 
   base::ListValue experiments_list;
 
   base::CommandLine command_line(base::CommandLine::NO_PROGRAM);
   ::about_flags::ConvertFlagsToSwitches(this, &command_line,
-                                        ::about_flags::kNoSentinels);
+                                        ::flags_ui::kNoSentinels);

[Alexei Svitkine (slow), 2015/11/16 21:52:54]

Nit: No need for leading ::, since there's no longer a conflict here.

[sdefresne, 2015/11/17 16:28:56]

Done.

   base::CommandLine::StringVector switches = command_line.argv();
   for (base::CommandLine::StringVector::const_iterator it =
            switches.begin() + 1;
        it != switches.end(); ++it) {
     experiments_list.Append(new base::StringValue(*it));
   }
   owner_settings_service_->Set(kStartUpFlags, experiments_list);
 
   return true;
 }
 
 }  // namespace about_flags
 }  // namespace chromeos