Skip to content

feat: add getBackgroundThrottling() + backgroundThrottling property#21036

Merged
alexeykuzmin merged 1 commit into
masterfrom
miniak/background-throttling-property
May 14, 2020
Merged

feat: add getBackgroundThrottling() + backgroundThrottling property#21036
alexeykuzmin merged 1 commit into
masterfrom
miniak/background-throttling-property

Conversation

@miniak

@miniak miniak commented Nov 7, 2019

Copy link
Copy Markdown
Contributor

Description of Change

Add contents.getBackgroundThrottling() method and contents.backgroundThrottling property

Checklist

Release Notes

Notes: Added contents.getBackgroundThrottling() method and contents.backgroundThrottling property.

@miniak miniak added the wip ⚒ label Nov 7, 2019
@miniak miniak self-assigned this Nov 7, 2019
@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label Nov 7, 2019
@miniak

miniak commented Nov 7, 2019

Copy link
Copy Markdown
Contributor Author

@codebytere should the property also be documented in BrowserWindow? The setBackgroundThrottling method is not, but it's available and forwards it to the webContents.

@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label Nov 8, 2019

@codebytere codebytere left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@miniak we made a decision on the propertyification a while back - we will be supporting both approaches (meaning that we will add support for ES6 get/set while also keeping previous getX()/setX(newX) functionality. If you can update this accordingly we can move it forward.

@miniak miniak force-pushed the miniak/background-throttling-property branch from 11c38df to 339d3f8 Compare May 11, 2020 16:17
@miniak miniak changed the title feat: convert contents.setBackgroundThrottling() to a property + add getter May 11, 2020
@miniak miniak requested a review from codebytere May 11, 2020 16:17
@miniak miniak marked this pull request as ready for review May 11, 2020 16:17
@miniak miniak removed the wip ⚒ label May 11, 2020
@miniak miniak force-pushed the miniak/background-throttling-property branch from 339d3f8 to 762aa31 Compare May 11, 2020 17:18
@miniak miniak changed the title feat: add contents.getBackgroundThrottling() + backgroundThrottling property May 11, 2020
@miniak miniak added the wip ⚒ label May 11, 2020
@miniak miniak force-pushed the miniak/background-throttling-property branch from 762aa31 to 1d1eebf Compare May 13, 2020 14:02
@miniak miniak removed the wip ⚒ label May 13, 2020
Comment thread spec-main/api-web-contents-spec.ts Outdated
Comment thread spec-main/api-web-contents-spec.ts

@alexeykuzmin alexeykuzmin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Check the tests.

@miniak miniak force-pushed the miniak/background-throttling-property branch from 1d1eebf to bae5677 Compare May 13, 2020 18:21
@miniak miniak requested a review from alexeykuzmin May 13, 2020 18:22
Comment thread spec-main/api-web-contents-spec.ts
Comment thread spec-main/api-web-contents-spec.ts
@alexeykuzmin alexeykuzmin merged commit 4afc756 into master May 14, 2020
@release-clerk

release-clerk Bot commented May 14, 2020

Copy link
Copy Markdown

Release Notes Persisted

Added contents.getBackgroundThrottling() method and contents.backgroundThrottling property.

@alexeykuzmin alexeykuzmin deleted the miniak/background-throttling-property branch May 14, 2020 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants