Skip to content

Commit

Permalink
Merge pull request #192 from Polymer/unique-browser
Browse files Browse the repository at this point in the history
Add all browser config parameters to signature
  • Loading branch information
aomarks authored Sep 18, 2020
2 parents be7a23d + f97c344 commit 08920d8
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
6 changes: 5 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,11 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](http://keepachangelog.com/) and this
project adheres to [Semantic Versioning](http://semver.org/).

<!-- ## Unreleased -->
## Unreleased

- Fix bug where a unique browser would not be launched if the only difference
between browser configurations was `binary`, `addArguments`,
`removeArguments`, `cpuThrottlingRate` or `preferences`.

## [0.5.3] 2020-09-11

Expand Down
7 changes: 6 additions & 1 deletion src/browser.ts
Original file line number Diff line number Diff line change
Expand Up @@ -85,9 +85,14 @@ export function browserSignature(config: BrowserConfig): string {
return JSON.stringify([
config.name,
config.headless,
config.remoteUrl || '',
config.remoteUrl ?? '',
config.windowSize.width,
config.windowSize.height,
config.binary ?? '',
config.addArguments ?? [],
config.removeArguments ?? [],
config.cpuThrottlingRate ?? 1,
config.preferences ?? {},
]);
}

Expand Down

0 comments on commit 08920d8

Please sign in to comment.