Skip to content

refactor: try just using regular [Sync] for MessageSync#21776

Merged
zcbenz merged 1 commit into7-1-xfrom
sync-hang-fix
Jan 22, 2020
Merged

refactor: try just using regular [Sync] for MessageSync#21776
zcbenz merged 1 commit into7-1-xfrom
sync-hang-fix

Conversation

@MarshallOfSound
Copy link
Member

@MarshallOfSound MarshallOfSound commented Jan 14, 2020

Backport of #20797

See that PR for details.

Notes: fixed potential hang when sending syncronous IPC messages on process shutdown

@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jan 14, 2020
@trop trop bot added 7-1-x backport This is a backport PR labels Jan 14, 2020
@nornagon nornagon requested a review from zcbenz January 14, 2020 22:25
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jan 14, 2020
@zcbenz
Copy link
Contributor

zcbenz commented Jan 15, 2020

Did you accidentally pushed the "tmp lots of CI" commit?

@zcbenz
Copy link
Contributor

zcbenz commented Jan 20, 2020

@MarshallOfSound Can you remove the "tmp lots of CI" commit?

Copy link
Contributor

@nornagon nornagon left a comment

Choose a reason for hiding this comment

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

let's give it a go

@nornagon
Copy link
Contributor

(make sure to kill the CI patch before merging)

@nornagon
Copy link
Contributor

also maybe some release notes?

@MarshallOfSound MarshallOfSound added fast-track 🚅 Indicates that this PR is intended to bypass the 24 hour rule. Needs approval from Releases and removed wip ⚒ labels Jan 21, 2020
@zcbenz zcbenz merged commit ef94a79 into 7-1-x Jan 22, 2020
@release-clerk
Copy link

release-clerk bot commented Jan 22, 2020

Release Notes Persisted

fixed potential hang when sending syncronous IPC messages on process shutdown

@zcbenz zcbenz deleted the sync-hang-fix branch January 22, 2020 01:39
tessus added a commit to tessus/joplin that referenced this pull request Feb 8, 2020
This is only a minor bump but several issues have been fixed since 7.1.9:

- Fixed an edge case in checkbox logic on Windows. electron/electron#21860
- Fixed an issue where window.print() only worked once on a single BrowserWindow. electron/electron#21911
- Fixed an issue where the credits set in About Panel credits were not dark mode aware on macOS. electron/electron#21924
- Fixed error thrown when importing powerMonitor on Linux before app's 'ready' event. electron/electron#21941
- Fixed fuzzy font rendering when hot-plugging displays on macOS Catalina. electron/electron#21872
- Fixed BrowserWindow.setFocusable(true) not working on Windows. electron/electron#21855
- Fixed set-cookie header not passed in net module. electron/electron#21770
- Fixed an issue where custom stream protocols would sometimes not complete responses when the data stream ended. electron/electron#21758
- Fixed crash when restoring minimized hidden window on Windows. electron/electron#21820
- Fixed issue where non-zero size pixels in CSS styles could be rounded down to zero size pixels. electron/electron#21857
- Fixed memory leak when using javascript generator functions. electron/electron#21773
- Fixed potential hang when sending synchronous IPC messages on process shutdown. electron/electron#21776
tessus added a commit to tessus/joplin that referenced this pull request Feb 12, 2020
This is only a minor bump but several issues have been fixed since 7.1.9:

- Fixed an issue where sending complex objects over IPC could in some cases cause the renderer process to be terminated. electron/electron#21922
- Fixed crash with Date.toLocaleString for invalid locale and locale of the format aa@BB. electron/electron#21969
- Fixed flash plugin not working. electron/electron#22109
- Fixed issue where renderers could crash during GC when using the contextBridge module. electron/electron#22112
- Fixed netLog.stopLogging returning undefined instead of the path to the log. electron/electron#21988
- Fixed an edge case in checkbox logic on Windows. electron/electron#21860
- Fixed an issue where window.print() only worked once on a single BrowserWindow. electron/electron#21911
- Fixed an issue where the credits set in About Panel credits were not dark mode aware on macOS. electron/electron#21924
- Fixed error thrown when importing powerMonitor on Linux before app's 'ready' event. electron/electron#21941
- Fixed fuzzy font rendering when hot-plugging displays on macOS Catalina. electron/electron#21872
- Fixed BrowserWindow.setFocusable(true) not working on Windows. electron/electron#21855
- Fixed set-cookie header not passed in net module. electron/electron#21770
- Fixed an issue where custom stream protocols would sometimes not complete responses when the data stream ended. electron/electron#21758
- Fixed crash when restoring minimized hidden window on Windows. electron/electron#21820
- Fixed issue where non-zero size pixels in CSS styles could be rounded down to zero size pixels. electron/electron#21857
- Fixed memory leak when using javascript generator functions. electron/electron#21773
- Fixed potential hang when sending synchronous IPC messages on process shutdown. electron/electron#21776
laurent22 pushed a commit to laurent22/joplin that referenced this pull request Feb 13, 2020
This is only a minor bump but several issues have been fixed since 7.1.9:

- Fixed an issue where sending complex objects over IPC could in some cases cause the renderer process to be terminated. electron/electron#21922
- Fixed crash with Date.toLocaleString for invalid locale and locale of the format aa@BB. electron/electron#21969
- Fixed flash plugin not working. electron/electron#22109
- Fixed issue where renderers could crash during GC when using the contextBridge module. electron/electron#22112
- Fixed netLog.stopLogging returning undefined instead of the path to the log. electron/electron#21988
- Fixed an edge case in checkbox logic on Windows. electron/electron#21860
- Fixed an issue where window.print() only worked once on a single BrowserWindow. electron/electron#21911
- Fixed an issue where the credits set in About Panel credits were not dark mode aware on macOS. electron/electron#21924
- Fixed error thrown when importing powerMonitor on Linux before app's 'ready' event. electron/electron#21941
- Fixed fuzzy font rendering when hot-plugging displays on macOS Catalina. electron/electron#21872
- Fixed BrowserWindow.setFocusable(true) not working on Windows. electron/electron#21855
- Fixed set-cookie header not passed in net module. electron/electron#21770
- Fixed an issue where custom stream protocols would sometimes not complete responses when the data stream ended. electron/electron#21758
- Fixed crash when restoring minimized hidden window on Windows. electron/electron#21820
- Fixed issue where non-zero size pixels in CSS styles could be rounded down to zero size pixels. electron/electron#21857
- Fixed memory leak when using javascript generator functions. electron/electron#21773
- Fixed potential hang when sending synchronous IPC messages on process shutdown. electron/electron#21776
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

7-1-x backport This is a backport PR fast-track 🚅 Indicates that this PR is intended to bypass the 24 hour rule. Needs approval from Releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants