DevTools: Show screenshots on the main flamechart

DevTools: Show screenshots on the main flamechart

The patch puts the screenshots on the frame bar and makes it expandable.

BUG=705054
Review-Url: https://codereview.chromium.org/2830343004
Cr-Commit-Position: refs/heads/master@{#466821}
Committed: https://chromium.googlesource.com/chromium/src/+/575cce8631ed083645523d47a35090d6ca6ccdfd

[alph, 2017-04-22 01:21:57]

alph@chromium.org changed reviewers:
+ dgozman@chromium.org

[alph, 2017-04-22 01:22:04]

The CQ bit was checked by alph@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-22 01:22:31]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-22 02:56:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-22 02:56:22]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[alph, 2017-04-24 18:47:26]

alph@chromium.org changed reviewers:
+ pfeldman@chromium.org
- dgozman@chromium.org

[alph, 2017-04-24 18:47:27]

Pavel, ptal

[pfeldman, 2017-04-24 21:07:17]

lgtm

https://codereview.chromium.org/2830343004/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChartDataProvider.js
(right):

https://codereview.chromium.org/2830343004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChartDataProvider.js:587:
var image = await imagePromise;
// -------- ASYNC --------

[alph, 2017-04-24 21:18:05]

https://codereview.chromium.org/2830343004/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChartDataProvider.js
(right):

https://codereview.chromium.org/2830343004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChartDataProvider.js:587:
var image = await imagePromise;
On 2017/04/24 21:07:17, pfeldman wrote:
> // -------- ASYNC --------

Done.

[alph, 2017-04-24 21:18:27]

The CQ bit was checked by alph@chromium.org

[alph, 2017-04-24 21:18:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/2830343004/#ps40001
(title: "added comment")

[commit-bot: I haz the power, 2017-04-24 21:18:59]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-24 23:26:16]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493068707257550,
"parent_rev": "da6a43e5fabf5aba0ee611e593d71809851727b0", "commit_rev":
"575cce8631ed083645523d47a35090d6ca6ccdfd"}

[commit-bot: I haz the power, 2017-04-24 23:26:30]

Description was changed from

==========
DevTools: Show screenshots on the main flamechart

The patch puts the screenshots on the frame bar and makes it expandable.

BUG=705054
==========

to

==========
DevTools: Show screenshots on the main flamechart

The patch puts the screenshots on the frame bar and makes it expandable.

BUG=705054

Review-Url: https://codereview.chromium.org/2830343004
Cr-Commit-Position: refs/heads/master@{#466821}
Committed:
https://chromium.googlesource.com/chromium/src/+/575cce8631ed083645523d47a350...
==========

[commit-bot: I haz the power, 2017-04-24 23:26:31]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/575cce8631ed083645523d47a350...

[alph, 2017-04-28 18:38:41]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2854473002/ by alph@chromium.org.

The reason for reverting is: Panning does not work, as well as HiDPI..