See https://phabricator.babeljs.io/T6779. Supercedes #3149.

function-name = babel-plugin-transform-es2015-function-name
shorthand-properties = babel-plugin-transform-es2015-shorthand-properties

Currently function-name transforms method definition (e.g. {x () {}}) to function expression. That seems wrong. shorthand-properties also performs that transformation and seems like the one that should be responsible for doing that and the one you should enable to opt-in to it. It seems like it should be possible to at the same time apply function-name to anonymous function expressions and not transform method definitions.

Example:

Input:

var z = {
  x () {},
  y: function () {},
};

shorthand-properties

babel.transform(input, {
  plugins: [
    "transform-es2015-shorthand-properties"
  ]
})

Output before and after PR:

var z = {
  x: function () {},

  y: function () {}
};

function-name

babel.transform(input, {
  plugins: [
    "transform-es2015-function-name"
  ]
})

Output before PR:

var z = {
  x: function x() {},

  y: function y() {}
};

Output after PR:

var z = {
  x() {},
  y: function y() {}
};

The current output of function-name can currently be achieved by using both transforms:

babel.transform(input, {
  plugins: [
    "transform-es2015-shorthand-properties",
    "transform-es2015-function-name"
  ]
})

Output before and after PR:

var z = {
  x: function x() {},

  y: function y() {}
};

The old implementation visited ObjectExpression and looped properties. I guess that was more efficient when transforming both ObjectMethod|ObjectProperty. I changed it to visit ObjectProperty directly.