DevTools: preload treeoutline triangles image

DevTools: preload treeoutline triangles image

The "black patches" happen due to the treeoutline image being slow to load.
This patch adds a preloading for the image.

BUG=690427
R=pfeldman

Review-Url: https://codereview.chromium.org/2688623007
Cr-Commit-Position: refs/heads/master@{#450255}
Committed: https://chromium.googlesource.com/chromium/src/+/6f380e767298a49098c17a1bea12ff283fe25054

[lushnikov, 2017-02-10 23:28:53]

please, take a look

[lushnikov, 2017-02-10 23:28:57]

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

[commit-bot: I haz the power, 2017-02-10 23:29:35]

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-02-11 01:20:42]

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

[commit-bot: I haz the power, 2017-02-11 01:20:44]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2017-02-11 03:40:07]

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

[lushnikov, 2017-02-11 03:40:18]

addressed comments; please take another look

[commit-bot: I haz the power, 2017-02-11 03:40:34]

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-02-11 05:37:35]

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

[commit-bot: I haz the power, 2017-02-11 05:37:35]

Dry run: This issue passed the CQ dry run.

[pfeldman, 2017-02-13 18:18:12]

https://codereview.chromium.org/2688623007/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/ui/treeoutline.js (right):

https://codereview.chromium.org/2688623007/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/ui/treeoutline.js:1217: (function()
{
Is this temporary? I wish all the generic images were a part of a single sprite.

[lushnikov, 2017-02-13 23:37:17]

well, if we have three separate spritesheets with icons of different sizes, than
we'll have to have this in some way

[pfeldman, 2017-02-14 00:34:00]

yeah, I guess it makes sense, thanks!

lgtm

[lushnikov, 2017-02-14 00:34:39]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2017-02-14 00:35:14]

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-02-14 04:52:58]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487032479929480,
"parent_rev": "190ccc9054ea38a03149507e29f6d31692470d46", "commit_rev":
"6f380e767298a49098c17a1bea12ff283fe25054"}

[commit-bot: I haz the power, 2017-02-14 04:53:31]

Description was changed from

==========
DevTools: preload treeoutline triangles image

The "black patches" happen due to the treeoutline image being slow to load.
This patch adds a preloading for the image.

BUG=690427
R=pfeldman
==========

to

==========
DevTools: preload treeoutline triangles image

The "black patches" happen due to the treeoutline image being slow to load.
This patch adds a preloading for the image.

BUG=690427
R=pfeldman

Review-Url: https://codereview.chromium.org/2688623007
Cr-Commit-Position: refs/heads/master@{#450255}
Committed:
https://chromium.googlesource.com/chromium/src/+/6f380e767298a49098c17a1bea12...
==========

[commit-bot: I haz the power, 2017-02-14 04:53:32]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/6f380e767298a49098c17a1bea12...