Show progress bar notifications for Mac.

ui/message_center/cocoa/notification_controller.mm

 // Copyright (c) 2013 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.
 
 #import "ui/message_center/cocoa/notification_controller.h"
 
 #include "base/mac/foundation_util.h"
 #include "base/strings/string_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "grit/ui_resources.h"
 #include "grit/ui_strings.h"
 #include "skia/ext/skia_utils_mac.h"
 #import "ui/base/cocoa/hover_image_button.h"
 #include "ui/base/l10n/l10n_util_mac.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/text/text_elider.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/message_center_style.h"
 #include "ui/message_center/notification.h"
 
+const CGFloat kProgressBarThickness = 8;
+const CGFloat kProgressBarTopPadding = 12;
+const SkColor kProgressBarBackgroundColor = SkColorSetRGB(230, 230, 230);

[Robert Sesek, 2013/07/29 18:50:26]

Should these be shared in the constants file?

[jianli, 2013/07/29 19:05:40]

The constants used to paint progress bar for Windows is defined in
ui/views/controls/progress_bar.cc that is used in quite a few other places other
than progress bar notifications. So we can't pull these constants out into a
common file since they're only used for Cocoa implementation.

+const SkColor kProgressBarBackgroundBorderColor = SkColorSetRGB(208, 208, 208);
+const SkColor kProgressBarSliceColor = SkColorSetRGB(78, 156, 245);
+const SkColor kProgressBarSliceBorderColor = SkColorSetRGB(110, 188, 249);
+
+@interface MCNotificationProgressBar : NSProgressIndicator {

[Robert Sesek, 2013/07/29 18:50:26]

nit: remove {}

[jianli, 2013/07/29 19:05:40]

Done.

+}
+@end
+
+@implementation MCNotificationProgressBar
+- (void)drawRect:(NSRect)dirtyRect {
+  NSRect sliceRect, remainderRect;
+  double progressFraction = ([self doubleValue] - [self minValue]) /
+      ([self maxValue] - [self minValue]);
+  NSDivideRect(dirtyRect, &sliceRect, &remainderRect,
+               NSWidth(dirtyRect) * progressFraction, NSMinXEdge);
+
+  // For slice part.
+  [gfx::SkColorToCalibratedNSColor(kProgressBarSliceBorderColor)
+      drawSwatchInRect:sliceRect];
+  [gfx::SkColorToCalibratedNSColor(kProgressBarSliceColor)
+      drawSwatchInRect:NSInsetRect(sliceRect, 1.0, 1.0)];
+
+  // For remainder part.
+  [gfx::SkColorToCalibratedNSColor(kProgressBarBackgroundBorderColor)
+      drawSwatchInRect:remainderRect];
+  [gfx::SkColorToCalibratedNSColor(kProgressBarBackgroundColor)
+      drawSwatchInRect:NSInsetRect(remainderRect, 1.0, 1.0)];
+}
+@end
+
+////////////////////////////////////////////////////////////////////////////////
+
 @interface MCNotificationButtonCell : NSButtonCell {
   BOOL hovered_;
 }
 @end
 
 @implementation MCNotificationButtonCell
 - (void)drawBezelWithFrame:(NSRect)frame inView:(NSView*)controlView {
   // Else mouseEntered: and mouseExited: won't be called and hovered_ won't be
   // valid.
   DCHECK([self showsBorderOnlyWhileMouseInside]);
@@ skipping 279 matching lines @@
     // TODO(thakis): The spacing is not completely right.
     CGFloat listTopPadding =
         message_center::kTextTopPadding - messageTopGap;
     listFrame.size.height = y;
     listFrame.origin.y =
         NSMinY(messageFrame) - listTopPadding - NSHeight(listFrame);
     [listItemView_ setFrame:listFrame];
     [[self view] addSubview:listItemView_];
   }
 
+  // Create the progress bar view if needed.
+  [progressBarView_ removeFromSuperview];
+  NSRect progressBarFrame = NSZeroRect;
+  if (notification->type() == message_center::NOTIFICATION_TYPE_PROGRESS) {
+    progressBarFrame = [self currentContentRect];
+    progressBarFrame.origin.y = NSMinY(messageFrame) -
+        kProgressBarTopPadding - kProgressBarThickness;
+    progressBarFrame.size.height = kProgressBarThickness;
+    progressBarView_.reset([[MCNotificationProgressBar alloc]
+                            initWithFrame:progressBarFrame]);

[Robert Sesek, 2013/07/29 18:50:26]

nit: does this fit if you break after the .reset( and put the entire ObjC call
on its own line?

[jianli, 2013/07/29 19:05:40]

Done.

+    // Setting indeterminate to NO does not work with custom drawRect.
+    [progressBarView_ setIndeterminate:YES];
+    [progressBarView_ setStyle:NSProgressIndicatorBarStyle];
+    [progressBarView_ setDoubleValue:notification->progress()];
+    [[self view] addSubview:progressBarView_];
+  }
+
   // If the bottom-most element so far is out of the rootView's bounds, resize
   // the view.
   CGFloat minY = NSMinY(messageFrame);
   if (listItemView_ && NSMinY(listFrame) < minY)
     minY = NSMinY(listFrame);
+  if (progressBarView_ && NSMinY(progressBarFrame) < minY)
+    minY = NSMinY(progressBarFrame);
   if (minY < messagePadding) {
     CGFloat delta = messagePadding - minY;
     rootFrame.size.height += delta;
     titleFrame.origin.y += delta;
     messageFrame.origin.y += delta;
     listFrame.origin.y += delta;
+    progressBarFrame.origin.y += delta;
   }
 
   // Add the bottom container view.
   NSRect frame = rootFrame;
   frame.size.height = 0;
   [bottomView_ removeFromSuperview];
   bottomView_.reset([[NSView alloc] initWithFrame:frame]);
   CGFloat y = 0;
 
   // Create action buttons if appropriate, bottom-up.
@@ skipping 49 matching lines @@
     [bottomView_ addSubview:imageView];
   }
 
   [bottomView_ setFrame:frame];
   [[self view] addSubview:bottomView_];
 
   rootFrame.size.height += NSHeight(frame);
   titleFrame.origin.y += NSHeight(frame);
   messageFrame.origin.y += NSHeight(frame);
   listFrame.origin.y += NSHeight(frame);
+  progressBarFrame.origin.y += NSHeight(frame);
 
   [[self view] setFrame:rootFrame];
   [title_ setFrame:titleFrame];
   [message_ setFrame:messageFrame];
   [listItemView_ setFrame:listFrame];
+  [progressBarView_ setFrame:progressBarFrame];
 
   return rootFrame;
 }
 
 - (void)close:(id)sender {
   [closeButton_ setTarget:nil];
   messageCenter_->RemoveNotification([self notificationID], /*by_user=*/true);
 }
 
 - (void)buttonClicked:(id)button {
@@ skipping 166 matching lines @@
     if (font.GetStringWidth(last) > width)
       last = ui::ElideText(last, font, width, ui::ELIDE_AT_END);
     wrapped.resize(lines - 1);
     wrapped.push_back(last);
   }
 
   return JoinString(wrapped, '\n');
 }
 
 @end