Remove unused TerminalView.has_new_content (#7429)

Thorsten Ball created 2 years ago

Found this last week with @osiewicz and we realized that it's unused. So
I think it's fine to remove it, but I want to hear whether @mikayla-maki
has some thoughts here.



Release Notes:

- N/A

Change summary

crates/terminal_view/src/terminal_view.rs | 10 ----------
1 file changed, 10 deletions(-)

Detailed changes

crates/terminal_view/src/terminal_view.rs

@@ -77,7 +77,6 @@ pub struct TerminalView {
     terminal: Model<Terminal>,
     workspace: WeakView<Workspace>,
     focus_handle: FocusHandle,
-    has_new_content: bool,
     //Currently using iTerm bell, show bell emoji in tab until input is received
     has_bell: bool,
     context_menu: Option<(View<ContextMenu>, gpui::Point<Pixels>, Subscription)>,
@@ -142,9 +141,6 @@ impl TerminalView {
         cx.observe(&terminal, |_, _, cx| cx.notify()).detach();
         cx.subscribe(&terminal, move |this, _, event, cx| match event {
             Event::Wakeup => {
-                if !this.focus_handle.is_focused(cx) {
-                    this.has_new_content = true;
-                }
                 cx.notify();
                 cx.emit(Event::Wakeup);
                 cx.emit(ItemEvent::UpdateTab);
@@ -309,7 +305,6 @@ impl TerminalView {
         Self {
             terminal,
             workspace: workspace_handle,
-            has_new_content: true,
             has_bell: false,
             focus_handle,
             context_menu: None,
@@ -327,10 +322,6 @@ impl TerminalView {
         &self.terminal
     }
 
-    pub fn has_new_content(&self) -> bool {
-        self.has_new_content
-    }
-
     pub fn has_bell(&self) -> bool {
         self.has_bell
     }
@@ -688,7 +679,6 @@ impl TerminalView {
     }
 
     fn focus_in(&mut self, cx: &mut ViewContext<Self>) {
-        self.has_new_content = false;
         self.terminal.read(cx).focus_in();
         self.blink_cursors(self.blink_epoch, cx);
         cx.notify();