Skip to content

Commit 553f38f

Browse files
rickhanloniirickhanlonii
committed
Delete SyncBatchedLane
1 parent 25bfa28 commit 553f38f

File tree

7 files changed

+200
-225
lines changed

7 files changed

+200
-225
lines changed

‎packages/react-devtools-scheduling-profiler/src/constants.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,4 @@ export {
1212
COMPACT_LINE_HEIGHT,
1313
} from 'react-devtools-shared/src/constants.js';
1414

15-
export const REACT_TOTAL_NUM_LANES = 31;
15+
export const REACT_TOTAL_NUM_LANES = 30;

‎packages/react-devtools-scheduling-profiler/src/import-worker/__tests__/preprocessData-test.internal.js‎

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ describe(getLanesFromTransportDecimalBitmask, () => {
4242
it('should ignore lanes outside REACT_TOTAL_NUM_LANES', () => {
4343
// Sanity check; this test may need to be updated when the no. of fiber
4444
// lanes are changed.
45-
expect(REACT_TOTAL_NUM_LANES).toBe(31);
45+
expect(REACT_TOTAL_NUM_LANES).toBe(30);
4646

4747
expect(
4848
getLanesFromTransportDecimalBitmask(
@@ -404,7 +404,7 @@ describe(preprocessData, () => {
404404
{
405405
componentStack: '',
406406
laneLabels: ['Default'],
407-
lanes: [5],
407+
lanes: [4],
408408
timestamp: 0.005,
409409
type: 'schedule-render',
410410
},
@@ -413,7 +413,7 @@ describe(preprocessData, () => {
413413
componentStack: '',
414414
isCascading: false,
415415
laneLabels: ['Default'],
416-
lanes: [5],
416+
lanes: [4],
417417
timestamp: 0.013,
418418
type: 'schedule-state-update',
419419
},
@@ -425,7 +425,7 @@ describe(preprocessData, () => {
425425
depth: 0,
426426
duration: 0.004999999999999999,
427427
laneLabels: ['Default'],
428-
lanes: [5],
428+
lanes: [4],
429429
timestamp: 0.006,
430430
type: 'render-idle',
431431
},
@@ -434,7 +434,7 @@ describe(preprocessData, () => {
434434
depth: 0,
435435
duration: 0.001,
436436
laneLabels: ['Default'],
437-
lanes: [5],
437+
lanes: [4],
438438
timestamp: 0.006,
439439
type: 'render',
440440
},
@@ -443,7 +443,7 @@ describe(preprocessData, () => {
443443
depth: 0,
444444
duration: 0.002999999999999999,
445445
laneLabels: ['Default'],
446-
lanes: [5],
446+
lanes: [4],
447447
timestamp: 0.008,
448448
type: 'commit',
449449
},
@@ -452,7 +452,7 @@ describe(preprocessData, () => {
452452
depth: 1,
453453
duration: 0.0010000000000000009,
454454
laneLabels: ['Default'],
455-
lanes: [5],
455+
lanes: [4],
456456
timestamp: 0.009,
457457
type: 'layout-effects',
458458
},
@@ -461,7 +461,7 @@ describe(preprocessData, () => {
461461
depth: 0,
462462
duration: 0.002,
463463
laneLabels: ['Default'],
464-
lanes: [5],
464+
lanes: [4],
465465
timestamp: 0.012,
466466
type: 'passive-effects',
467467
},
@@ -470,7 +470,7 @@ describe(preprocessData, () => {
470470
depth: 0,
471471
duration: 0.005000000000000001,
472472
laneLabels: ['Default'],
473-
lanes: [5],
473+
lanes: [4],
474474
timestamp: 0.015,
475475
type: 'render-idle',
476476
},
@@ -479,7 +479,7 @@ describe(preprocessData, () => {
479479
depth: 0,
480480
duration: 0.0010000000000000009,
481481
laneLabels: ['Default'],
482-
lanes: [5],
482+
lanes: [4],
483483
timestamp: 0.015,
484484
type: 'render',
485485
},
@@ -488,7 +488,7 @@ describe(preprocessData, () => {
488488
depth: 0,
489489
duration: 0.002999999999999999,
490490
laneLabels: ['Default'],
491-
lanes: [5],
491+
lanes: [4],
492492
timestamp: 0.017,
493493
type: 'commit',
494494
},
@@ -497,7 +497,7 @@ describe(preprocessData, () => {
497497
depth: 1,
498498
duration: 0.0010000000000000009,
499499
laneLabels: ['Default'],
500-
lanes: [5],
500+
lanes: [4],
501501
timestamp: 0.018,
502502
type: 'layout-effects',
503503
},
@@ -506,7 +506,7 @@ describe(preprocessData, () => {
506506
depth: 0,
507507
duration: 0.0009999999999999974,
508508
laneLabels: ['Default'],
509-
lanes: [5],
509+
lanes: [4],
510510
timestamp: 0.021,
511511
type: 'passive-effects',
512512
},

‎packages/react-reconciler/src/ReactFiberLane.new.js‎

Lines changed: 43 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,7 @@ import {
4646
NoPriority as NoSchedulerPriority,
4747
} from './SchedulerWithReactIntegration.new';
4848

49-
export const SyncLanePriority: LanePriority = 13;
50-
export const SyncBatchedLanePriority: LanePriority = 12;
49+
export const SyncLanePriority: LanePriority = 12;
5150

5251
const InputContinuousHydrationLanePriority: LanePriority = 11;
5352
export const InputContinuousLanePriority: LanePriority = 10;
@@ -72,55 +71,54 @@ export const NoLanePriority: LanePriority = 0;
7271
// Lane values below should be kept in sync with getLabelsForLanes(), used by react-devtools-scheduling-profiler.
7372
// If those values are changed that package should be rebuilt and redeployed.
7473

75-
export const TotalLanes = 31;
76-
77-
export const NoLanes: Lanes = /* */ 0b0000000000000000000000000000000;
78-
export const NoLane: Lane = /* */ 0b0000000000000000000000000000000;
79-
80-
export const SyncLane: Lane = /* */ 0b0000000000000000000000000000001;
81-
export const SyncBatchedLane: Lane = /* */ 0b0000000000000000000000000000010;
82-
83-
const InputContinuousHydrationLane: Lane = /* */ 0b0000000000000000000000000000100;
84-
export const InputContinuousLane: Lanes = /* */ 0b0000000000000000000000000001000;
85-
86-
export const DefaultHydrationLane: Lane = /* */ 0b0000000000000000000000000010000;
87-
export const DefaultLane: Lanes = /* */ 0b0000000000000000000000000100000;
88-
89-
const TransitionHydrationLane: Lane = /* */ 0b0000000000000000000000001000000;
90-
const TransitionLanes: Lanes = /* */ 0b0000000011111111111111110000000;
91-
const TransitionLane1: Lane = /* */ 0b0000000000000000000000010000000;
92-
const TransitionLane2: Lane = /* */ 0b0000000000000000000000100000000;
93-
const TransitionLane3: Lane = /* */ 0b0000000000000000000001000000000;
94-
const TransitionLane4: Lane = /* */ 0b0000000000000000000010000000000;
95-
const TransitionLane5: Lane = /* */ 0b0000000000000000000100000000000;
96-
const TransitionLane6: Lane = /* */ 0b0000000000000000001000000000000;
97-
const TransitionLane7: Lane = /* */ 0b0000000000000000010000000000000;
98-
const TransitionLane8: Lane = /* */ 0b0000000000000000100000000000000;
99-
const TransitionLane9: Lane = /* */ 0b0000000000000001000000000000000;
100-
const TransitionLane10: Lane = /* */ 0b0000000000000010000000000000000;
101-
const TransitionLane11: Lane = /* */ 0b0000000000000100000000000000000;
102-
const TransitionLane12: Lane = /* */ 0b0000000000001000000000000000000;
103-
const TransitionLane13: Lane = /* */ 0b0000000000010000000000000000000;
104-
const TransitionLane14: Lane = /* */ 0b0000000000100000000000000000000;
105-
const TransitionLane15: Lane = /* */ 0b0000000001000000000000000000000;
106-
const TransitionLane16: Lane = /* */ 0b0000000010000000000000000000000;
107-
108-
const RetryLanes: Lanes = /* */ 0b0000111100000000000000000000000;
109-
const RetryLane1: Lane = /* */ 0b0000000100000000000000000000000;
110-
const RetryLane2: Lane = /* */ 0b0000001000000000000000000000000;
111-
const RetryLane3: Lane = /* */ 0b0000010000000000000000000000000;
112-
const RetryLane4: Lane = /* */ 0b0000100000000000000000000000000;
74+
export const TotalLanes = 30;
75+
76+
export const NoLanes: Lanes = /* */ 0b000000000000000000000000000000;
77+
export const NoLane: Lane = /* */ 0b000000000000000000000000000000;
78+
79+
export const SyncLane: Lane = /* */ 0b000000000000000000000000000001;
80+
81+
const InputContinuousHydrationLane: Lane = /* */ 0b000000000000000000000000000010;
82+
export const InputContinuousLane: Lanes = /* */ 0b000000000000000000000000000100;
83+
84+
export const DefaultHydrationLane: Lane = /* */ 0b000000000000000000000000001000;
85+
export const DefaultLane: Lanes = /* */ 0b000000000000000000000000010000;
86+
87+
const TransitionHydrationLane: Lane = /* */ 0b000000000000000000000000100000;
88+
const TransitionLanes: Lanes = /* */ 0b000000001111111111111111000000;
89+
const TransitionLane1: Lane = /* */ 0b000000000000000000000001000000;
90+
const TransitionLane2: Lane = /* */ 0b000000000000000000000010000000;
91+
const TransitionLane3: Lane = /* */ 0b000000000000000000000100000000;
92+
const TransitionLane4: Lane = /* */ 0b000000000000000000001000000000;
93+
const TransitionLane5: Lane = /* */ 0b000000000000000000010000000000;
94+
const TransitionLane6: Lane = /* */ 0b000000000000000000100000000000;
95+
const TransitionLane7: Lane = /* */ 0b000000000000000001000000000000;
96+
const TransitionLane8: Lane = /* */ 0b000000000000000010000000000000;
97+
const TransitionLane9: Lane = /* */ 0b000000000000000100000000000000;
98+
const TransitionLane10: Lane = /* */ 0b000000000000001000000000000000;
99+
const TransitionLane11: Lane = /* */ 0b000000000000010000000000000000;
100+
const TransitionLane12: Lane = /* */ 0b000000000000100000000000000000;
101+
const TransitionLane13: Lane = /* */ 0b000000000001000000000000000000;
102+
const TransitionLane14: Lane = /* */ 0b000000000010000000000000000000;
103+
const TransitionLane15: Lane = /* */ 0b000000000100000000000000000000;
104+
const TransitionLane16: Lane = /* */ 0b000000001000000000000000000000;
105+
106+
const RetryLanes: Lanes = /* */ 0b000011110000000000000000000000;
107+
const RetryLane1: Lane = /* */ 0b000000010000000000000000000000;
108+
const RetryLane2: Lane = /* */ 0b000000100000000000000000000000;
109+
const RetryLane3: Lane = /* */ 0b000001000000000000000000000000;
110+
const RetryLane4: Lane = /* */ 0b000010000000000000000000000000;
113111

114112
export const SomeRetryLane: Lane = RetryLane1;
115113

116-
export const SelectiveHydrationLane: Lane = /* */ 0b0001000000000000000000000000000;
114+
export const SelectiveHydrationLane: Lane = /* */ 0b000100000000000000000000000000;
117115

118-
const NonIdleLanes = /* */ 0b0001111111111111111111111111111;
116+
const NonIdleLanes = /* */ 0b000111111111111111111111111111;
119117

120-
export const IdleHydrationLane: Lane = /* */ 0b0010000000000000000000000000000;
121-
const IdleLane: Lanes = /* */ 0b0100000000000000000000000000000;
118+
export const IdleHydrationLane: Lane = /* */ 0b001000000000000000000000000000;
119+
const IdleLane: Lanes = /* */ 0b010000000000000000000000000000;
122120

123-
export const OffscreenLane: Lane = /* */ 0b1000000000000000000000000000000;
121+
export const OffscreenLane: Lane = /* */ 0b100000000000000000000000000000;
124122

125123
// This function is used for the experimental scheduling profiler (react-devtools-scheduling-profiler)
126124
// It should be kept in sync with the Lanes values above.
@@ -130,9 +128,6 @@ export function getLabelsForLanes(lanes: Lanes): Array<string> | void {
130128
if (lanes & SyncLane) {
131129
labels.push('Sync');
132130
}
133-
if (lanes & SyncBatchedLane) {
134-
labels.push('SyncBatched');
135-
}
136131
if (lanes & InputContinuousHydrationLane) {
137132
labels.push('InputContinuousHydration');
138133
}
@@ -194,9 +189,6 @@ function getHighestPriorityLanes(lanes: Lanes | Lane): Lanes {
194189
case SyncLane:
195190
return_highestLanePriority = SyncLanePriority;
196191
return SyncLane;
197-
case SyncBatchedLane:
198-
return_highestLanePriority = SyncBatchedLanePriority;
199-
return SyncBatchedLane;
200192
case InputContinuousHydrationLane:
201193
return_highestLanePriority = InputContinuousHydrationLanePriority;
202194
return InputContinuousHydrationLane;
@@ -265,7 +257,6 @@ export function lanePriorityToSchedulerPriority(
265257
): ReactPriorityLevel {
266258
switch (lanePriority) {
267259
case SyncLanePriority:
268-
case SyncBatchedLanePriority:
269260
return ImmediateSchedulerPriority;
270261
case InputContinuousHydrationLanePriority:
271262
case InputContinuousLanePriority:
@@ -547,8 +538,6 @@ export function findUpdateLane(lanePriority: LanePriority): Lane {
547538
break;
548539
case SyncLanePriority:
549540
return SyncLane;
550-
case SyncBatchedLanePriority:
551-
return SyncBatchedLane;
552541
case InputContinuousLanePriority:
553542
return InputContinuousLane;
554543
case DefaultLanePriority:
@@ -805,7 +794,6 @@ export function getBumpedLaneForHydration(
805794
let lane;
806795
switch (highestLanePriority) {
807796
case SyncLanePriority:
808-
case SyncBatchedLanePriority:
809797
lane = NoLane;
810798
break;
811799
case InputContinuousLanePriority:

0 commit comments

Comments
 (0)