Removes unnecessary calls to mb_strtolower

Author: jeremykendall

library/Pdp/Parser.php

@@ -128,7 +128,6 @@ public function getPublicSuffix($host)
             return null;
         }
 
-        $host = mb_strtolower($host, 'UTF-8');
         $parts = array_reverse(explode('.', $host));
         $publicSuffix = array();
         $publicSuffixList = $this->publicSuffixList;
@@ -217,7 +216,6 @@ public function getRegisterableDomain($host)
      */
     public function getSubdomain($host)
     {
-        $host = mb_strtolower($host, 'UTF-8');
         $registerableDomain = $this->getRegisterableDomain($host);
 
         if ($registerableDomain === null || $host == $registerableDomain) {
@@ -238,7 +236,7 @@ public function getSubdomain($host)
      *
      * @return array
      */
-    public function mbParseUrl($url)
+    protected function mbParseUrl($url)
     {
         $enc_url = preg_replace_callback(
             '%[^:/@?&=#]+%usD',

tests/library/Pdp/ParserTest.php

@@ -23,6 +23,7 @@ protected function tearDown()
 
     /**
      * @covers Pdp\Parser::parseUrl()
+     * @covers Pdp\Parser::mbParseUrl()
      */
     public function testParseBadUrlThrowsInvalidArgumentException()
     {
@@ -108,7 +109,7 @@ public function parseDataProvider()
         // url, public suffix, registerable domain, subdomain, host part
         return array(
             array('http://www.waxaudio.com.au/audio/albums/the_mashening', 'com.au', 'waxaudio.com.au', 'www', 'www.waxaudio.com.au'),
-            array('example.com', 'com', 'example.com', null, 'example.com'),
+            array('example.COM', 'com', 'example.com', null, 'example.com'),
             array('giant.yyyy', 'yyyy', 'giant.yyyy', null, 'giant.yyyy'),
             array('cea-law.co.il', 'co.il', 'cea-law.co.il', null, 'cea-law.co.il'),
             array('http://edition.cnn.com/WORLD/', 'com', 'cnn.com', 'edition', 'edition.cnn.com'),