Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 58 users
Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Aug 2009
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug

Restricted
  • Only users with Commit permission may comment.



Sign in to add a comment
Head in place of close button
Reported by tshn...@gmail.com, Aug 4 2009 Back to list
Chrome Version       : 3.0.197.0 (22354)
OS + version : Debian AMD64(Using 32-bit libs, inapplicable)
CPU architecture (32-bit / 64-bit): x86_64
window manager : GNOME(gtk)
URLs (if applicable) : All
Behavior in Firefox 3.x (if applicable): X button
Behavior in Chrome for Windows (optional): X button

What steps will reproduce the problem?
1. Look at what the X button is supposed to be
2.
3.

What is the expected result?
an X button

What happens instead?
Some guys head

Please provide any additional information below. Attach a screenshot
and backtrace if possible.
I think it's rather self-explanatory
http://img262.imageshack.us/img262/7919/screenshot9.png

I figure it's best to tell the developers who probably don't test **every** 
version.

I suspect it was either a prank or a mistake.

 
Labels: -Area-Misc Area-BrowserUI Pri-1
Status: Assigned
this is tragic
Comment 2 by evan@chromium.org, Aug 4 2009
Aw, I kinda like him peeking over like that...
Still there in 22373...
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22391 

------------------------------------------------------------------------
r22391 | evan@chromium.org | 2009-08-04 10:43:10 -0700 (Tue, 04 Aug 2009) | 5 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/app_resources.grd?r1=22391&r2=22390
   M http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_close.png?r1=22391&r2=22390
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_close_glen.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_close_h.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_close_mask.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_close_p.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_maximize.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_maximize_h.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_maximize_mask.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_maximize_p.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_minimize.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_minimize_h.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_minimize_mask.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_minimize_p.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_restore.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_restore_h.png
   A http://src.chromium.org/viewvc/chrome/trunk/src/app/resources/linux_restore_p.png
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_titlebar.cc?r1=22391&r2=22390

linux: use linux-specific window control graphics from glen

BUG= 15952 , 18385 

Review URL: http://codereview.chromium.org/159859
------------------------------------------------------------------------

Comment 5 by evan@chromium.org, Aug 4 2009
Status: Fixed
 Issue 18397  has been merged into this issue.
Comment 7 Deleted
 Issue 18421  has been merged into this issue.
just wondering what else can get in thru code reviews. you guys... ah, better not to
say anything. such a nice application and now this. there should be more buzz about
it to make less people using chromium, such a insecure bullshit, where everybody can
commit whatever. today it's creepy face, tomorrow it will be malicious code, nice work.
Comment 10 by Deleted ...@, Aug 5 2009
Funniest bug ever.

But I'm a bit newbie, what's the way to fix it without using GTK+ horrible theme? I 
want a X, no Glen's head.

...sorry, Glen.
@missingno770: Just right click and check [ ] Use system title bar in browser. 
Comment 12 Deleted
Oh come on, don't be so abusive. It's an alpha version, it's not even out yet.
Possibly an easter egg gone wrong ;)
Comment 14 Deleted
This is awesome, I love you google.
@dusan.maliarik, calm down.  There's nothing malicious about what just happened here, it's simply just 
motivation for someone specific to get something done.  It's fixed now, so clearly this passive-agressive 
behaiour got the message to the right places. ;)
Comment 17 by Deleted ...@, Aug 5 2009
Good to know we'll be able to theme almost all aspects of Chromium once its finished :)
Comment 18 by Deleted ...@, Aug 5 2009
@denis.zawada Thank you. Is that easy. I never realized how idiot I could be =P 
Comment 19 by Deleted ...@, Aug 5 2009
hold up - that is a feature, not a bug. official mascot of chrome - some guys head.
Comment 20 by Deleted ...@, Aug 5 2009
I rather want an attractive female head than this melon head...
Comment 21 by Deleted ...@, Aug 5 2009
Thanks for a good laugh
Comment 22 Deleted
Comment 23 by dhw@chromium.org, Aug 5 2009
 Issue 18506  has been merged into this issue.
Comment 24 by dhw@chromium.org, Aug 5 2009
 Issue 18469  has been merged into this issue.
Comment 25 by cer...@gmail.com, Aug 5 2009
I never realized a browser could be creepy
So who is this Glen person??? nice, although it should of been on the spy on the 
incognito mode, and on the incognito notice, it should of said watch out Glen is 
watching or something lol!

