Revert "Set site logo url in ResourceLoaderSiteModule instead of inline styles"

This caused bug 71334. The logo should be split out
into its own ResourceLoader module rather than
being in the site module.

This reverts commit efe0169c591d4987872143a3760228a694af33c4.

Bug: 71334
Change-Id: I0234e78f93c2eb6574bae4ad48941498709ae528

diff --git a/RELEASE-NOTES-1.25 b/RELEASE-NOTES-1.25
index c8bea32..c935ffe 100644 (file)
--- a/RELEASE-NOTES-1.25
+++ b/RELEASE-NOTES-1.25
@@ -27,8 +27,6 @@ changes to languages because of Bugzilla reports.
   removed. See https://www.mediawiki.org/wiki/Manual:Skin_autodiscovery for
   migration guide for creators and users of custom skins that relied on it. 
 * Javascript variable 'wgFileCanRotate' now only available on Special:Upload.
-* (bug 56257) Set site logo url in ResourceLoaderSiteModule instead of inline
-  styles in the HTML output.
 
 == Compatibility ==
 

diff --git a/includes/resourceloader/ResourceLoaderSiteModule.php b/includes/resourceloader/ResourceLoaderSiteModule.php
index 6de3d73..1d9721a 100644 (file)
--- a/includes/resourceloader/ResourceLoaderSiteModule.php
+++ b/includes/resourceloader/ResourceLoaderSiteModule.php
@@ -53,49 +53,6 @@ class ResourceLoaderSiteModule extends ResourceLoaderWikiModule {
 
        /* Methods */
 
-       /**
-        * @param $context ResourceLoaderContext
-        * @return array
-        */
-       public function getStyles( ResourceLoaderContext $context ) {
-               global $wgLogo;
-
-               $styles = parent::getStyles( $context );
-               $styles['all'][] = '.mw-wiki-logo { background-image: ' .
-                       CSSMin::buildUrlValue( $wgLogo ) .
-                       '; }';
-
-               return $styles;
-       }
-
-       /**
-        * @param $context ResourceLoaderContext
-        * @return boolean
-        */
-       public function isKnownEmpty( ResourceLoaderContext $context ) {
-               // Regardless of whether the wiki page(s) exist, we always
-               // provide mw-wiki-logo styles.
-               return false;
-       }
-
-       /**
-        * @param $context ResourceLoaderContext
-        * @return int|mixed
-        */
-       public function getModifiedTime( ResourceLoaderContext $context ) {
-               $parentMTime = parent::getModifiedTime( $context );
-               return max( $parentMTime, $this->getHashMtime( $context ) );
-       }
-
-       /**
-        * @param $context ResourceLoaderContext
-        * @return string: Hash
-        */
-       public function getModifiedHash( ResourceLoaderContext $context ) {
-               global $wgLogo;
-               return md5( parent::getModifiedHash( $context ) . $wgLogo );
-       }
-
        /**
         * Gets group name
         *