From 94483920f2a32071505b913881eb7b342c910535 Mon Sep 17 00:00:00 2001 From: Shin'ya Ueoka Date: Sun, 28 Mar 2021 11:02:05 +0900 Subject: Move classes to a correct directory --- src/background/controllers/ConsoleController.ts | 2 +- src/background/controllers/ConsoleFrameClient.ts | 15 --------------- src/background/controllers/ConsoleUseCase.ts | 21 --------------------- 3 files changed, 1 insertion(+), 37 deletions(-) delete mode 100644 src/background/controllers/ConsoleFrameClient.ts delete mode 100644 src/background/controllers/ConsoleUseCase.ts (limited to 'src/background/controllers') diff --git a/src/background/controllers/ConsoleController.ts b/src/background/controllers/ConsoleController.ts index 11ee09d..72b9b8e 100644 --- a/src/background/controllers/ConsoleController.ts +++ b/src/background/controllers/ConsoleController.ts @@ -1,5 +1,5 @@ import { injectable } from "tsyringe"; -import ConsoleUseCase from "./ConsoleUseCase"; +import ConsoleUseCase from "../usecases/ConsoleUseCase"; @injectable() export default class ConsoleController { diff --git a/src/background/controllers/ConsoleFrameClient.ts b/src/background/controllers/ConsoleFrameClient.ts deleted file mode 100644 index 170566d..0000000 --- a/src/background/controllers/ConsoleFrameClient.ts +++ /dev/null @@ -1,15 +0,0 @@ -import * as messages from "../../shared/messages"; - -export default interface ConsoleFrameClient { - resize(tabId: number, width: number, height: number): Promise; -} - -export class ConsoleFrameClientImpl implements ConsoleFrameClient { - async resize(tabId: number, width: number, height: number): Promise { - await browser.tabs.sendMessage(tabId, { - type: messages.CONSOLE_RESIZE, - width, - height, - }); - } -} diff --git a/src/background/controllers/ConsoleUseCase.ts b/src/background/controllers/ConsoleUseCase.ts deleted file mode 100644 index 556aaf8..0000000 --- a/src/background/controllers/ConsoleUseCase.ts +++ /dev/null @@ -1,21 +0,0 @@ -import { inject, injectable } from "tsyringe"; -import ConsoleFrameClient from "./ConsoleFrameClient"; -import TabPresenter from "../presenters/TabPresenter"; - -@injectable() -export default class ConsoleUseCase { - constructor( - @inject("TabPresenter") - private readonly tabPresenter: TabPresenter, - @inject("ConsoleFrameClient") - private readonly consoleFrameClient: ConsoleFrameClient - ) {} - - async resize(width: number, height: number): Promise { - const tabId = (await this.tabPresenter.getCurrent()).id; - if (typeof tabId === "undefined") { - return; - } - return this.consoleFrameClient.resize(tabId, width, height); - } -} -- cgit v1.2.3 From a2a4531527c7786af18611e1d423c1504bb9245e Mon Sep 17 00:00:00 2001 From: Shin'ya Ueoka Date: Sun, 28 Mar 2021 11:06:19 +0900 Subject: Send resize message to the sender tab instead of the current tab. Sending resize message to the current table occurs an error, just after switching the tab by a `tabopen` command. The background script unable to connect the current (new tab) because it is in an initialization. --- src/background/controllers/ConsoleController.ts | 4 ++-- src/background/infrastructures/ContentMessageListener.ts | 14 +++++++++++--- src/background/usecases/ConsoleUseCase.ts | 9 +-------- 3 files changed, 14 insertions(+), 13 deletions(-) (limited to 'src/background/controllers') diff --git a/src/background/controllers/ConsoleController.ts b/src/background/controllers/ConsoleController.ts index 72b9b8e..2a0d75f 100644 --- a/src/background/controllers/ConsoleController.ts +++ b/src/background/controllers/ConsoleController.ts @@ -5,7 +5,7 @@ import ConsoleUseCase from "../usecases/ConsoleUseCase"; export default class ConsoleController { constructor(private readonly consoleUseCase: ConsoleUseCase) {} - resize(width: number, height: number) { - return this.consoleUseCase.resize(width, height); + resize(senderTabId: number, width: number, height: number) { + return this.consoleUseCase.resize(senderTabId, width, height); } } diff --git a/src/background/infrastructures/ContentMessageListener.ts b/src/background/infrastructures/ContentMessageListener.ts index ec87938..6023047 100644 --- a/src/background/infrastructures/ContentMessageListener.ts +++ b/src/background/infrastructures/ContentMessageListener.ts @@ -81,7 +81,11 @@ export default class ContentMessageListener { case messages.CONSOLE_ENTER_COMMAND: return this.onConsoleEnterCommand(message.text); case messages.CONSOLE_RESIZE: - return this.onConsoleResize(message.width, message.height); + return this.onConsoleResize( + senderTab.id!, + message.width, + message.height + ); case messages.SETTINGS_QUERY: return this.onSettingsQuery(); case messages.FIND_GET_KEYWORD: @@ -116,8 +120,12 @@ export default class ContentMessageListener { return this.commandController.exec(text); } - onConsoleResize(width: number, height: number): Promise { - return this.consoleController.resize(width, height); + onConsoleResize( + senderTabId: number, + width: number, + height: number + ): Promise { + return this.consoleController.resize(senderTabId, width, height); } async onSettingsQuery(): Promise { diff --git a/src/background/usecases/ConsoleUseCase.ts b/src/background/usecases/ConsoleUseCase.ts index 329e4ee..7b461e5 100644 --- a/src/background/usecases/ConsoleUseCase.ts +++ b/src/background/usecases/ConsoleUseCase.ts @@ -1,21 +1,14 @@ import { inject, injectable } from "tsyringe"; import ConsoleFrameClient from "../clients/ConsoleFrameClient"; -import TabPresenter from "../presenters/TabPresenter"; @injectable() export default class ConsoleUseCase { constructor( - @inject("TabPresenter") - private readonly tabPresenter: TabPresenter, @inject("ConsoleFrameClient") private readonly consoleFrameClient: ConsoleFrameClient ) {} - async resize(width: number, height: number): Promise { - const tabId = (await this.tabPresenter.getCurrent()).id; - if (typeof tabId === "undefined") { - return; - } + async resize(tabId: number, width: number, height: number): Promise { return this.consoleFrameClient.resize(tabId, width, height); } } -- cgit v1.2.3