Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Unified Diff: chrome/browser/ui/gtk/tabs/tab_strip_gtk.cc

Issue 8800029: GTK: Add TRACE_EVENTs around gtk ui events. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: fix nit Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/gtk/tabs/tab_strip_gtk.cc
diff --git a/chrome/browser/ui/gtk/tabs/tab_strip_gtk.cc b/chrome/browser/ui/gtk/tabs/tab_strip_gtk.cc
index 79110e46dd225d11fef8bd60a8d01a8670f7102a..7d13eede05906018f684c4f466a8783a4f9a615e 100644
--- a/chrome/browser/ui/gtk/tabs/tab_strip_gtk.cc
+++ b/chrome/browser/ui/gtk/tabs/tab_strip_gtk.cc
@@ -7,6 +7,7 @@
#include <algorithm>
#include "base/bind.h"
+#include "base/debug/trace_event.h"
#include "base/i18n/rtl.h"
#include "base/string_util.h"
#include "base/utf_string_conversions.h"
@@ -961,6 +962,8 @@ GtkWidget* TabStripGtk::GetWidgetForViewID(ViewID view_id) {
void TabStripGtk::TabInsertedAt(TabContentsWrapper* contents,
int index,
bool foreground) {
+ TRACE_EVENT0("ui::gtk", "TabStripGtk::TabInsertedAt");
+
DCHECK(contents);
DCHECK(index == TabStripModel::kNoTab || model_->ContainsIndex(index));
@@ -1034,6 +1037,7 @@ void TabStripGtk::ActiveTabChanged(TabContentsWrapper* old_contents,
TabContentsWrapper* new_contents,
int index,
bool user_gesture) {
+ TRACE_EVENT0("ui::gtk", "TabStripGtk::ActiveTabChanged");
ReStack();
}
@@ -1604,6 +1608,8 @@ bool TabStripGtk::IsCursorInTabStripZone() const {
}
void TabStripGtk::ReStack() {
+ TRACE_EVENT0("ui::gtk", "TabStripGtk::ReStack");
+
if (!gtk_widget_get_realized(tabstrip_.get())) {
// If the window isn't realized yet, we can't stack them yet. It will be
// done by the OnMap signal handler.
@@ -1800,6 +1806,8 @@ TabStripGtk::DropInfo::~DropInfo() {
gboolean TabStripGtk::DropInfo::OnExposeEvent(GtkWidget* widget,
GdkEventExpose* event) {
+ TRACE_EVENT0("ui::gtk", "TabStripGtk::DropInfo::OnExposeEvent");
+
if (ui::IsScreenComposited()) {
SetContainerTransparency();
} else {
@@ -1984,6 +1992,8 @@ void TabStripGtk::OnMap(GtkWidget* widget) {
}
gboolean TabStripGtk::OnExpose(GtkWidget* widget, GdkEventExpose* event) {
+ TRACE_EVENT0("ui::gtk", "TabStripGtk::OnExpose");
+
if (gdk_region_empty(event->region))
return TRUE;
@@ -2063,6 +2073,8 @@ gboolean TabStripGtk::OnExpose(GtkWidget* widget, GdkEventExpose* event) {
}
void TabStripGtk::OnSizeAllocate(GtkWidget* widget, GtkAllocation* allocation) {
+ TRACE_EVENT0("ui::gtk", "TabStripGtk::OnSizeAllocate");
+
gfx::Rect bounds = gfx::Rect(allocation->x, allocation->y,
allocation->width, allocation->height);

Powered by Google App Engine
This is Rietveld 408576698