perf/gtkwidgetprofiler.c: Use accessor functions to access GtkWidget

[~andy/gtk] / perf / gtkwidgetprofiler.c

diff --git a/perf/gtkwidgetprofiler.c b/perf/gtkwidgetprofiler.c
index 01ae8420ed832ed8faa436ca2fd85f7cecab8898..c5b2cddbd633680e06267aa1ad3d52f2bfc97af7 100644 (file)
--- a/perf/gtkwidgetprofiler.c
+++ b/perf/gtkwidgetprofiler.c
@@ -168,7 +168,7 @@ create_widget_via_emission (GtkWidgetProfiler *profiler)
   if (!widget)
     g_error ("The profiler emitted the \"create-widget\" signal but the signal handler returned no widget!");
 
-  if (GTK_WIDGET_VISIBLE (widget) || GTK_WIDGET_MAPPED (widget))
+  if (gtk_widget_get_visible (widget) || gtk_widget_get_mapped (widget))
     g_error ("The handler for \"create-widget\" must return an unmapped and unshown widget");
 
   return widget;
@@ -205,12 +205,12 @@ toplevel_idle_after_expose_cb (gpointer data)
   profiler = GTK_WIDGET_PROFILER (data);
   priv = profiler->priv;
 
-  gdk_property_change (priv->toplevel->window,
+  gdk_property_change (gtk_widget_get_window (priv->toplevel),
                       priv->profiler_atom,
                       gdk_atom_intern ("STRING", FALSE),
                       8,
                       GDK_PROP_MODE_REPLACE,
-                      "hello",
+                      (guchar *) "hello",
                       strlen ("hello"));
 
   return FALSE;
@@ -223,7 +223,7 @@ toplevel_expose_event_cb (GtkWidget *widget, GdkEventExpose *event, gpointer dat
 
   profiler = GTK_WIDGET_PROFILER (data);
 
-  g_idle_add_full (G_PRIORITY_HIGH, toplevel_idle_after_expose_cb, profiler, NULL);
+  gdk_threads_add_idle_full (G_PRIORITY_HIGH, toplevel_idle_after_expose_cb, profiler, NULL);
   return FALSE;
 }
 
@@ -250,7 +250,7 @@ ensure_and_get_toplevel (GtkWidget *widget)
        GtkWidget *window;
 
        toplevel = gtk_widget_get_toplevel (widget);
-       if (GTK_WIDGET_TOPLEVEL (toplevel))
+       if (gtk_widget_is_toplevel (toplevel))
                return toplevel;
 
        g_assert (toplevel == widget); /* we don't want extraneous ancestors */
@@ -274,13 +274,11 @@ get_instrumented_toplevel (GtkWidgetProfiler *profiler,
 }
 
 static void
-profile_map_expose (GtkWidgetProfiler *profiler)
+map_widget (GtkWidgetProfiler *profiler)
 {
   GtkWidgetProfilerPrivate *priv;
-  gdouble elapsed;
 
   priv = profiler->priv;
-
   g_assert (priv->state == STATE_INSTRUMENTED_NOT_MAPPED);
 
   /* Time map.
@@ -289,12 +287,24 @@ profile_map_expose (GtkWidgetProfiler *profiler)
    * to happen.  Should we rename GTK_WIDGET_PROFILER_REPORT_MAP report to something else?
    */
 
-  g_timer_reset (priv->timer);
-
   gtk_widget_show_all (priv->toplevel);
   priv->state = STATE_INSTRUMENTED_MAPPED;
+}
 
+static void
+profile_map_expose (GtkWidgetProfiler *profiler)
+{
+  GtkWidgetProfilerPrivate *priv;
+  gdouble elapsed;
+
+  priv = profiler->priv;
+
+  g_assert (priv->state == STATE_INSTRUMENTED_NOT_MAPPED);
+
+  g_timer_reset (priv->timer);
+  map_widget (profiler);
   elapsed = g_timer_elapsed (priv->timer, NULL);
+
   report (profiler, GTK_WIDGET_PROFILER_REPORT_MAP, elapsed);
 
   /* Time expose; this gets recorded in toplevel_property_notify_event_cb() */
@@ -321,29 +331,46 @@ profile_destroy (GtkWidgetProfiler *profiler)
 }
 
 static void
-profile_boot (GtkWidgetProfiler *profiler)
+create_widget (GtkWidgetProfiler *profiler)
 {
   GtkWidgetProfilerPrivate *priv;
-  gdouble elapsed;
 
   priv = profiler->priv;
 
   g_assert (priv->state == STATE_NOT_CREATED);
 
-  /* Time creation */
-
-  g_timer_reset (priv->timer);
-
   priv->profiled_widget = create_widget_via_emission (profiler);
   priv->toplevel = get_instrumented_toplevel (profiler, priv->profiled_widget);
 
   priv->state = STATE_INSTRUMENTED_NOT_MAPPED;
+}
 
-  /* Here we include the time to anchor the widget to a toplevel, if
-   * the toplevel was missing --- hopefully not a too-long extra time.
-   */
+/* The "boot time" of a widget is the time needed to
+ *
+ *   1. Create the widget
+ *   2. Map it
+ *   3. Expose it
+ *   4. Destroy it.
+ *
+ * This runs a lot of interesting code:  instantiation, size requisition and
+ * allocation, realization, mapping, exposing, destruction.
+ */
+static void
+profile_boot (GtkWidgetProfiler *profiler)
+{
+  GtkWidgetProfilerPrivate *priv;
+  gdouble elapsed;
 
+  priv = profiler->priv;
+
+  g_assert (priv->state == STATE_NOT_CREATED);
+
+  /* Time creation */
+
+  g_timer_reset (priv->timer);
+  create_widget (profiler);
   elapsed = g_timer_elapsed (priv->timer, NULL);
+
   report (profiler, GTK_WIDGET_PROFILER_REPORT_CREATE, elapsed);
 
   /* Start timing map/expose */
@@ -355,6 +382,50 @@ profile_boot (GtkWidgetProfiler *profiler)
   profile_destroy (profiler);
 }
 
+/* To measure expose time, we trigger a full expose on the toplevel window.  We
+ * do the same as xrefresh(1), i.e. we map and unmap a window to make the other
+ * one expose.
+ */
+static void
+profile_expose (GtkWidgetProfiler *profiler)
+{
+  GtkWidgetProfilerPrivate *priv;
+  GtkAllocation allocation;
+  GdkWindow *window;
+  GdkWindowAttr attr;
+  int attr_mask;
+
+  priv = profiler->priv;
+
+  g_assert (priv->state == STATE_INSTRUMENTED_MAPPED);
+
+  /* Time creation */
+
+  gtk_widget_get_allocation (priv->toplevel, &allocation);
+
+  attr.x = 0;
+  attr.y = 0;
+  attr.width = allocation.width;
+  attr.height = allocation.width;
+  attr.wclass = GDK_INPUT_OUTPUT;
+  attr.window_type = GDK_WINDOW_CHILD;
+
+  attr_mask = GDK_WA_X | GDK_WA_Y;
+
+  window = gdk_window_new (gtk_widget_get_window (priv->toplevel),
+                           &attr, attr_mask);
+  gdk_window_set_back_pixmap (window, NULL, TRUE); /* avoid flicker */
+
+  gdk_window_show (window);
+  gdk_window_hide (window);
+  gdk_window_destroy (window);
+
+  /* Time expose; this gets recorded in toplevel_property_notify_event_cb() */
+
+  g_timer_reset (priv->timer);
+  gtk_main ();
+}
+
 void
 gtk_widget_profiler_profile_boot (GtkWidgetProfiler *profiler)
 {
@@ -375,3 +446,29 @@ gtk_widget_profiler_profile_boot (GtkWidgetProfiler *profiler)
 
   priv->profiling = FALSE;
 }
+
+void
+gtk_widget_profiler_profile_expose (GtkWidgetProfiler *profiler)
+{
+  GtkWidgetProfilerPrivate *priv;
+  int i, n;
+
+  g_return_if_fail (GTK_IS_WIDGET_PROFILER (profiler));
+
+  priv = profiler->priv;
+  g_return_if_fail (!priv->profiling);
+
+  reset_state (profiler);
+  priv->profiling = TRUE;
+
+  create_widget (profiler);
+  map_widget (profiler);
+
+  n = priv->n_iterations;
+  for (i = 0; i < n; i++)
+    profile_expose (profiler);
+
+  priv->profiling = FALSE;
+
+  reset_state (profiler);
+}