fix: Address Copilot review comments for VshareWebActivity

- Fix navigation handling: Return false in shouldOverrideUrlLoading to let WebView handle standard navigation
- Add security settings: Disable file/content access and enable Safe Browsing for API >= 26
- Fix theme switching: Compute URL in onCreate to handle live theme changes
- Remove redundant progress bar logic: Consolidate visibility handling in WebChromeClient
- Optimize cache handling: Use clearCache(false) to avoid purging shared cache
- Fix layout theming: Use app-themed attribute (page_bg_color) instead of android:attr

All issues identified by GitHub Copilot have been addressed.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: graycreate

app/src/main/java/me/ghui/v2er/module/vshare/VshareWebActivity.java

@@ -28,7 +28,6 @@
  */
 public class VshareWebActivity extends AppCompatActivity {
 
-    private static final String KEY_URL = "url";
     private static final String VSHARE_BASE_URL = "https://v2er.app/vshare";
 
     @BindView(R.id.webview)
@@ -39,14 +38,6 @@ public class VshareWebActivity extends AppCompatActivity {
 
     public static void open(Context context) {
         Intent intent = new Intent(context, VshareWebActivity.class);
-        // Append theme parameter based on current app theme
-        String url = VSHARE_BASE_URL;
-        if (DarkModelUtils.isDarkMode()) {
-            url += "?theme=dark";
-        } else {
-            url += "?theme=light";
-        }
-        intent.putExtra(KEY_URL, url);
         context.startActivity(intent);
     }
 
@@ -69,11 +60,14 @@ protected void onCreate(Bundle savedInstanceState) {
 
         setupWebView();
 
-        // Load the URL
-        String url = getIntent().getStringExtra(KEY_URL);
-        if (url != null) {
-            mWebView.loadUrl(url);
+        // Compute URL with theme parameter based on current app theme
+        String url = VSHARE_BASE_URL;
+        if (DarkModelUtils.isDarkMode()) {
+            url += "?theme=dark";
+        } else {
+            url += "?theme=light";
         }
+        mWebView.loadUrl(url);
     }
 
     @SuppressLint("SetJavaScriptEnabled")
@@ -86,6 +80,15 @@ private void setupWebView() {
         // Enable DOM storage
         settings.setDomStorageEnabled(true);
 
+        // Disable file and content access for security
+        settings.setAllowFileAccess(false);
+        settings.setAllowContentAccess(false);
+
+        // Enable Safe Browsing if API >= 26
+        if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.O) {
+            settings.setSafeBrowsingEnabled(true);
+        }
+
         // Enable responsive layout
         settings.setUseWideViewPort(true);
         settings.setLoadWithOverviewMode(true);
@@ -102,9 +105,8 @@ private void setupWebView() {
         mWebView.setWebViewClient(new WebViewClient() {
             @Override
             public boolean shouldOverrideUrlLoading(WebView view, WebResourceRequest request) {
-                // Keep navigation within the WebView
-                view.loadUrl(request.getUrl().toString());
-                return true;
+                // Let the WebView handle the navigation
+                return false;
             }
 
             @Override
@@ -116,7 +118,6 @@ public void onPageStarted(WebView view, String url, Bitmap favicon) {
             @Override
             public void onPageFinished(WebView view, String url) {
                 super.onPageFinished(view, url);
-                mProgressBar.setVisibility(View.GONE);
 
                 // Inject CSS to ensure proper theme is applied
                 String theme = DarkModelUtils.isDarkMode() ? "dark" : "light";
@@ -153,7 +154,7 @@ public void onBackPressed() {
     protected void onDestroy() {
         if (mWebView != null) {
             mWebView.clearHistory();
-            mWebView.clearCache(true);
+            mWebView.clearCache(false);
             mWebView.loadUrl("about:blank");
             mWebView.pauseTimers();
             mWebView.destroy();

app/src/main/res/layout/activity_vshare_web.xml

@@ -2,7 +2,7 @@
 <FrameLayout xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
-    android:background="?android:attr/windowBackground">
+    android:background="?attr/page_bg_color">
 
     <WebView
         android:id="@+id/webview"