Skip to content

Commit

Permalink
First quick check if lastEmittedState and appState are different
Browse files Browse the repository at this point in the history
Resolves brave#10195

Auditors: @bridiver @bbondy

Test Plan:
- open two windows
- add bookmark in one
- bookmark should appear in both bookmark toolbars
  • Loading branch information
NejcZdovc authored and dfperry5 committed Aug 18, 2017
1 parent 3c0afc2 commit 69fba5e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion js/stores/appStore.js
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ class AppStore extends EventEmitter {
}

emitChanges () {
if (this.lastEmittedState) {
if (this.lastEmittedState && this.lastEmittedState !== appState) {
const d = diff(this.lastEmittedState, appState)
if (!d.isEmpty()) {
BrowserWindow.getAllWindows().forEach((wnd) => {
Expand Down

1 comment on commit 69fba5e

@bridiver
Copy link

Choose a reason for hiding this comment

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

++

Please sign in to comment.