Adds animated touch point and the hint box for touch calibration UX

chrome/browser/chromeos/display/touch_calibrator/touch_calibrator_view.h

 // Copyright 2016 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_DISPLAY_TOUCH_CALIBRATOR_TOUCH_CALIBRATOR_VIEW_H_
 #define CHROME_BROWSER_CHROMEOS_DISPLAY_TOUCH_CALIBRATOR_TOUCH_CALIBRATOR_VIEW_H_
 
 #include "base/macros.h"
 #include "ui/display/display.h"
 #include "ui/gfx/animation/animation_delegate.h"
 #include "ui/views/view.h"
 
 namespace views {
 class Label;
 class Widget;
 }
 
 namespace gfx {
 class Animation;
 class LinearAnimation;
 }
 
 namespace chromeos {
 
+class CircularThrobberView;
+class HintBox;
+
 // An overlay view used during touch calibration. This view is responsible for
 // all animations and UX during touch calibration on all displays currently
 // active on the device. The view on the display being calibrated is the primary
 // touch calibration view.
 // |TouchCalibratorView| acts as a state machine and has an API to toggle its
 // state or get the current state.
 class TouchCalibratorView : public views::View, public gfx::AnimationDelegate {
  public:
   // Different states of |TouchCalibratorView| in order.
   enum State {
@@ skipping 48 matching lines @@
 
   // Returns the current state of the view.
   State state() { return state_; }
 
  private:
   void InitViewContents();
 
   // The target display on which this view is rendered on.
   const display::Display display_;
 
+  // Offset from the edge of the screens for the touch points.
+  const int touch_point_offset_;
+
   // True if this view is on the display that is being calibrated.
   bool is_primary_view_ = false;
 
   std::unique_ptr<views::Widget> widget_;
 
   SkPaint paint_;
 
   // Defines the bounds for the background animation.
   gfx::RectF background_rect_;
 
   // Text label indicating how to exit the touch calibration.
   views::Label* exit_label_;
 
   // Start and end opacity values used during the fade animation. This is set
   // before the animation begins.
   float start_opacity_value_;
   float end_opacity_value_;
 
   // Linear animation used for various aniations including fade-in, fade out,
   // and view translation.
   std::unique_ptr<gfx::LinearAnimation> animator_;
 
+  // View responsible for displaying the animated circular icon that the user
+  // touches to calibrate the screen.
+  CircularThrobberView* throbber_circle_;
+
+  // A hint box displayed next to the first touch point to assist user with
+  // information about the next step.
+  HintBox* hint_box_view_;

[oshima, 2017/01/05 23:29:04]

looks like these views can also be just views::View*

[malaykeshav, 2017/01/09 18:59:41]

I need a rounded rectangle for the hint box. Would that be possible with just a
views::View* ?

[oshima, 2017/01/09 19:44:16]

You can do that when you create the view, not later right?

[malaykeshav, 2017/01/09 22:29:57]

Done

+
+  // View that contains the animated throbber circle and a text label informing
+  // the user to tap the circle to continue calibration.
+  views::View* touch_point_view_;
+
   State state_ = UNKNOWN;
 
   DISALLOW_COPY_AND_ASSIGN(TouchCalibratorView);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_DISPLAY_TOUCH_CALIBRATOR_TOUCH_CALIBRATOR_VIEW_H_