Skip to content

Conversation

@shai-almog
Copy link
Collaborator

No description provided.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 117 to 120
} catch (Throwable ex) {
Log.e(ex);
} finally {
queued = false;
Util.cleanup(i);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Reset queued flag when background load fails

The background load in StorageImageAsync.getImageData() now clears queued only inside the serially executed UI callback, but the finally block no longer resets it. If Storage.getInstance().readObject(fileName) throws before the callback is scheduled, the code reaches this catch/finally without ever clearing queued. Subsequent calls will keep returning early because queued is still true, so the image never retries loading after a transient storage failure. Clearing queued in the error path or finally block avoids permanently disabling future loads.

Useful? React with 👍 / 👎.

@shai-almog
Copy link
Collaborator Author

shai-almog commented Nov 12, 2025

Android screenshot updates

  • BrowserComponent — updated screenshot. Screenshot differs (320x616 px, bit depth 8).

    BrowserComponent
    Preview info: JPEG preview quality 70; JPEG preview quality 70.
    Full-resolution PNG saved as BrowserComponent.png in workflow artifacts.

@shai-almog
Copy link
Collaborator Author

shai-almog commented Nov 12, 2025

✅ Native iOS screenshot tests passed.

@github-actions
Copy link

✅ Continuous Quality Report

Test & Coverage

Static Analysis

Generated automatically by the PR CI workflow.

@shai-almog shai-almog merged commit c5b1d0c into master Nov 13, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants