[Qt] Remove an unnecessary masking from swapBgrToRgb()
authorrgabor@webkit.org <rgabor@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 3 Dec 2012 10:15:16 +0000 (10:15 +0000)
committerrgabor@webkit.org <rgabor@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 3 Dec 2012 10:15:16 +0000 (10:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=103630

Reviewed by Zoltan Herczeg.

Get rid of a masking command in swapBgrToRgb() to speed up a little bit.

* platform/graphics/qt/GraphicsContext3DQt.cpp:
(WebCore::swapBgrToRgb):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@136375 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/qt/GraphicsContext3DQt.cpp

index c0b1939..2867e26 100644 (file)
@@ -1,3 +1,15 @@
+2012-12-03  Gabor Rapcsanyi  <rgabor@webkit.org>
+
+        [Qt] Remove an unnecessary masking from swapBgrToRgb()
+        https://bugs.webkit.org/show_bug.cgi?id=103630
+
+        Reviewed by Zoltan Herczeg.
+
+        Get rid of a masking command in swapBgrToRgb() to speed up a little bit.
+
+        * platform/graphics/qt/GraphicsContext3DQt.cpp:
+        (WebCore::swapBgrToRgb):
+
 2012-12-03  Mikhail Pozdnyakov  <mikhail.pozdnyakov@intel.com>
 
         CSS Device Adaptation: window.innerWidth returns wrong value if CSS viewport descriptors are applied
index a50ceb9..540c501 100644 (file)
@@ -204,7 +204,7 @@ GraphicsContext3DPrivate::~GraphicsContext3DPrivate()
 
 static inline quint32 swapBgrToRgb(quint32 pixel)
 {
-    return ((pixel << 16) & 0xff0000) | ((pixel >> 16) & 0xff) | (pixel & 0xff00ff00);
+    return (((pixel << 16) | (pixel >> 16)) & 0x00ff00ff) | (pixel & 0xff00ff00);
 }
 
 #if USE(ACCELERATED_COMPOSITING)