So was this done in order to emphasize the unindented Restore Down button when pressed?
This was done as a passive-aggressive motivation to get someone to create the linux 
close/min/max/restore buttons, I believe.  And the message got through, and the work 
was done. ;)
@drewshays nice! thanks for the report :)
Comment 29 by evan@chromium.org, Aug 5 2009
Comment 30 by cer...@gmail.com, Aug 5 2009
P.S. Not to say that Glen is creepy, just that the last thing you would expect in a 
browser is someone watching you :)
@ ceryn: Actually, the FIRST thing I expect in a browser is that someone is watching
me....
Comment 32 by Deleted ...@, Aug 5 2009
TOASTY!
Comment 33 Deleted
Comment 34 by Deleted ...@, Aug 5 2009
sweet
Comment 35 Deleted
Comment 36 Deleted
Comment 37 Deleted
Comment 38 Deleted
Comment 39 by Deleted ...@, Aug 5 2009
Comment 40 by Deleted ...@, Aug 5 2009
@dusan.maliarik: you can probably learn to be more appreciative

go google!! 
"don't be evil" (tm)
Comment 41 Deleted
Comment 42 Deleted
Comment 43 Deleted
Comment 44 Deleted
Comment 45 Deleted
Good for us it's just "some guys head"...:)
Comment 47 by Deleted ...@, Aug 6 2009
This would of been funny if it was Rick Astley's head
When you go incognito, it should give Glen a blindfold.
Comment 49 by Deleted ...@, Aug 6 2009
"would have" or "would've", not "would of". :)
@drewshays: c'mon, somebody has to make people panic ;)
Comment 51 by Deleted ...@, Aug 6 2009
Kilroy was there.
Comment 52 by Deleted ...@, Aug 6 2009
Comment 53 Deleted
Comment 54 by Deleted ...@, Aug 6 2009
I got one too. It was in Ubuntu Jaunty, when I was using Chrome.
Comment 55 by omat...@gmail.com, Aug 6 2009
The changelog implies that the original /src/app/resources/linux_close.png file was 
just moved to /src/app/resources/linux_close_glen.png .

Since this file is no longer referenced, isn't it just cruft in the codebase?  It 
should be removed, even though it isn't included in the final binary (unless it is part 
of a useful test, in which case it should be renamed and moved to reflect this)
Comment 56 by yure...@gmail.com, Aug 6 2009
New buttons are the worst close\resotre buttons I've ever seen. How can I return old 
ones? 
Comment 57 by Deleted ...@, Aug 6 2009
do you want the head back?
re 56: you might be seeing the spacing issues on the new buttons. We've fixed them to 
have proper spacing but that change did not get into the latest dev channel release, 
so you will have to live with the crappy spacing for ~1 week if you are on google-
chrome (if you are using the ubuntu nightlies, you will only have to live with it for 
a day or so).
Comment 59 by evan@chromium.org, Aug 6 2009
re 58: actually, mmoss set me straight -- we did make this dev channel.
the official release should go straight from vista buttons to new-style buttons, no 
heads or other ugly bits in the interim.  (hopefully QA would catch something like the 
glenhead.)
Reminds me of the "Toasty!" guy from mortal kombat.
Comment 61 by Deleted ...@, Aug 7 2009
Damn that's awesome!

http://www.p4o.net
Comment 62 by glen@chromium.org, Aug 7 2009
 Issue 18756  has been merged into this issue.
Comment 63 by Deleted ...@, Aug 8 2009
Glen is a dead hacker and he's prying into your desktop. Be afraid. Be very afraid!
Comment 64 by lpica...@gmail.com, Aug 10 2009
TOASTY [2]
From Mega Drive...

Comment 65 by elc...@gmail.com, Aug 10 2009
I already miss Glen...
Comment 66 by Deleted ...@, Aug 17 2009
http://meg2.com/
Comment 67 by evan@chromium.org, Aug 18 2009
You can use the --glen flag to bring him back.
Comment 68 by Deleted ...@, Aug 21 2009
I think it's a mischief.
Comment 69 by Deleted ...@, Oct 6 2009
Are you serious about --glen flag?

I mean, yes, it's a easter egg... but it's a quite weird and unpolite :).
Comment 70 by dbeam@chromium.org, Jan 27 2012
Please, bring back the --glen flag (if it ever existed), perhaps to replace all multi-profile user icons (and names)?
Comment 71 by glen@chromium.org, Jan 27 2012
--glen?

Y NOT glen++?
better off implementing glen as an extension.
Project Member Comment 73 by bugdroid1@chromium.org, Oct 13 2012
Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Sign in to add a comment