Implement progress bar spec (determinate and indeterminate).

ui/views/controls/progress_bar.h

 // Copyright (c) 2011 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 UI_VIEWS_CONTROLS_PROGRESS_BAR_H_
 #define UI_VIEWS_CONTROLS_PROGRESS_BAR_H_
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
+#include "ui/gfx/animation/animation_delegate.h"
 #include "ui/views/view.h"
 
+namespace gfx {
+class LinearAnimation;
+}
+
 namespace views {
 
 // Progress bar is a control that indicates progress visually.
 class VIEWS_EXPORT ProgressBar : public View {
  public:
   // The value range defaults to [0.0, 1.0].
   ProgressBar();
   ~ProgressBar() override;
 
   double current_value() const { return current_value_; }
 
+  virtual bool is_indeterminate() = 0;
+
   // Gets a normalized current value in [0.0, 1.0] range based on current value
   // range and the min/max display value range.
   double GetNormalizedValue() const;
 
   // Sets the inclusive range of values to be displayed.  Values outside of the
   // range will be capped when displayed.
   void SetDisplayRange(double min_display_value, double max_display_value);
 
   // Sets the current value.  Values outside of the range [min_display_value_,
   // max_display_value_] will be stored unmodified and capped for display.
   void SetValue(double value);
 
   // Sets the tooltip text.  Default behavior for a progress bar is to show no
   // tooltip on mouse hover. Calling this lets you set a custom tooltip.  To
   // revert to default behavior, call this with an empty string.
   void SetTooltipText(const base::string16& tooltip_text);
 
   // Overridden from View:
   bool GetTooltipText(const gfx::Point& p,
                       base::string16* tooltip) const override;
   void GetAccessibleState(ui::AXViewState* state) override;
+  gfx::Size GetPreferredSize() const override;
+  const char* GetClassName() const override;
+  void OnPaint(gfx::Canvas* canvas) override = 0;
+
+ protected:
+  // The color of the progress portion.
+  SkColor GetForegroundColor() const;
+  // The color of the portion that displays potential progress.
+  SkColor GetBackgroundColor() const;
 
  private:
   static const char kViewClassName[];
 
-  // Overridden from View:
-  gfx::Size GetPreferredSize() const override;
-  const char* GetClassName() const override;
-  void OnPaint(gfx::Canvas* canvas) override;
-
   // Inclusive range used when displaying values.
   double min_display_value_;
   double max_display_value_;
 
   // Current value.  May be outside of [min_display_value_, max_display_value_].
   double current_value_;
 
   // Tooltip text.
   base::string16 tooltip_text_;
 
   DISALLOW_COPY_AND_ASSIGN(ProgressBar);
 };
 
+class VIEWS_EXPORT DeterminateProgressBar : public ProgressBar {

[tdanderson, 2016/09/12 17:11:13]

Class-level descriptions would be nice to include here and above line 91.

[Evan Stade, 2016/09/13 02:05:14]

Done.

+ public:
+  DeterminateProgressBar();
+  ~DeterminateProgressBar() override;
+
+  bool is_indeterminate() override;
+
+ private:
+  // Overriden from views::ProgressBar (originally from views::View)
+  void OnPaint(gfx::Canvas* canvas) override;
+
+  DISALLOW_COPY_AND_ASSIGN(DeterminateProgressBar);
+};
+
+class VIEWS_EXPORT IndeterminateProgressBar : public DeterminateProgressBar,

[tdanderson, 2016/09/12 17:11:13]

I don't understand why you have chosen to have IPB extend DPB rather than have
both extend ProgressBar.

[Evan Stade, 2016/09/13 02:05:14]

I don't understand either. Just a brain fart.

I started deleting unused code and at some point I realized there's no need for
multiple classes (much less three of them), so I've consolidated it all down to
a single class. You get indeterminate behavior if you SetValue(-1).

+                                              public gfx::AnimationDelegate {
+ public:
+  IndeterminateProgressBar();
+  ~IndeterminateProgressBar() override;
+
+  bool is_indeterminate() override;
+
+ private:
+  // Overriden from views::ProgressBar (originally from views::View)
+  void OnPaint(gfx::Canvas* canvas) override;
+
+  // Overriden from gfx::AnimationDelegate
+  void AnimationProgressed(const gfx::Animation* animation) override;
+  void AnimationEnded(const gfx::Animation* animation) override;
+
+  std::unique_ptr<gfx::LinearAnimation> indeterminate_bar_animation_;
+
+  DISALLOW_COPY_AND_ASSIGN(IndeterminateProgressBar);
+};
+
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_PROGRESS_BAR_H_