Move more keys to static argument instead of spread

Summary: This pattern is deprecated. Key has to be spread as a static argument instead of through props. These are some of our highest firing callsites.

Reviewed By: mrkev, gaearon

Differential Revision: D19873502

fbshipit-source-id: 9dd43ae625edffbb3c5dbe29237f7b769726fa04

src/component/contents/DraftEditorBlock.react.js

@@ -213,7 +213,6 @@ class DraftEditorBlock extends React.Component<Props> {
           contentState: this.props.contentState,
           decoratedText,
           dir: dir,
-          key: decoratorOffsetKey,
           start,
           end,
           blockKey,
@@ -222,7 +221,10 @@ class DraftEditorBlock extends React.Component<Props> {
         };
 
         return (
-          <DecoratorComponent {...decoratorProps} {...commonProps}>
+          <DecoratorComponent
+            {...decoratorProps}
+            {...commonProps}
+            key={decoratorOffsetKey}>
             {leaves}
           </DecoratorComponent>
         );

src/model/decorators/DraftDecorator.js

@@ -66,7 +66,6 @@ export type DraftDecoratorComponentProps = {
   // undefined. That's why `getEntityKeyAt()` is typed to return `?string`.
   // See https://github.com/facebook/draft-js/blob/2da3dcb1c4c106d1b2a0f07b3d0275b8d724e777/src/model/immutable/BlockNode.js#L51
   entityKey: ?string,
-  key: React.Key,
   offsetKey: string,
   start: number,
   ...