Fix text clipping in CompositorFrameTime Display

Fix text clipping in CompositorFrameTime Display

Because different OS have different fonts for HUD.
Set the width of CompositorFrameTime Display to be maximum
of graph width and text width to prevent text clipping.

BUG=445877
Committed: https://crrev.com/88b4cf8d33b499f972e1c447e741230170beeb8d
Cr-Commit-Position: refs/heads/master@{#322807}

[patro, 2015-03-26 05:07:01]

behara.ms@samsung.com changed reviewers:
+ danakj@chromium.org

[patro, 2015-03-26 05:07:01]

Kindly Review

[patro, 2015-03-26 17:08:31]

On 2015/03/26 05:07:01, patro wrote:
> Kindly Review

Text Clipping is occurring on MacOS because the text length is not same on all
OS.
Because the fonts are different in different OS.

Refer this link for MacOS text clipping issue
https://code.google.com/p/chromium/issues/detail?id=445877#c11

Kindly refer screenshot for this patch
https://code.google.com/p/chromium/issues/detail?id=445877#c14

[danakj, 2015-03-26 22:53:20]

https://codereview.chromium.org/1027393004/diff/1/cc/layers/heads_up_display_...
File cc/layers/heads_up_display_layer_impl.cc (right):

https://codereview.chromium.org/1027393004/diff/1/cc/layers/heads_up_display_...
cc/layers/heads_up_display_layer_impl.cc:616: paint.setTextSize(kFontHeight);
Can you add a MeasureText that sits beside DrawText and sets up the paint in all
the exact same ways?

https://codereview.chromium.org/1027393004/diff/1/cc/layers/heads_up_display_...
cc/layers/heads_up_display_layer_impl.cc:622: int contentsWidth =
std::max(title_text_width, kGraphWidth);
contents_width

[patro, 2015-03-28 17:42:35]

On 2015/03/26 22:53:20, danakj wrote:
>
https://codereview.chromium.org/1027393004/diff/1/cc/layers/heads_up_display_...
> File cc/layers/heads_up_display_layer_impl.cc (right):
> 
>
https://codereview.chromium.org/1027393004/diff/1/cc/layers/heads_up_display_...
> cc/layers/heads_up_display_layer_impl.cc:616: paint.setTextSize(kFontHeight);
> Can you add a MeasureText that sits beside DrawText and sets up the paint in
all
> the exact same ways?
> 
>
https://codereview.chromium.org/1027393004/diff/1/cc/layers/heads_up_display_...
> cc/layers/heads_up_display_layer_impl.cc:622: int contentsWidth =
> std::max(title_text_width, kGraphWidth);
> contents_width

Kindly Review

Thanks
Shyam Patro

[danakj, 2015-03-30 16:46:08]

LGTM

[danakj, 2015-03-30 16:46:12]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2015-03-30 16:46:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1027393004/20001

[commit-bot: I haz the power, 2015-03-30 17:48:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-30 17:49:30]

Patchset 2 (id:??) landed as
https://crrev.com/88b4cf8d33b499f972e1c447e741230170beeb8d
Cr-Commit-Position: refs/heads/master@{#322807}

[jianli, 2015-03-30 20:40:24]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1047013002/ by jianli@chromium.org.

The reason for reverting is: Speculative revert for Blink layout test failures
in WebKit Mac10.8 (retina):
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.8%20%28r....