<table><tr><td style="">alex updated this revision to Diff 82789.<br />alex retitled this revision from "WIP BookmarksRunner: Change caching databases" to "BookmarksRunner: Change caching databases".<br />alex edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-vymdppe4dratwdt/">(Show Details)</a><br />alex edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-yl3e4t6qxi42emy/">(Show Details)</a><br />alex 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/D29726">View Revision</a></tr></table><br /><div><div><p>Copy chrome files only if they changed</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>The <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">default behaviour for chrome is keept (for now)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">chrome favicon database is also only copied if needed<br />
and the check is also made in the prepare method and not just when<br />
the profile is initialized</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">S</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">For both firefox and chrome: <br />
Plugin s</span>hows icons for results.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D29726?vs=82772&id=82789">https://phabricator.kde.org/D29726?vs=82772&id=82789</a></div></div><br /><div><strong>BRANCH</strong><div><div>avid_unnecessary_copying (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29726">https://phabricator.kde.org/D29726</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>runners/bookmarks/browser.h<br />
runners/bookmarks/browsers/chrome.cpp<br />
runners/bookmarks/browsers/chromefindprofile.cpp<br />
runners/bookmarks/browsers/findprofile.h<br />
runners/bookmarks/browsers/firefox.cpp<br />
runners/bookmarks/faviconfromblob.cpp<br />
runners/bookmarks/fetchsqlite.cpp<br />
runners/bookmarks/fetchsqlite.h<br />
runners/bookmarks/tests/testchromebookmarks.cpp</div></div></div><br /><div><strong>To: </strong>alex<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>