Added usage of ScreenContext in EulaScreen.

chrome/browser/chromeos/login/screens/base_screen.cc

Index: chrome/browser/chromeos/login/screens/base_screen.cc
diff --git a/chrome/browser/chromeos/login/screens/base_screen.cc b/chrome/browser/chromeos/login/screens/base_screen.cc
index 82f38861ed19d96d9a7870e0b69cc4ff17a89110..b0b8200751d1f52e142d3f1eb101d324e9023342 100644
--- a/chrome/browser/chromeos/login/screens/base_screen.cc
+++ b/chrome/browser/chromeos/login/screens/base_screen.cc
@@ -37,6 +37,8 @@ bool BaseScreen::IsPermanent() {
 }
 
 std::string BaseScreen::GetID() const {
+  // TODO (ygorshenin, crbug.com/433797): elimitate intermediate
+  // GetName() ASAP.
   return GetName();
 }
 
@@ -48,11 +50,19 @@ void BaseScreen::SetContext(::login::ScreenContext* context) {
 }
 
 void BaseScreen::OnButtonPressed(const std::string& button_id) {
-  LOG(WARNING) << "BaseScreen::OnButtonPressed(): button_id=" << button_id;
+  LOG(WARNING) << "Unhandled button click: button_id=" << button_id;

[Denis Kuznetsov (DE-MUC), 2014/11/27 13:47:38]

note: I agree that naming can be better: like OnUserAction(...), but it can be
done out of the this CL scope.

[ygorshenin1, 2014/11/27 14:12:06]

I renamed it in https://codereview.chromium.org/763563002/.

 }
 
-void BaseScreen::OnContextChanged(const base::DictionaryValue* diff) {
-  LOG(WARNING) << "BaseScreen::OnContextChanged()";
+void BaseScreen::OnContextKeyUpdated(
+    const ::login::ScreenContext::KeyType& key) {
+  LOG(WARNING) << "Unhandled context change: key=" << key;
+}
+
+void BaseScreen::OnContextChanged(const base::DictionaryValue& diff) {
+  std::vector<::login::ScreenContext::KeyType> keys;
+  context_.ApplyChanges(diff, &keys);
+  for (const auto& key : keys)
+    OnContextKeyUpdated(key);
 }
 
 }  // namespace chromeos