Fix opening shopping ads on Android

README.md

@@ -257,8 +257,6 @@ The followings are the default values in the config file.
         4. Connect ADB to your phone wirelessly.
             * `adb connect <phone_ip>:5555`
 
-    * Shopping ads will still be opened on browser.
-
     * Watch an example run
 
         [![Send to Android](assets/send_to_android_cover.png)](https://youtu.be/XrZeECSXASg)

adb.py

@@ -52,11 +52,12 @@ def open_url(url: str, device_id: str) -> None:
                 "shell",
                 "am",
                 "start",
+                "-n",
+                "com.android.chrome/com.google.android.apps.chrome.Main",
                 "-a",
                 "android.intent.action.VIEW",
                 "-d",
                 url,
-                "com.android.chrome",
             ]
             result = subprocess.run(command, capture_output=True, text=True)
 

search_controller.py

@@ -26,7 +26,7 @@
 from config_reader import config
 from logger import logger
 from stats import SearchStats
-from utils import Direction, add_cookies, solve_recaptcha, get_random_sleep
+from utils import Direction, add_cookies, solve_recaptcha, get_random_sleep, resolve_redirect
 
 
 LinkElement = selenium.webdriver.remote.webelement.WebElement
@@ -222,9 +222,12 @@ def click_shopping_ads(self, shopping_ads: AdList) -> None:
                 if self._hooks_enabled:
                     hooks.before_ad_click_hook(self._driver)
 
-                self._handle_browser_click(
-                    ad_link_element, ad_link, True, original_window_handle, category="Shopping"
-                )
+                if config.behavior.send_to_android and self._android_device_id:
+                    self._handle_android_click(ad_link_element, ad_link, True, category="Shopping")
+                else:
+                    self._handle_browser_click(
+                        ad_link_element, ad_link, True, original_window_handle, category="Shopping"
+                    )
 
             except Exception:
                 logger.debug(f"Failed to click ad element [{ad_title}]!")
@@ -280,6 +283,8 @@ def _extract_link_info(self, link: Any, is_ad_element: bool) -> tuple:
         :param link: (ad, ad_link, ad_title) for ads LinkElement for non-ads
         :type is_ad_element: bool
         :param is_ad_element: Whether it is an ad or non-ad link
+        :rtype: tuple
+        :returns: (link_element, link_url, ad_title) tuple
         """
 
         if is_ad_element:
@@ -314,6 +319,8 @@ def _handle_android_click(
 
         url = link_url if category == "Shopping" else link_element.get_attribute("href")
 
+        url = resolve_redirect(url)
+
         adb_controller.open_url(url, self._android_device_id)
 
         click_time = datetime.now().strftime("%H:%M:%S")

utils.py

@@ -606,3 +606,21 @@ def get_locale_language(country_code: str) -> str:
     logger.debug(f"Locale language code for {country_code}: {locale_language[0]}")
 
     return locale_language
+
+
+def resolve_redirect(url: str) -> str:
+    """Resolve any redirects and return the final destination URL
+
+    :type url: str
+    :param url: Input url to resolve
+    :rtype: str
+    :returns: Final destination URL
+    """
+
+    try:
+        response = requests.get(url, allow_redirects=True)
+        return response.url
+
+    except requests.RequestException as exp:
+        logger.error(f"Error resolving URL redirection: {exp}")
+        return url