From dbfffdcd12b51dda02df1413a2edbf2425d58225 Mon Sep 17 00:00:00 2001
From: Chris Double <chris.double@double.co.nz>
Date: Tue, 23 Apr 2013 08:10:31 +1200
Subject: [PATCH] Bug 845729 - Blocklist some Samsung ICS qcom devices for
 stagefright decoding - r=bjacob

---
 widget/android/GfxInfo.cpp | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/widget/android/GfxInfo.cpp b/widget/android/GfxInfo.cpp
index 25f1afbca4b8..259b4dd359ee 100644
--- a/widget/android/GfxInfo.cpp
+++ b/widget/android/GfxInfo.cpp
@@ -426,7 +426,10 @@ GfxInfo::GetFeatureStatusImpl(int32_t aFeature,
       else if (CompareVersions(mOSVersion.get(), "4.1.0") < 0)
       {
         // Whitelist:
-        //   All Samsung ICS devices
+        //   All Samsung ICS devices, except for:
+        //     Samsing SGH-I717 (Bug 845729)
+        //     Samsing SGH-I727 (Bug 845729)
+        //     Samsing SGH-T989 (Bug 845729)
         //   All Galaxy nexus ICS devices
         //   Sony Xperia Ion (LT28) ICS devices
         bool isWhitelisted =
@@ -434,6 +437,13 @@ GfxInfo::GetFeatureStatusImpl(int32_t aFeature,
           cManufacturer.Equals("samsung", nsCaseInsensitiveCStringComparator()) ||
           cModel.Equals("galaxy nexus", nsCaseInsensitiveCStringComparator()); // some Galaxy Nexus have manufacturer=amazon
 
+        if (cModel.Equals("SGH-I717", nsCaseInsensitiveCStringComparator()) ||
+            cModel.Equals("SGH-I727", nsCaseInsensitiveCStringComparator()) ||
+            cModel.Equals("SGH-T989", nsCaseInsensitiveCStringComparator()))
+        {
+          isWhitelisted = false;
+        }
+
         if (!isWhitelisted) {
           *aStatus = nsIGfxInfo::FEATURE_BLOCKED_DEVICE;
           return NS_OK;