Data: Avoid calling listeners on unchanging state (#5339)

data/index.js

@@ -72,7 +72,19 @@ export function registerReducer( reducerKey, reducer ) {
 	}
 	const store = createStore( reducer, flowRight( enhancers ) );
 	stores[ reducerKey ] = store;
-	store.subscribe( globalListener );
+
+	// Customize subscribe behavior to call listeners only on effective change,
+	// not on every dispatch.
+	let lastState = store.getState();
+	store.subscribe( () => {
+		const state = store.getState();
+		const hasChanged = state !== lastState;
+		lastState = state;
+
+		if ( hasChanged ) {
+			globalListener();
+		}
+	} );
 
 	return store;
 }

data/test/index.js

@@ -253,6 +253,26 @@ describe( 'withSelect', () => {
 
 			store.dispatch( { type: 'increment' } );
 		} );
+
+		itWithExtraAssertions( 'should not rerun selection on unchanging state', () => {
+			const store = registerReducer( 'unchanging', ( state = {} ) => state );
+
+			registerSelectors( 'unchanging', {
+				getState: ( state ) => state,
+			} );
+
+			const mapSelectToProps = jest.fn();
+
+			const Component = compose( [
+				withSelectImplementation( mapSelectToProps ),
+			] )( () => <div /> );
+
+			wrapper = mount( <Component /> );
+
+			store.dispatch( { type: 'dummy' } );
+
+			expect( mapSelectToProps ).toHaveBeenCalledTimes( 1 );
+		} );
 	}
 
 	cases( withSelect );
@@ -380,6 +400,16 @@ describe( 'subscribe', () => {
 
 		expect( secondListener ).toHaveBeenCalled();
 	} );
+
+	it( 'does not call listeners if state has not changed', () => {
+		const store = registerReducer( 'unchanging', ( state = {} ) => state );
+		const listener = jest.fn();
+		subscribeWithUnsubscribe( listener );
+
+		store.dispatch( { type: 'dummy' } );
+
+		expect( listener ).not.toHaveBeenCalled();
+	} );
 } );
 
 describe( 'dispatch', () => {