The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/a594226807407dbbff625b9207bccf31c84cc28d

commit a594226807407dbbff625b9207bccf31c84cc28d
Author: David Reveman <reveman@chromium.org>
Date: Mon Apr 16 16:20:38 2018

vm_tools: Remove show window title option from sommelier.

We want to be able to show the window title in overview mode. In
order to support this, we need to always set the title and instead
have chrome hide it while not in overview mode.

TBR=smbarber@chromium.org
BUG= chromium:831383 
TEST='sommelier -X xterm' shows window title in overview mode

Change-Id: I670bb7a1726533c4a56adeb82e4f2d7f111f3e99
Reviewed-on: https://chromium-review.googlesource.com/1013245
Commit-Ready: David Reveman <reveman@chromium.org>
Tested-by: David Reveman <reveman@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>

[modify] https://crrev.com/a594226807407dbbff625b9207bccf31c84cc28d/vm_tools/sommelier/sommelier.c

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e4f22c7abbb0a2df8c56394511e549e12520f06a

commit e4f22c7abbb0a2df8c56394511e549e12520f06a
Author: Jun Mukai <mukai@google.com>
Date: Wed Apr 18 00:00:55 2018

Hide the title in the title bar for ARC apps

As is said in the bug, we decided not to show the title in the
title bar for ARC apps. However, debugging information (stored
in extra_title_) wants to appear, and the overview mode wants
to show the actual title (app name) instead of the debugging
information.

So this CL changed ClientControlledShellSurface as:
- GetWindowTitle always return extra_title_ and no title_ so
  title won't appear anymore
- when the widget's title is updated, directly specify the
  native window's title which is used by the overview mode

Bug:  831383 
Test: manually, exo_unittests
Change-Id: Ifa275bd6f5c72b8ca36cccf7f9c629498c4a670e
Reviewed-on: https://chromium-review.googlesource.com/1012783
Commit-Queue: Jun Mukai <mukai@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551529}
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/ash/frame/custom_frame_view_ash.cc
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/ash/frame/custom_frame_view_ash.h
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/ash/frame/default_frame_header.cc
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/ash/frame/default_frame_header.h
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/ash/frame/header_view.h
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/components/exo/client_controlled_shell_surface.cc
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/components/exo/client_controlled_shell_surface.h
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/components/exo/client_controlled_shell_surface_unittest.cc
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/components/exo/shell_surface_base.cc
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/components/exo/shell_surface_base.h
[modify] https://crrev.com/e4f22c7abbb0a2df8c56394511e549e12520f06a/components/exo/shell_surface_unittest.cc

Requesting merge of #2 crrev.com/551529 to M67.

Not a small change; is this mostly cosmetic?    User impact if declined?

Your change meets the bar and is auto-approved for M67. Please go ahead and merge the CL to branch 3396 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

re #5: indeed it touches several files, but most of the changes are tiny, adding some interfaces to pass through, I believe that's safe.  This is a part of issue 820528, and will cause user visible effect (only on ARC++ apps in ChromeOS though). 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/02ea6fb617628215443190177739edd6fcfe1358

commit 02ea6fb617628215443190177739edd6fcfe1358
Author: Jun Mukai <mukai@google.com>
Date: Mon Apr 23 17:58:05 2018

Hide the title in the title bar for ARC apps

As is said in the bug, we decided not to show the title in the
title bar for ARC apps. However, debugging information (stored
in extra_title_) wants to appear, and the overview mode wants
to show the actual title (app name) instead of the debugging
information.

So this CL changed ClientControlledShellSurface as:
- GetWindowTitle always return extra_title_ and no title_ so
  title won't appear anymore
- when the widget's title is updated, directly specify the
  native window's title which is used by the overview mode

TBR=mukai@google.com

(cherry picked from commit e4f22c7abbb0a2df8c56394511e549e12520f06a)

Bug:  831383 
Test: manually, exo_unittests
Change-Id: Ifa275bd6f5c72b8ca36cccf7f9c629498c4a670e
Reviewed-on: https://chromium-review.googlesource.com/1012783
Commit-Queue: Jun Mukai <mukai@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#551529}
Reviewed-on: https://chromium-review.googlesource.com/1024247
Reviewed-by: Jun Mukai <mukai@chromium.org>
Cr-Commit-Position: refs/branch-heads/3396@{#227}
Cr-Branched-From: 9ef2aa869bc7bc0c089e255d698cca6e47d6b038-refs/heads/master@{#550428}
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/ash/frame/custom_frame_view_ash.cc
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/ash/frame/custom_frame_view_ash.h
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/ash/frame/default_frame_header.cc
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/ash/frame/default_frame_header.h
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/ash/frame/header_view.h
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/components/exo/client_controlled_shell_surface.cc
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/components/exo/client_controlled_shell_surface.h
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/components/exo/client_controlled_shell_surface_unittest.cc
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/components/exo/shell_surface_base.cc
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/components/exo/shell_surface_base.h
[modify] https://crrev.com/02ea6fb617628215443190177739edd6fcfe1358/components/exo/shell_surface_unittest.cc

Verified on 10575.32.0, 67.0.3396.41.

I'm still seeing this issue on M68 (10684.0.0, 68.0.3432.0). Actual title (app name) is shown on Title bar of ARC app, both in normal & tablet mode.

There was a regression, but the fix was merged recently as http://crrev.com/560373. Marking as fixed.

Verified on M68 (10715.0.0, 68.0.3438.0) caroline device.