mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

safer casts, clearer if-else chain

Browse source
This commit is contained in:
Patrick Reynolds 2013-11-17 22:25:24 -06:00
parent c9e948dc58
commit 9f9aeac59f
1 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
brightray/browser/linux/inspectable_web_contents_view_linux.cc
View file

@ -31,7 +31,7 @@ static void dump_one(GtkWidget *wat, int indent) {
gtk_widget_get_allocation(wat, &alloc);
fprintf(stderr, "%*s[%p] %s @%d,%d %dx%d",
indent, "", wat,
g_type_name_from_instance(reinterpret_cast<GTypeInstance*>(wat)),
g_type_name_from_instance(static_cast<GTypeInstance*>(wat)),
alloc.x, alloc.y, alloc.width, alloc.height);
if (GTK_IS_WINDOW(wat)) {
fprintf(stderr, " - \"%s\"", gtk_window_get_title(GTK_WINDOW(wat)));
@ -106,7 +106,7 @@ void InspectableWebContentsViewLinux::ShowDevTools() {
DLOG(INFO) << base::StringPrintf(
"InspectableWebContentsViewLinux::ShowDevTools - " \
"parent=%s@%p window=%p dockside=\"%s\"",
g_type_name_from_instance(reinterpret_cast<GTypeInstance*>(parent)),
g_type_name_from_instance(static_cast<GTypeInstance*>(parent)),
parent,
devtools_window_,
dockside_.c_str());
@ -132,15 +132,14 @@ void InspectableWebContentsViewLinux::CloseDevTools() {
DLOG(INFO) << base::StringPrintf(
"InspectableWebContentsViewLinux::CloseDevTools - " \
"parent=%s@%p window=%p dockside=\"%s\"",
g_type_name_from_instance(reinterpret_cast<GTypeInstance*>(parent)),
g_type_name_from_instance(static_cast<GTypeInstance*>(parent)),
parent,
devtools_window_,
dockside_.c_str());
if (!parent)
if (!parent) {
return; // Not visible -> nothing to do
if (GTK_IS_PANED(parent)) {
} else if (GTK_IS_PANED(parent)) {
GtkWidget *browser = GetBrowserWindow();
GtkWidget *view = GetNativeView();