> On July 17, 2014, 11:21 a.m., David Edmundson wrote: > > src/declarativeimports/core/framesvgitem.h, line 171 > > <https://git.reviewboard.kde.org/r/119330/diff/1/?file=290720#file290720line171> > > > > We probably don't want these 2 public.
Edit: this whole class is not exported and we use these two from the node. Makes sense like this. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119330/#review62567 ----------------------------------------------------------- On July 17, 2014, 8:44 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119330/ > ----------------------------------------------------------- > > (Updated July 17, 2014, 8:44 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > Use FrameSVG as 9 tiles instead of uploading a big texture of the finished > frame each time. > > This also saves the cache being populated with full created frames in > different sizes; which end up taking up space in the disk and shared memory > cache as well as the GPU memory. > > A code path falls back to the original uploading the entire texture if > obscure settings are used, i.e overlay. > > Benchmarks: > - apitrace when resizing a frame goes from an average of 7.6ms per frame of > *CPU* time just for the swizzling and uploading to 1.4ms > > - GPU time also drops from 40us to 10us > > Themes will need to remove stretch-borders (when we gain nothing from > stretching; i.e Breeze) to get the most out of it. > > > Diffs > ----- > > src/declarativeimports/core/framesvgitem.cpp 8320212 > src/declarativeimports/core/svgitem.cpp 1ed0631 > src/plasma/framesvg.h dd6d8da > src/plasma/framesvg.cpp fcc6809 > src/plasma/private/framesvg_p.h 8aceef2 > tests/dialog.qml PRE-CREATION > tests/testborders.qml PRE-CREATION > src/declarativeimports/core/framesvgitem.h e155f6a > > Diff: https://git.reviewboard.kde.org/r/119330/diff/ > > > Testing > ------- > > Tested oxygen + breeze + some random (and ugly) themes from kde-look. > > Theme changes work. > > Everything looks the same; including the borders on oxygen. > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel