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

Side by Side Diff: content/browser/renderer_host/render_widget_host_view_gtk.cc

Issue 14335017: content: Use base::MessageLoop. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 7 months 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/render_widget_host_view_gtk.h" 5 #include "content/browser/renderer_host/render_widget_host_view_gtk.h"
6 6
7 #include <cairo/cairo.h> 7 #include <cairo/cairo.h>
8 #include <gdk/gdk.h> 8 #include <gdk/gdk.h>
9 #include <gdk/gdkkeysyms.h> 9 #include <gdk/gdkkeysyms.h>
10 #include <gdk/gdkx.h> 10 #include <gdk/gdkx.h>
(...skipping 910 matching lines...) Expand 10 before | Expand all | Expand 10 after
921 // this code is currently executing within the context of a gtk signal 921 // this code is currently executing within the context of a gtk signal
922 // handler. Note that |view_| is still alive after this call. It will be 922 // handler. Note that |view_| is still alive after this call. It will be
923 // deallocated in the destructor. 923 // deallocated in the destructor.
924 // See http://crbug.com/11847 for details. 924 // See http://crbug.com/11847 for details.
925 gtk_widget_destroy(view_.get()); 925 gtk_widget_destroy(view_.get());
926 } 926 }
927 927
928 // The RenderWidgetHost's destruction led here, so don't call it. 928 // The RenderWidgetHost's destruction led here, so don't call it.
929 host_ = NULL; 929 host_ = NULL;
930 930
931 MessageLoop::current()->DeleteSoon(FROM_HERE, this); 931 base::MessageLoop::current()->DeleteSoon(FROM_HERE, this);
932 } 932 }
933 933
934 void RenderWidgetHostViewGtk::SetTooltipText(const string16& tooltip_text) { 934 void RenderWidgetHostViewGtk::SetTooltipText(const string16& tooltip_text) {
935 // Maximum number of characters we allow in a tooltip. 935 // Maximum number of characters we allow in a tooltip.
936 const int kMaxTooltipLength = 8 << 10; 936 const int kMaxTooltipLength = 8 << 10;
937 // Clamp the tooltip length to kMaxTooltipLength so that we don't 937 // Clamp the tooltip length to kMaxTooltipLength so that we don't
938 // accidentally DOS the user with a mega tooltip (since GTK doesn't do 938 // accidentally DOS the user with a mega tooltip (since GTK doesn't do
939 // this itself). 939 // this itself).
940 // I filed https://bugzilla.gnome.org/show_bug.cgi?id=604641 upstream. 940 // I filed https://bugzilla.gnome.org/show_bug.cgi?id=604641 upstream.
941 const string16 clamped_tooltip = 941 const string16 clamped_tooltip =
(...skipping 633 matching lines...) Expand 10 before | Expand all | Expand 10 after
1575 gfx::PluginWindowHandle id) { 1575 gfx::PluginWindowHandle id) {
1576 plugin_container_manager_.CreatePluginContainer(id); 1576 plugin_container_manager_.CreatePluginContainer(id);
1577 } 1577 }
1578 1578
1579 void RenderWidgetHostViewGtk::OnDestroyPluginContainer( 1579 void RenderWidgetHostViewGtk::OnDestroyPluginContainer(
1580 gfx::PluginWindowHandle id) { 1580 gfx::PluginWindowHandle id) {
1581 plugin_container_manager_.DestroyPluginContainer(id); 1581 plugin_container_manager_.DestroyPluginContainer(id);
1582 } 1582 }
1583 1583
1584 } // namespace content 1584 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698