Add ICU search and prepaint pre-decision boundary - #19
Draft
isomorphisms wants to merge 1 commit into
Draft
Conversation
isomorphisms
changed the base branch from
resource-constrained-information-renderer
to
main
August 25, 2026 13:51
isomorphisms
force-pushed
the
prepaint-network-predecision
branch
from
August 25, 2026 13:57
4589034 to
511a436
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
prepaint_textentry point for ordinary text and URL-list pre-paintsprepaint_full_textpath and regression proving article<p>bodies survive extraction<a href>nested inside a paragraph without dropping the text before or after it<a href><img ...></a>as one image item carrying source, alternate text, caption/title, and link target+) and runsicu get https://www.google.com/search?q=...digfor A/NS evidence andtraceroutefor a sampled eight-hop routeImportant limits
digandtracerouteare opt-in/sampled diagnostics, not every-navigation work. Route latency is not bandwidth and cannot predict total page load. The only five-minute decision in this stub uses explicit known asset bytes divided by an explicit assumed/measured byte rate; missing evidence yields “measure assets before heavy renderer.”The Android display APK remains network-free. This branch defines the ICU handoff that the integrated IB shell will later connect to the phone surface.
Remote images are not guessed from filename extensions. The extractor retains the remote
srcplus any enclosing anchor. The background path must fetch with ICU, validate response type/bytes, and replace that source with an already-fetched local/content source in a later pre-paint. The linked-image display field is implemented in sibling PR #16.Current architecture boundary
This branch now sits directly on
mainrather than on merged PR #15. It preserves the current ownership split:InformationView, HTML/text extraction, URL-candidate recognition, and the renderer-neutral linked-image record.The current centralized browser-foundation workflow invokes these tests through
bin/ci_browser_foundation.grease; the branch does not restore the older inline CI orchestration.Verification
At rebased head
511a436418a7abf66b6272d90a8793191ae2a86f, directly onmainc06dea7d18fa804f36e6fbe7e3b46a05347368ac, GitHub Actions run32856520033is green:getplussmall+fast+browserURLThe branch is now a clean sibling on
main; it is no longer stacked on PR #15.