Browse Source

refactor: make StatusIconType an enum class (#36500)

Co-authored-by: Milan Burda <[email protected]>
Milan Burda 2 years ago
parent
commit
909ee0ed6b
2 changed files with 8 additions and 7 deletions
  1. 5 4
      shell/browser/ui/tray_icon_gtk.cc
  2. 3 3
      shell/browser/ui/tray_icon_gtk.h

+ 5 - 4
shell/browser/ui/tray_icon_gtk.cc

@@ -30,7 +30,8 @@ gfx::ImageSkia GetBestImageRep(const gfx::ImageSkia& image) {
 }  // namespace
 
 TrayIconGtk::TrayIconGtk()
-    : status_icon_(new StatusIconLinuxDbus), status_icon_type_(kTypeDbus) {
+    : status_icon_(new StatusIconLinuxDbus),
+      status_icon_type_(StatusIconType::kDbus) {
   status_icon_->SetDelegate(this);
 }
 
@@ -68,11 +69,11 @@ ui::MenuModel* TrayIconGtk::GetMenuModel() const {
 
 void TrayIconGtk::OnImplInitializationFailed() {
   switch (status_icon_type_) {
-    case kTypeDbus:
+    case StatusIconType::kDbus:
       status_icon_ = nullptr;
-      status_icon_type_ = kTypeNone;
+      status_icon_type_ = StatusIconType::kNone;
       return;
-    case kTypeNone:
+    case StatusIconType::kNone:
       NOTREACHED();
   }
 }

+ 3 - 3
shell/browser/ui/tray_icon_gtk.h

@@ -36,9 +36,9 @@ class TrayIconGtk : public TrayIcon, public ui::StatusIconLinux::Delegate {
   void OnImplInitializationFailed() override;
 
  private:
-  enum StatusIconType {
-    kTypeDbus,
-    kTypeNone,
+  enum class StatusIconType {
+    kDbus,
+    kNone,
   };
 
   scoped_refptr<StatusIconLinuxDbus> status_icon_;