Call the parent methods in MessageView::OnPaint, OnFocus, and OnBlur

ui/message_center/views/message_view.cc

 // 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.
 
 #include "ui/message_center/views/message_view.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "ui/accessibility/ax_node_data.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/models/simple_menu_model.h"
@@ skipping 134 matching lines @@
   // Space key handling is triggerred at key-release timing. See
   // ui/views/controls/buttons/custom_button.cc for why.
   if (event.flags() != ui::EF_NONE || event.key_code() != ui::VKEY_SPACE)
     return false;
 
   controller_->ClickOnNotification(notification_id_);
   return true;
 }
 
 void MessageView::OnPaint(gfx::Canvas* canvas) {
+  views::View::OnPaint(canvas);

[yoshiki, 2017/05/08 11:22:17]

OnPaint draws a border so it's necessary to fix crbug.com/719414

   views::Painter::PaintFocusPainter(this, canvas, focus_painter_.get());
 }
 
 void MessageView::OnFocus() {
+  views::View::OnFocus();

[yoshiki, 2017/05/08 11:22:17]

OnFocus fires an a11y event so I think it's needed.

   // We paint a focus indicator.
   SchedulePaint();
 }
 
 void MessageView::OnBlur() {
+  views::View::OnBlur();

[yoshiki, 2017/05/08 11:22:17]

Actually View::OnBlur() is empty but I added this for consistency with OnFocus.

   // We paint a focus indicator.
   SchedulePaint();
 }
 
 void MessageView::Layout() {
   gfx::Rect content_bounds = GetContentsBounds();
 
   // Background.
   background_view_->SetBoundsRect(content_bounds);
 #if defined(OS_CHROMEOS)
@@ skipping 50 matching lines @@
 }
 
 void MessageView::SetDrawBackgroundAsActive(bool active) {
   background_view_->background()->
       SetNativeControlColor(active ? kHoveredButtonBackgroundColor :
                                      kNotificationBackgroundColor);
   SchedulePaint();
 }
 
 }  // namespace message_center