Implemented event-dispatching for button clicks.

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

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 62 matching lines @@
   // Returns the identifier of the screen.
   virtual std::string GetID() const;
 
  protected:
   // Screen can call this method to notify framework that it have finished
   // it's work with |outcome|.
   void Finish(BaseScreenDelegate::ExitCodes exit_code);
 
   // Called when button with |button_id| was pressed. Notification
   // about this event comes from the JS counterpart.
-  virtual void OnButtonPressed(const std::string& button_id);
+  virtual void OnButtonClicked(const std::string& button_id);

[dzhioev (left Google), 2014/11/27 12:08:51]

I propose to rename this method to something more general. For example,
"OnUserActed(action_id)". 

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

OnUserAction() seems better to me.

[ygorshenin1, 2014/11/27 14:11:13]

Done.

 
   // The method is called each time some key in screen context is updated by JS
   // side. Should be implemented in subclasses.
   virtual void OnContextKeyUpdated(const ::login::ScreenContext::KeyType& key);
 
   BaseScreenDelegate* get_base_screen_delegate() const {
     return base_screen_delegate_;
   }
 
   ::login::ScreenContext context_;
@@ skipping 16 matching lines @@
   void OnContextChanged(const base::DictionaryValue& diff);
 
   BaseScreenDelegate* base_screen_delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(BaseScreen);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_SCREENS_BASE_SCREEN_H_