Added usage of ScreenContext in EulaScreen.

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

Index: chrome/browser/chromeos/login/screens/base_screen.h
diff --git a/chrome/browser/chromeos/login/screens/base_screen.h b/chrome/browser/chromeos/login/screens/base_screen.h
index 2cbde5956edd99ef67f578aa3b7dfb7dd9154084..a9483929f94aef4023aae0e53fc10235acd4e7c4 100644
--- a/chrome/browser/chromeos/login/screens/base_screen.h
+++ b/chrome/browser/chromeos/login/screens/base_screen.h
@@ -82,10 +82,9 @@ class BaseScreen {
   // about this event comes from the JS counterpart.
   virtual void OnButtonPressed(const std::string& button_id);
 
-  // Called when context for the currenct screen was
-  // changed. Notification about this event comes from the JS
-  // counterpart.
-  virtual void OnContextChanged(const base::DictionaryValue* diff);
+  // The method is called each time some key in screen context is updated by JS
+  // side. Should be implemented in subclasses.

[dzhioev (left Google), 2014/11/27 11:54:02]

This comment needs to be updated. Implementation of this method is optional.

[ygorshenin1, 2014/11/27 13:07:33]

Done.

+  virtual void OnContextKeyUpdated(const ::login::ScreenContext::KeyType& key);

[dzhioev (left Google), 2014/11/27 11:54:02]

I think we need another method, notifying about all changes at once. Something
like "OnContextUpdated(const std::vector<Type>& keys)".

[ygorshenin1, 2014/11/27 13:07:33]

From my point of view it will be quite easy to implement OnContextUpdated() when
we'll have screens wanting the method.

 
   BaseScreenDelegate* get_base_screen_delegate() const {
     return base_screen_delegate_;
@@ -98,12 +97,18 @@ class BaseScreen {
   FRIEND_TEST_ALL_PREFIXES(EnrollmentScreenTest, TestSuccess);
   FRIEND_TEST_ALL_PREFIXES(ProvisionedEnrollmentScreenTest, TestBackButton);
 
+  friend class BaseScreenHandler;
   friend class NetworkScreenTest;
   friend class ScreenManager;
   friend class UpdateScreenTest;
 
   void SetContext(::login::ScreenContext* context);
 
+  // Called when context for the currenct screen was

[dzhioev (left Google), 2014/11/27 11:54:02]

*current

[ygorshenin1, 2014/11/27 13:07:33]

Done.

+  // changed. Notification about this event comes from the JS
+  // counterpart.
+  void OnContextChanged(const base::DictionaryValue& diff);
+
   BaseScreenDelegate* base_screen_delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(BaseScreen);