-
Notifications
You must be signed in to change notification settings - Fork 9
Refresh browser pool guides and standardize on "browser pools" #464
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
andrewleesteele
wants to merge
27
commits into
main
Choose a base branch
from
hypeship/browser-pools-docs-refresh
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
45cf958
Refresh browser pool guides and standardize on "browser pools"
andrewleesteele bf5ab05
Add reuse-isolation, throughput, and profile guidance to pools docs
andrewleesteele 0ca62af
Remove the browser-pools quickstart, point entry links to the overview
andrewleesteele ad048aa
Reframe home-page pools blurb: production step, not a day-one default
andrewleesteele 8f38449
Fold the browser-pools FAQ into the overview and delete it
andrewleesteele fd9c595
Spell out "browser pool" in prose; stop using bare "pool"
andrewleesteele 802f277
Merge remote-tracking branch 'origin/main' into hypeship/browser-pool…
andrewleesteele f2bc19e
Collapse browser pools to a single page at /browsers/pools
andrewleesteele 1a43c11
Disambiguate non-browser-pool "pool" mentions
andrewleesteele bb8cca9
Soften the pool-sizing threshold and add a Limitations section
andrewleesteele 042447f
Drop the ~50 threshold from pool sizing guidance
andrewleesteele b6987e8
Rewrite the Scale page around benefit → when-to-use, soften home CTA
andrewleesteele 8618c4e
Stop describing browser pools as "warm"
andrewleesteele 58517c6
Tidy the Browser Pools page description
andrewleesteele 99e5ce2
Reword pools intro: configure a set of identical browsers
andrewleesteele 6b87a04
Dedupe the Browser Pools page
andrewleesteele 59fcb57
Point the profiles cross-link at the whole pools profiles section
andrewleesteele bfe3a06
Flag the profile constraints in the pools intro
andrewleesteele 0f19f9f
Tighten the Browser Pools page to behavior and reference
andrewleesteele 52b75d5
Trim the pools page to behavior and the operations that carry it
andrewleesteele 6f322a2
Reorder pools sections and trim the limitations to pointers
andrewleesteele d75d11b
Apply review feedback to the pools and pricing pages
andrewleesteele f6abf40
Polish the pools page after review
andrewleesteele a4c7e45
Open the pools page with a definition
andrewleesteele d67b61c
Tighten browser pools copy and fix update-size docs
andrewleesteele 327ca95
Replace "the right tool" in the Scale intro
andrewleesteele cd11471
Add pools/faq redirect and drop remaining reserved-browsers wording
andrewleesteele File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is connection pool the same as browser pool?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"connection pool" refers to chromium's internal socket pool (HTTP/1.x sockets), unrelated to browser pools.