'[Differential] [Request, 147 lines] D2046: [autotests] Add a new test case which can verify the rend'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    [Differential] [Request, 147 lines] D2046: [autotests] Add a new test case which can verify the rend
From:       graesslin_(Martin_Gräßlin) <noreply () phabricator ! kde ! or
Date:       2016-06-29 17:32:35
Message-ID: differential-rev-PHID-DREV-hkqws25loy6rmtoickns-req () phabricator ! kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

graesslin created this revision.
graesslin added reviewers: KWin, Plasma on Wayland.
Restricted Application added subscribers: kwin, plasma-devel.
Restricted Application added projects: Plasma on Wayland, KWin.

REVISION SUMMARY
  The idea behind this autotest is inspired by bug 356328 which produced
  incorrect rendering results. Also it's inspired by openQA which performs
  image reference comparisons.

  This test case tries to go further. It creates reference images which
  must match the rendering result exactly. So far the test case verifies
  the start condition - kwin started and one frame is rendered with default
  cursor in the middle of the screen. And it verifies the moving of the
  cursor without any windows shown. Whenever the cursor moves a repaint
  should be triggered and the old and new area should be properly
  repainted.

  To support this the test needs some minor changes in KWin:

  - Scene provides a frameRendered signal - needed for waiting on frame
  - Scene and SceneQPainter are exported
  - SceneQPainter provides access to it's Backend, so that we get to the backbuffer
  - ScriptedEffectLoader is exported for getting a list of all scripted effects - (we \
don't want fade to manipulate the rendering)

REPOSITORY
  rKWIN KWin

BRANCH
  autotest-qpainter-rendering

REVISION DETAIL
  https://phabricator.kde.org/D2046

AFFECTED FILES
  autotests/integration/CMakeLists.txt
  autotests/integration/scene_qpainter_test.cpp
  effectloader.h
  scene.h
  scene_qpainter.cpp
  scene_qpainter.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: plasma-devel, kwin, hardening, jensreuterberg, sebas

[Attachment #5 (text/html)]

<table><tr><td style="">graesslin created this revision.<br />graesslin added \
reviewers: KWin, Plasma on Wayland.<br />Restricted Application added subscribers: \
kwin, plasma-devel.<br />Restricted Application added projects: Plasma on Wayland, \
KWin.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; \
float: right; color: #464C5C; font-weight: bold; border-radius: 3px; \
background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); \
display: inline-block; border: 1px solid rgba(71,87,120,.2);" \
href="https://phabricator.kde.org/D2046" rel="noreferrer">View \
Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The idea \
behind this autotest is inspired by bug 356328 which produced<br /> incorrect \
rendering results. Also it&#039;s inspired by openQA which performs<br /> image \
reference comparisons.</p>

<p>This test case tries to go further. It creates reference images which<br />
must match the rendering result exactly. So far the test case verifies<br />
the start condition - kwin started and one frame is rendered with default<br />
cursor in the middle of the screen. And it verifies the moving of the<br />
cursor without any windows shown. Whenever the cursor moves a repaint<br />
should be triggered and the old and new area should be properly<br />
repainted.</p>

<p>To support this the test needs some minor changes in KWin:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Scene provides a frameRendered signal - needed for \
waiting on frame</li> <li class="remarkup-list-item">Scene and SceneQPainter are \
exported</li> <li class="remarkup-list-item">SceneQPainter provides access to \
it&#039;s Backend, so that we get to the backbuffer</li> <li \
class="remarkup-list-item">ScriptedEffectLoader is exported for getting a list of all \
scripted effects - (we don&#039;t want fade to manipulate the rendering)</li> \
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN \
KWin</div></div></div><br \
/><div><strong>BRANCH</strong><div><div>autotest-qpainter-rendering</div></div></div><br \
/><div><strong>REVISION DETAIL</strong><div><a \
href="https://phabricator.kde.org/D2046" \
rel="noreferrer">https://phabricator.kde.org/D2046</a></div></div><br \
/><div><strong>AFFECTED \
FILES</strong><div><div>autotests/integration/CMakeLists.txt<br /> \
autotests/integration/scene_qpainter_test.cpp<br /> effectloader.h<br />
scene.h<br />
scene_qpainter.cpp<br />
scene_qpainter.h</div></div></div><br /><div><strong>EMAIL \
PREFERENCES</strong><div><a \
href="https://phabricator.kde.org/settings/panel/emailpreferences/" \
rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br \
/><div><strong>To: </strong>graesslin, KWin, Plasma on Wayland<br /><strong>Cc: \
</strong>plasma-devel, kwin, hardening, jensreuterberg, sebas<br /></div>

[Attachment #6 (text/plain)]

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

[prev in list] [next in list] [prev in thread] [next in thread] 