Clean up debug information

Author: agg23

playwright/_impl/_assertions.py

@@ -51,7 +51,6 @@ async def _expect_impl(
         expect_options: FrameExpectOptions,
         expected: Any,
         message: str,
-        # title: str,
     ) -> None:
         __tracebackhide__ = True
         expect_options["isNot"] = self._is_not
@@ -61,7 +60,6 @@ async def _expect_impl(
             message = message.replace("expected to", "expected not to")
         if "useInnerText" in expect_options and expect_options["useInnerText"] is None:
             del expect_options["useInnerText"]
-        # result = await self._actual_locator._expect(expression, expect_options, title)
         result = await self._actual_locator._expect(expression, expect_options)
         if result["matches"] == self._is_not:
             actual = result.get("received")
@@ -125,10 +123,7 @@ async def to_have_url(
         base_url = self._actual_page.context._options.get("baseURL")
         if isinstance(urlOrRegExp, str) and base_url:
             urlOrRegExp = urljoin(base_url, urlOrRegExp)
-            print("Changed", urlOrRegExp)
-        print("Expecting URL", urlOrRegExp)
         expected_text = to_expected_text_values([urlOrRegExp], ignoreCase=ignoreCase)
-        print("Expected text", expected_text)
         await self._expect_impl(
             "to.have.url",
             FrameExpectOptions(expectedText=expected_text, timeout=timeout),

playwright/_impl/_browser.py

@@ -225,7 +225,6 @@ async def inner() -> Page:
             context._owner_page = page
             return page
 
-        # TODO: Args
         return await self._connection.wrap_api_call(inner)
 
     async def close(self, reason: str = None) -> None:

playwright/_impl/_browser_context.py

@@ -315,7 +315,7 @@ async def _initialize_har_from_options(
         record_har_path = str(record_har_path)
         if len(record_har_path) == 0:
             return
-        # Forcibly list type to satisfy mypy
+        # Forcibly provide type to satisfy mypy
         default_policy: HarContentPolicy = (
             "attach" if record_har_path.endswith(".zip") else "embed"
         )
@@ -570,24 +570,16 @@ def expect_event(
 
     def _on_close(self) -> None:
         self._closing_or_closed = True
-        print("Closing")
         if self._browser:
             try:
                 self._browser._contexts.remove(self)
             except KeyError:
-                print("Context already removed from browser contexts")
                 pass
             try:
-                print("Removing context from browser selectors")
                 self._browser._browser_type._playwright.selectors._contextsForSelectors.remove(
                     self
                 )
-                print(
-                    "Successfully removed context from browser selectors",
-                    self._browser._browser_type._playwright.selectors._contextsForSelectors,
-                )
             except KeyError:
-                print("Context already removed from browser selectors")
                 pass
 
         self._dispose_har_routers()
@@ -596,9 +588,7 @@ def _on_close(self) -> None:
 
     async def close(self, reason: str = None) -> None:
         if self._closing_or_closed:
-            print("Context already closed", self)
             return
-        print(f"Closing context {self}, contexts: {self._browser._contexts}")  # type: ignore
         self._close_reason = reason
         self._closing_or_closed = True
 

playwright/_impl/_selectors.py

@@ -44,9 +44,6 @@ async def register(
         engine: Dict[str, Any] = dict(name=name, source=script)
         if contentScript:
             engine["contentScript"] = contentScript
-        print(
-            f"Registering selector engine: {name} with contexts {self._contextsForSelectors}"
-        )
         for context in self._contextsForSelectors:
             await context._channel.send(
                 "registerSelectorEngine", dict(selectorEngine=engine)