导读:
Gtk-minimo中的几个BUG
转载时请注明出处和作者联系方式
作者联系方式:李先静
颜色不正常:背景颜色和字体颜色随机变化,一会儿蓝一会白,甚至有时全变白色了,什么都看不见。我怀疑是内存越界或者使用了没有初始化的变量。但是用valgrind运行时没有任何发现迹象。只好手工去调试了,先在draw_glyph中设置断点,发现gc的颜色值不正常,进行一步跟踪发现UpdateGC后颜色没有生效,最后发现是nsGCCache::ReuseGC出了问题。原来在ReuseGC里面对于GTK+/DirectFB的处理有误,导致entry->gc的内容没有正常更新。我看了一下,那些判断并不会提高性能,就直接调用了gdk_gc_set_values强制更新。之后颜色恢复正常。
void nsGCCache::ReuseGC(GCCacheEntry *entry, GdkGCValues *gcv, GdkGCValuesMask flags)
{
gdk_gc_set_values(entry->gc, gcv, flags);
}
状态栏隐藏之后不消失。我们发现在gtk_moz_embed_size_allocate函数中的参数完全正常,gtk_moz_embed的大小已经调整过了,但它就是没有重绘界面。mozilla的排版有点复杂,决定先花了两天时间看view/layout/dom目录中的代码再说。在有了大致的了解之后,我断定是viewmanager处理paint事件有问题,后来证实是判断view可见性有误。根据注释的提示,去掉对ViewObserver可见性的判断,gtk_moz_embed就刷新正常了。
static PRBool IsViewVisible(nsView *aView)
...{
for (nsIView *view = aView; view; view = view->GetParent()) ...{
// We don't check widget visibility here because in the future (with
// the better approach to this that's in attachment 160801 on bug
// 227361), callers of the equivalent to this function should be able
// to rely on being notified when the result of this function changes.
if (view->GetVisibility() == nsViewVisibility_kHide)
7 return PR_FALSE;
}
return PR_TRUE;
// Find out if the root view is visible by asking the view observer
// (this won't be needed anymore if we link view trees across chrome /
// content boundaries in DocumentViewerImpl::MakeWindow).
nsIViewObserver* vo = aView->GetViewManager()->GetViewObserver();
return vo && vo->IsVisible();
}
~~end~~
本文转自
http://blog.csdn.net/absurd/archive/2007/09/20/1793677.aspx
Gtk-minimo中的几个BUG
转载时请注明出处和作者联系方式
作者联系方式:李先静
颜色不正常:背景颜色和字体颜色随机变化,一会儿蓝一会白,甚至有时全变白色了,什么都看不见。我怀疑是内存越界或者使用了没有初始化的变量。但是用valgrind运行时没有任何发现迹象。只好手工去调试了,先在draw_glyph中设置断点,发现gc的颜色值不正常,进行一步跟踪发现UpdateGC后颜色没有生效,最后发现是nsGCCache::ReuseGC出了问题。原来在ReuseGC里面对于GTK+/DirectFB的处理有误,导致entry->gc的内容没有正常更新。我看了一下,那些判断并不会提高性能,就直接调用了gdk_gc_set_values强制更新。之后颜色恢复正常。
![](https://i-blog.csdnimg.cn/blog_migrate/6810355c2f78c12e91b7997a8e8c583a.gif)
void nsGCCache::ReuseGC(GCCacheEntry *entry, GdkGCValues *gcv, GdkGCValuesMask flags)
![](https://i-blog.csdnimg.cn/blog_migrate/6810355c2f78c12e91b7997a8e8c583a.gif)
{
![](https://i-blog.csdnimg.cn/blog_migrate/6810355c2f78c12e91b7997a8e8c583a.gif)
gdk_gc_set_values(entry->gc, gcv, flags);
![](https://i-blog.csdnimg.cn/blog_migrate/6810355c2f78c12e91b7997a8e8c583a.gif)
}
状态栏隐藏之后不消失。我们发现在gtk_moz_embed_size_allocate函数中的参数完全正常,gtk_moz_embed的大小已经调整过了,但它就是没有重绘界面。mozilla的排版有点复杂,决定先花了两天时间看view/layout/dom目录中的代码再说。在有了大致的了解之后,我断定是viewmanager处理paint事件有问题,后来证实是判断view可见性有误。根据注释的提示,去掉对ViewObserver可见性的判断,gtk_moz_embed就刷新正常了。
![](https://i-blog.csdnimg.cn/blog_migrate/6810355c2f78c12e91b7997a8e8c583a.gif)
static PRBool IsViewVisible(nsView *aView)
![](https://i-blog.csdnimg.cn/blog_migrate/a41954a27d6ad96fa2c2cf816e677448.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/1327ab569c1ae82736693a50b8e33378.gif)
...{
![](https://i-blog.csdnimg.cn/blog_migrate/37c8bf68cdc3cc81759c34160776bc53.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/7ff8d92cded7e0ce15e7ca1acc870052.gif)
for (nsIView *view = aView; view; view = view->GetParent()) ...{
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// We don't check widget visibility here because in the future (with
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// the better approach to this that's in attachment 160801 on bug
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// 227361), callers of the equivalent to this function should be able
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// to rely on being notified when the result of this function changes.
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
if (view->GetVisibility() == nsViewVisibility_kHide)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
7 return PR_FALSE;
![](https://i-blog.csdnimg.cn/blog_migrate/717446ca04a6125dc5b6b54e0fa14ab4.gif)
}
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
return PR_TRUE;
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// Find out if the root view is visible by asking the view observer
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// (this won't be needed anymore if we link view trees across chrome /
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
// content boundaries in DocumentViewerImpl::MakeWindow).
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
nsIViewObserver* vo = aView->GetViewManager()->GetViewObserver();
![](https://i-blog.csdnimg.cn/blog_migrate/6a9c071a08f1dae2d3e1c512000eef41.gif)
return vo && vo->IsVisible();
![](https://i-blog.csdnimg.cn/blog_migrate/0196c3df5ea9e936f21e9932cca91014.gif)
}
~~end~~
本文转自
http://blog.csdn.net/absurd/archive/2007/09/20/1793677.aspx