Fix a bug where we tried to get the position of the tabstrip

chrome/browser/gtk/tabs/tab_strip_gtk.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/gtk/tabs/tab_strip_gtk.h"
 
 #include <algorithm>
 
 #include "app/gfx/canvas_paint.h"
 #include "app/l10n_util.h"
@@ skipping 832 matching lines @@
   DCHECK(index >= 0 && index < GetTabCount());
   return tab_data_.at(index).ideal_bounds;
 }
 
 gfx::Point TabStripGtk::GetTabStripOriginForWidget(GtkWidget* target) {
   int x, y;
   if (!gtk_widget_translate_coordinates(widget(), target,
       -widget()->allocation.x, 0, &x, &y)) {
     // If the tab strip isn't showing, give the coordinates relative to the
     // toplevel instead.
-    gtk_widget_translate_coordinates(
-        gtk_widget_get_toplevel(widget()), target, 0, 0, &x, &y);
+    if (!gtk_widget_translate_coordinates(
+        gtk_widget_get_toplevel(widget()), target, 0, 0, &x, &y)) {
+      NOTREACHED();
+    }
   }
   if (GTK_WIDGET_NO_WINDOW(target)) {
     x += target->allocation.x;
     y += target->allocation.y;
   }
   return gfx::Point(x, y);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // TabStripGtk, TabStripModelObserver implementation:
@@ skipping 1068 matching lines @@
   message.set_type(TabOverviewTypes::Message::WM_SWITCH_TO_OVERVIEW_MODE);
   GtkWidget* browser_widget = GTK_WIDGET(
       static_cast<BrowserWindowGtk*>(browser->window())->GetNativeHandle());
   message.set_param(0, x11_util::GetX11WindowFromGtkWidget(browser_widget));
   TabOverviewTypes::instance()->SendMessage(message);
 
   UserMetrics::RecordAction(L"TabOverview_PressedTabOverviewButton",
                             tabstrip->model_->profile());
 }
 #endif