Commit d7e1e9b3 authored by Roman Aleksandrov's avatar Roman Aleksandrov Committed by Commit Bot

Relaunch Notification: Clean up

Move recommended timer to the desktop build because its usage in
Chrome OS was deleted. Update comments.

Bug: 963883
Change-Id: I05ece8ac88d5209b812f6a27bd72e69b96052705
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1674545
Commit-Queue: Roman Aleksandrov <raleksandrov@google.com>
Reviewed-by: default avatarGreg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#672033}
parent da0dcaa4
...@@ -2929,8 +2929,6 @@ jumbo_split_static_library("ui") { ...@@ -2929,8 +2929,6 @@ jumbo_split_static_library("ui") {
"views/relaunch_notification/relaunch_notification_controller.h", "views/relaunch_notification/relaunch_notification_controller.h",
"views/relaunch_notification/relaunch_notification_metrics.cc", "views/relaunch_notification/relaunch_notification_metrics.cc",
"views/relaunch_notification/relaunch_notification_metrics.h", "views/relaunch_notification/relaunch_notification_metrics.h",
"views/relaunch_notification/relaunch_recommended_timer.cc",
"views/relaunch_notification/relaunch_recommended_timer.h",
"views/relaunch_notification/relaunch_required_timer.cc", "views/relaunch_notification/relaunch_required_timer.cc",
"views/relaunch_notification/relaunch_required_timer.h", "views/relaunch_notification/relaunch_required_timer.h",
"views/relaunch_notification/relaunch_required_timer_internal.cc", "views/relaunch_notification/relaunch_required_timer_internal.cc",
...@@ -3208,6 +3206,8 @@ jumbo_split_static_library("ui") { ...@@ -3208,6 +3206,8 @@ jumbo_split_static_library("ui") {
"views/relaunch_notification/relaunch_notification_controller_platform_impl_desktop.h", "views/relaunch_notification/relaunch_notification_controller_platform_impl_desktop.h",
"views/relaunch_notification/relaunch_recommended_bubble_view.cc", "views/relaunch_notification/relaunch_recommended_bubble_view.cc",
"views/relaunch_notification/relaunch_recommended_bubble_view.h", "views/relaunch_notification/relaunch_recommended_bubble_view.h",
"views/relaunch_notification/relaunch_recommended_timer.cc",
"views/relaunch_notification/relaunch_recommended_timer.h",
"views/relaunch_notification/relaunch_required_dialog_view.cc", "views/relaunch_notification/relaunch_required_dialog_view.cc",
"views/relaunch_notification/relaunch_required_dialog_view.h", "views/relaunch_notification/relaunch_required_dialog_view.h",
"views/screen_capture_notification_ui_views.cc", "views/screen_capture_notification_ui_views.cc",
......
...@@ -11,8 +11,7 @@ ...@@ -11,8 +11,7 @@
#include "chrome/browser/ui/views/relaunch_notification/wall_clock_timer.h" #include "chrome/browser/ui/views/relaunch_notification/wall_clock_timer.h"
// Timer that handles notification title refresh for relaunch recommended // Timer that handles notification title refresh for relaunch recommended
// notification. Created either by RelaunchRecommendedBubbleView for Chrome // notification. Created by RelaunchRecommendedBubbleView for Chrome desktop.
// desktop or directly by the controller in the Chrome OS implementation.
// Title refresh is invoked with the |callback| provided at creation. // Title refresh is invoked with the |callback| provided at creation.
class RelaunchRecommendedTimer { class RelaunchRecommendedTimer {
public: public:
...@@ -44,8 +43,7 @@ class RelaunchRecommendedTimer { ...@@ -44,8 +43,7 @@ class RelaunchRecommendedTimer {
// A timer with which title refreshes are scheduled. // A timer with which title refreshes are scheduled.
WallClockTimer refresh_timer_; WallClockTimer refresh_timer_;
// Callback which triggers the actual title update, which differs on Chrome // Callback which triggers the actual title update on Chrome desktop.
// for desktop vs for Chrome OS.
base::RepeatingClosure callback_; base::RepeatingClosure callback_;
DISALLOW_COPY_AND_ASSIGN(RelaunchRecommendedTimer); DISALLOW_COPY_AND_ASSIGN(RelaunchRecommendedTimer);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment