<table><tr><td style="">head7 marked an inline comment as done.<br />head7 added a comment.
</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/D4250" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D4250#79740" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D4250#79740</a>, <a href="https://phabricator.kde.org/p/mwolff/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@mwolff</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>That sounds... odd - the rest of the MainWindow API we overload already works, or doesn't it? I.e. what we have in ktexteditorpluginintegration.h's MainWindow only ever gets called when it is the parent() object of the MainWindow in ktexteditor.</p>

<p>If that is not the case, then something is really wrong. We have a test case for this in shell/tests/test_ktexteditorpluginintegration.cpp - is that failing for you?</p></div>
</blockquote>

<p>No the test succeed on my computer:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Start testing of TestKTextEditorPluginIntegration <strong>*</strong>****</li>
</ul>

<p>Config: Using QtTest library 5.7.1, Qt 5.7.1 (x86_64-little_endian-lp64 shared (dynamic) release build; by GCC 6.2.0 20161027)<br />
QWARN  : TestKTextEditorPluginIntegration::initTestCase() cannot find .rc file "test_ktexteditorpluginintegrationui.rc" for component "test_ktexteditorpluginintegration"<br />
PASS   : TestKTextEditorPluginIntegration::initTestCase()<br />
PASS   : TestKTextEditorPluginIntegration::testApplication()<br />
PASS   : TestKTextEditorPluginIntegration::testMainWindow()<br />
PASS   : TestKTextEditorPluginIntegration::testPlugin()<br />
PASS   : TestKTextEditorPluginIntegration::testPluginUnload()<br />
PASS   : TestKTextEditorPluginIntegration::cleanupTestCase()<br />
Totals: 6 passed, 0 failed, 0 skipped, 0 blacklisted, 421ms</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Finished testing of TestKTextEditorPluginIntegration <strong>*</strong>****</li>
</ul>

<p>But there is no tests checking toKteView parent ?</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4250" rel="noreferrer">https://phabricator.kde.org/D4250</a></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>head7, brauch<br /><strong>Cc: </strong>mwolff, kdevelop-devel<br /></div>