Use Utility\Html class instead of Utility\String for decodeEntities() function

This looks great, let's see whether it passes.

Looks good to me, just please remove decodeEntities() from Drupal\Component\Utility\String.

Removing the deprecated function as suggested in #13.

Thanks !

+++ b/core/lib/Drupal/Component/Utility/String.php
@@ -32,22 +32,6 @@ public static function checkPlain($text) {
   /**
-   * Decodes all HTML entities including numerical ones to regular UTF-8 bytes.
-   *
-   * @param string $text
-   *   The text to decode entities in.
-   *
-   * @return string
-   *   The input $text, with all HTML entities decoded once.
-   *
-   * @deprecated in Drupal 8.x-dev, will be removed before Drupal 8.0.
-   *   Use \Drupal\Component\Utility\Html::decodeEntities() instead.
-   */
-  public static function decodeEntities($text) {
-    return Html::decodeEntities($text);
-  }

Let's remove this in a separate issue. Makes rollbacks and managing the rtbc queue easier. Also please attached this issue to the change record.

In fact we should should have a single issue to remove the string class that will go in after all the conversions are done.

I believe that was what was in patch in #11. Uploading here again. Setting to RTBC. Updating change record at https://www.drupal.org/node/2457593.

EDIT: Fixed link to change record.

Created follow up at #2458387: Remove Utility\String class.

Changing parent, so all these issues are visible in the php7 meta issue.

Unsurprisingly, this needs a re-roll after #2457887: Use Utility\SafeMarkup class instead of Utility\String for placeholder(), checkPlain(),format() functions.

Didn't get any merge conflicts in re-roll. Was that expected?

Helps if I update 8.0.x first. :-(

You may have forgotten to pull? :)

Let's see if this passes tests...

Checked, all converted and no "use" left

This should be critical since it's parent task is. Committed f3b8d07 and pushed to 8.0.x. Thanks!