Some more improvements/fixes to the VulkanViewer

Some more improvements/fixes to the VulkanViewer

* display GM name in title bar
* add ms timer for update loop
* add ms/frame meter
* fix vsync
* add some notes for later

GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1873453002

Committed: https://skia.googlesource.com/skia/+/3d6ed3ae75e45b36f92d7555893953f519c2a63d

[jvanverth1, 2016-04-07 15:36:21]

Description was changed from

==========
Some more improvements/fixes to the VulkanViewer
==========

to

==========
Some more improvements/fixes to the VulkanViewer
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[jvanverth1, 2016-04-07 15:41:39]

Description was changed from

==========
Some more improvements/fixes to the VulkanViewer
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Some more improvements/fixes to the VulkanViewer

* display GM name in title bar
* add ms timer for update loop
* add ms/frame meter
* fix vsync
* add some notes for later

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[jvanverth1, 2016-04-07 15:41:53]

jvanverth@google.com changed reviewers:
+ djsollen@google.com, egdaniel@google.com

[egdaniel, 2016-04-07 17:03:16]

lgtm with minor comments

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/VulkanTest...
File tools/vulkan/VulkanTestContext.h (right):

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/VulkanTest...
tools/vulkan/VulkanTestContext.h:84: sk_sp<SkSurface>* fSurfaces;       //
wrapped surface for those images
is this another line ending issue?

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/viewer/Vul...
File tools/vulkan/viewer/VulkanViewer.cpp (right):

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/viewer/Vul...
tools/vulkan/viewer/VulkanViewer.cpp:90: SkRect rect =
SkRect::MakeXYWH(SkIntToScalar(canvasSize.fWidth - kDisplayWidth -
kDisplayPadding),
100 chars

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/win/Window...
File tools/vulkan/win/Window_win.cpp (right):

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/win/Window...
tools/vulkan/win/Window_win.cpp:14: #include "Timer.h"
What does this guy need timer.h added? Was it just missing before?

[jvanverth1, 2016-04-07 17:57:03]

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/VulkanTest...
File tools/vulkan/VulkanTestContext.h (right):

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/VulkanTest...
tools/vulkan/VulkanTestContext.h:84: sk_sp<SkSurface>* fSurfaces;       //
wrapped surface for those images
On 2016/04/07 17:03:16, egdaniel wrote:
> is this another line ending issue?

Done.

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/viewer/Vul...
File tools/vulkan/viewer/VulkanViewer.cpp (right):

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/viewer/Vul...
tools/vulkan/viewer/VulkanViewer.cpp:90: SkRect rect =
SkRect::MakeXYWH(SkIntToScalar(canvasSize.fWidth - kDisplayWidth -
kDisplayPadding),
On 2016/04/07 17:03:16, egdaniel wrote:
> 100 chars

Done.

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/win/Window...
File tools/vulkan/win/Window_win.cpp (right):

https://codereview.chromium.org/1873453002/diff/20001/tools/vulkan/win/Window...
tools/vulkan/win/Window_win.cpp:14: #include "Timer.h"
On 2016/04/07 17:03:16, egdaniel wrote:
> What does this guy need timer.h added? Was it just missing before?

Done.

[jvanverth1, 2016-04-07 17:57:07]

The CQ bit was checked by jvanverth@google.com

[jvanverth1, 2016-04-07 17:57:08]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1873453002/#ps40001
(title: "Address comments")

[commit-bot: I haz the power, 2016-04-07 17:57:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1873453002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1873453002/40001

[commit-bot: I haz the power, 2016-04-07 18:09:54]

Description was changed from

==========
Some more improvements/fixes to the VulkanViewer

* display GM name in title bar
* add ms timer for update loop
* add ms/frame meter
* fix vsync
* add some notes for later

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Some more improvements/fixes to the VulkanViewer

* display GM name in title bar
* add ms timer for update loop
* add ms/frame meter
* fix vsync
* add some notes for later

GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/3d6ed3ae75e45b36f92d7555893953f519c2a63d
==========

[commit-bot: I haz the power, 2016-04-07 18:09:56]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/3d6ed3ae75e45b36f92d7555893953f519c2a63d