After chromium stop using gtk+ for Linux/X11 all not ASCII character in window title start looks li…

After chromium stop using gtk+ for Linux/X11 all not ASCII character in window title start looks like garbage. This patch fixes this issue, by using algorithm similar to gtk+/Qt's way of setting window titles.

BUG=378096
R=erg@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=286058

[Elliot Glaysher, 2014-07-28 17:55:25]

This looks good, but I don't see your name in the AUTHORS file.

Please:

1) Sign the CLA if you haven't already:
https://developers.google.com/open-source/cla/individual
2) Add your name to AUTHORS and reupload

Then I can commit this.

[dushistov, 2014-07-28 19:04:33]

On 2014/07/28 17:55:25, Elliot Glaysher wrote:
> This looks good, but I don't see your name in the AUTHORS file.
> 
> Please:
> 
> 1) Sign the CLA if you haven't already:
> https://developers.google.com/open-source/cla/individual

I did it before submit patch to review, after submiting
"electronical form" server reply to me with "we connect with you",
but no reaction, even email notification. Is it should be so?

> 2) Add your name to AUTHORS and reupload
> 

Forgot about that, reupload with AUTHORS modification.

[Elliot Glaysher, 2014-07-28 19:08:30]

lgtm; I'll CQ this immediately.

On 2014/07/28 19:04:33, dushistov wrote:
> On 2014/07/28 17:55:25, Elliot Glaysher wrote:
> > This looks good, but I don't see your name in the AUTHORS file.
> > 
> > Please:
> > 
> > 1) Sign the CLA if you haven't already:
> > https://developers.google.com/open-source/cla/individual
> 
> I did it before submit patch to review, after submiting
> "electronical form" server reply to me with "we connect with you",
> but no reaction, even email notification. Is it should be so?

I just checked internally, and saw that you're on the signers list. I apologize;
I had assumed that since you weren't in the AUTHORS file and hadn't modified it,
that you also didn't know about the CLA.

> > 2) Add your name to AUTHORS and reupload
> > 
> 
> Forgot about that, reupload with AUTHORS modification.

Thank you.

[Elliot Glaysher, 2014-07-28 19:08:34]

The CQ bit was checked by erg@chromium.org

[commit-bot: I haz the power, 2014-07-28 20:36:04]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dushistov@gmail.com/414413002/40001

[commit-bot: I haz the power, 2014-07-29 00:07:32]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_chromium_chromeos_clang_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[commit-bot: I haz the power, 2014-07-29 02:52:38]

Change committed as 286058

[hashimoto, 2014-07-29 04:56:02]

A revert of this CL has been created in
https://codereview.chromium.org/428823003/ by hashimoto@chromium.org.

The reason for reverting is: Speculative revert to fix
"_XTextListToTextProperty" leaking on Linux ASan LSan bots:
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Te...
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Te...
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Te....