From: Neil Roberts <neil@openedhand.com>
Date: Thu, 13 Nov 2008 11:37:35 +0000 (+0000)
Subject: 	* clutter/clutter-actor.c (clutter_actor_get_paint_visibility):
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d510a4b0cd500197d42968239d1f3dc896db889a;p=profile%2Fivi%2Fclutter.git

	* clutter/clutter-actor.c (clutter_actor_get_paint_visibility):
	Fix logic so that it won't return TRUE for a hidden stage.
---

diff --git a/ChangeLog b/ChangeLog
index 6108028..c806c65 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2008-11-13  Neil Roberts  <neil@linux.intel.com>
+
+	* clutter/clutter-actor.c (clutter_actor_get_paint_visibility):
+	Fix logic so that it won't return TRUE for a hidden stage.
+
 2008-11-13  Emmanuele Bassi  <ebassi@linux.intel.com>
 
 	* clutter/clutter-behaviour-ellipse.c:
diff --git a/clutter/clutter-actor.c b/clutter/clutter-actor.c
index ef09d0a..ed6d59b 100644
--- a/clutter/clutter-actor.c
+++ b/clutter/clutter-actor.c
@@ -5698,13 +5698,17 @@ clutter_actor_get_paint_visibility (ClutterActor *actor)
 {
   g_return_val_if_fail (CLUTTER_IS_ACTOR (actor), FALSE);
 
-  while (actor 
-         && !(CLUTTER_PRIVATE_FLAGS (actor) & CLUTTER_ACTOR_IS_TOPLEVEL)
-         && CLUTTER_ACTOR_IS_VISIBLE (actor))
-    actor = clutter_actor_get_parent (actor);
+  do
+    {
+      if (!CLUTTER_ACTOR_IS_VISIBLE (actor))
+        return FALSE;
 
-  return ((CLUTTER_PRIVATE_FLAGS (actor) & CLUTTER_ACTOR_IS_TOPLEVEL)
-          ? TRUE : FALSE);
+      if (CLUTTER_PRIVATE_FLAGS (actor) & CLUTTER_ACTOR_IS_TOPLEVEL)
+        return TRUE;
+    }
+  while ((actor = clutter_actor_get_parent (actor)) != NULL);
+
+  return FALSE;
 }
 
 /**