Skip to content

Commit 82dcc93

Browse files
committed
Remove getPropertyInfo() and remaining property configs
1 parent bf858ef commit 82dcc93

9 files changed

+0
-167
lines changed

src/renderers/dom/ReactDOMServerBrowserEntry.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ var ReactDOMStringRenderer = require('ReactDOMStringRenderer');
1515
var ReactVersion = require('ReactVersion');
1616
var invariant = require('fbjs/lib/invariant');
1717

18-
require('ReactDOMInjection');
19-
2018
module.exports = {
2119
renderToString: ReactDOMStringRenderer.renderToString,
2220
renderToStaticMarkup: ReactDOMStringRenderer.renderToStaticMarkup,

src/renderers/dom/ReactDOMServerNodeEntry.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ var ReactDOMStringRenderer = require('ReactDOMStringRenderer');
1515
var ReactDOMNodeStreamRenderer = require('ReactDOMNodeStreamRenderer');
1616
var ReactVersion = require('ReactVersion');
1717

18-
require('ReactDOMInjection');
19-
2018
module.exports = {
2119
renderToString: ReactDOMStringRenderer.renderToString,
2220
renderToStaticMarkup: ReactDOMStringRenderer.renderToStaticMarkup,

src/renderers/dom/ReactDOMServerStackEntry.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
var ReactServerRendering = require('ReactServerRendering');
1515
var ReactVersion = require('ReactVersion');
1616

17-
require('ReactDOMInjection');
1817
require('ReactDOMStackInjection');
1918

2019
var ReactDOMServerStack = {

src/renderers/dom/ReactDOMStackEntry.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ if (__DEV__) {
2828
var warning = require('fbjs/lib/warning');
2929
}
3030

31-
require('ReactDOMInjection');
3231
require('ReactDOMClientInjection');
3332
require('ReactDOMStackInjection');
3433

src/renderers/dom/fiber/ReactDOMFiberEntry.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,6 @@ if (__DEV__) {
8383
}
8484

8585
require('ReactDOMClientInjection');
86-
require('ReactDOMInjection');
8786
ReactControlledComponent.injection.injectFiberControlledHostComponent(
8887
ReactDOMFiberComponent,
8988
);

src/renderers/dom/shared/DOMProperty.js

Lines changed: 0 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -51,29 +51,6 @@ function setDOMValueAttribute(node, value) {
5151
}
5252
}
5353

54-
var DOMPropertyInjection = {
55-
/**
56-
* Inject some specialized knowledge about the DOM. This takes a config object
57-
* with the following properties:
58-
*
59-
* Properties: object mapping DOM property name to one of the
60-
* DOMPropertyInjection constants or null. If your attribute isn't in here,
61-
* it won't get written to the DOM.
62-
*
63-
* DOMPropertyNames: similar to DOMAttributeNames but for DOM properties.
64-
* Property names not specified use the normalized name.
65-
*
66-
* @param {object} domPropertyConfig the config as described above.
67-
*/
68-
injectDOMPropertyConfig: function(domPropertyConfig) {
69-
var Properties = domPropertyConfig.Properties || {};
70-
71-
for (var propName in Properties) {
72-
DOMProperty.properties[propName] = {};
73-
}
74-
},
75-
};
76-
7754
/* eslint-disable max-len */
7855
var ATTRIBUTE_NAME_START_CHAR =
7956
':A-Z_a-z\\u00C0-\\u00D6\\u00D8-\\u00F6\\u00F8-\\u02FF\\u0370-\\u037D\\u037F-\\u1FFF\\u200C-\\u200D\\u2070-\\u218F\\u2C00-\\u2FEF\\u3001-\\uD7FF\\uF900-\\uFDCF\\uFDF0-\\uFFFD';
@@ -89,15 +66,6 @@ var attributeNames = {
8966
var CAMELIZE = /[\-\:]([a-z])/g;
9067
var capitalize = token => token[1].toUpperCase();
9168

92-
// TODO: remove this.
93-
var svgConfig = {
94-
Properties: {
95-
autoReverse: 0,
96-
externalResourcesRequired: 0,
97-
preserveAlpha: 0,
98-
},
99-
};
100-
10169
/**
10270
* This is a list of all SVG attributes that need special casing,
10371
* namespacing, or boolean value assignment.
@@ -198,10 +166,6 @@ var svgConfig = {
198166
].forEach(svgAttributeName => {
199167
var reactName = svgAttributeName.replace(CAMELIZE, capitalize);
200168
attributeNames[reactName] = svgAttributeName;
201-
202-
// TODO: remove this very soon.
203-
// We only need it until we stop branching on propertyInfo existence.
204-
svgConfig.Properties[reactName] = 0;
205169
});
206170

207171
/**
@@ -225,8 +189,6 @@ var DOMProperty = {
225189
ATTRIBUTE_NAME_CHAR: ATTRIBUTE_NAME_START_CHAR +
226190
'\\-.0-9\\u00B7\\u0300-\\u036F\\u203F-\\u2040',
227191

228-
properties: {},
229-
230192
/**
231193
* Checks whether a property name is a writeable attribute.
232194
* @method
@@ -293,12 +255,6 @@ var DOMProperty = {
293255
}
294256
},
295257

296-
getPropertyInfo(name) {
297-
return DOMProperty.properties.hasOwnProperty(name)
298-
? DOMProperty.properties[name]
299-
: null;
300-
},
301-
302258
getExpectedValueType(propName) {
303259
switch (propName) {
304260
// Numeric properties.
@@ -422,12 +378,6 @@ var DOMProperty = {
422378
isReservedProp(name) {
423379
return RESERVED_PROPS.hasOwnProperty(name);
424380
},
425-
426-
injection: DOMPropertyInjection,
427381
};
428382

429-
// TODO: remove this very soon.
430-
// We only need it until we stop branching on propertyInfo existence.
431-
DOMProperty.injection.injectDOMPropertyConfig(svgConfig);
432-
433383
module.exports = DOMProperty;

src/renderers/dom/shared/HTMLDOMPropertyConfig.js

Lines changed: 0 additions & 75 deletions
This file was deleted.

src/renderers/dom/shared/ReactDOMInjection.js

Lines changed: 0 additions & 19 deletions
This file was deleted.

src/renderers/dom/shared/SVGDOMPropertyConfig.js

Lines changed: 0 additions & 16 deletions
This file was deleted.

0 commit comments

Comments
 (0)