aboutsummaryrefslogtreecommitdiffhomepage
path: root/debugger/QT/SkDebuggerGUI.h
Commit message (Collapse)AuthorAge
* Unbust the old debuggerGravatar robertphillips2016-03-01
| | | | | | | | AFAICT the recent STL changes have introduced some QT conflicts. GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1753813002 Review URL: https://codereview.chromium.org/1753813002
* Update debugger UI to auto-refresh the directory listingGravatar djsollen2015-11-18
| | | | Review URL: https://codereview.chromium.org/1462633002
* Clean up dead code: PictureRendererGravatar mtklein2015-11-18
| | | | | | | | Doesn't appear to actually be used by anything. BUG=skia: Review URL: https://codereview.chromium.org/1457753002
* Get debugger compiling againGravatar robertphillips2015-10-26
| | | | | | Recent house cleaning has broken the debugger - mainly around the defunct profiling feature. This makes CL it even more defunct. Review URL: https://codereview.chromium.org/1416723006
* Debugger: remove dead feature (SkPicture offset display) & fix bug ↵Gravatar robertphillips2015-03-26
| | | | | | | | | (unbalanced indents) Displaying the offset into an SkPicture hasn't worked for a while so this CL deletes the feature. When "Save Layer" was renamed to "SaveLayer" the code that computes the indent in the list view was broken. This CL patches the problem. Review URL: https://codereview.chromium.org/1034733004
* debugger: SW rasterize the picture only when it changesGravatar kkinnunen2015-01-13
| | | | | | | | | | | | | | Rasterize the picture only when it changes, not every time raster widget paints itself. Removes include SkForceLinking.h while organizing the SkDebuggerGUI.cpp includes. This is part of the work trying to remove bugs that come from SkDebugCanvas stateful draw. Part of the state comes from some optimization that this tries to reproduce in a different way. Review URL: https://codereview.chromium.org/839743003
* debugger: Update inspector view data consistently while pausedGravatar kkinnunen2015-01-07
| | | | | | | | | | | | Make all fields of inspector view (details tab, clipstack tab, geometry view) update the correct info when user selects a draw command. Also update the info regardless if the painting is paused or not. Current clip and matrix will not update consistently even after this patch, as they depend on stateful debug canvas draw (may be fixed later). Review URL: https://codereview.chromium.org/835903002
* debugger: Remove few redundant variablesGravatar kkinnunen2015-01-05
| | | | | | | | | | | Use the GUI widget states instead of instance variables reflecting the widget state. Fixes the case where pause state would be left on when switching files, but the image would change to the last frame of newly loaded image. Review URL: https://codereview.chromium.org/810033004
* debugger: Move Skia initialization out of the GUI widget classGravatar kkinnunen2015-01-02
| | | | | | | | | | Move Skia initialization out of the GUI widget class to the main function. Before, Skia may have been already called before the SkGraphics::Init was run. Review URL: https://codereview.chromium.org/822583003
* debugger: Make draw command image widget resizeGravatar kkinnunen2014-12-30
| | | | | | | | | | | | | | | | | | | | | | | | | | Make draw command image widget resize. The widget was not resizing, effectively preventing the window from being resized smaller. Make the rasterized draw command image be proportional to the widget size. The draw rasterization canvas is still an equilateral rectangle with dimensions of the smaller side of the widget. Makes the widget re-rasterize the image only when the draw command changes, not for each widget paint. Renames the widget from "image widget" to "draw command geometry widget". Makes the background of the image black, similar to the raster widget background. Adds a tooltip saying "Command geometry" for the widget, so that user might understand what the contents should be. This commit is part of work that tries to make the debugger window to be a bit more resizeable, so that it would fit 1900x1200 screen. Review URL: https://codereview.chromium.org/787143004
* debugger: Make settings widget resizeableGravatar kkinnunen2014-12-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The settings widget was laid out by hand, eg. the widths of most elements were hardcoded. This prevented it from being laid out by the Qt widget system. This in turn prevents the widget from shrinking to its optimal size. This in turn causes cascading effect where the geometry of many of the UI widgets has to be hard-coded. This in turn prevents proper resizing of the UI. Make Qt layout the settings widget by following changes: a) Group settings in QGroupBox groups instead of hand-written layouts and labels hardcoded in certain places. b) Remove "Expanding" size policy from settings widget. The widget calculates its own size based on the widgets inside. Thus "Preferred" is the correct policy to use, as expanding the widget will not bring any new content visible. c) Remove maximum width 250 from Settings widget d) Make "canvas settings and image layout", eg. the horizontal layout holding the settings widget divide the space between the picture and settings like so: settings uses up only as much as it needs (stretch factor 0), while picture uses up everything else (stretch factor 1). In order to do a) reasonably, reorganize the UI and the code a bit: a1) Rename settings group "visual filter" to "Visualizations". a2) Make "visual filter: on/off" combo box a checkbox in "Visualizations". a3) Move "Mega viz" setting checkbox from "raster" (or "render targets") section to "Visualizations" a4) Move "PathOps" setting checkbox from "raster" to "Visualizations" a5) Make Raster and GL checkboxes use QGroupBox checkbox feature a6) Move "current command", "command hitbox" and "zoom level" from "Settings" widget to part of "Inspector" concept. These pieces of information are now visualized as their own box in the right-hand bar, below settings. a7) Do not expose settings user selects through the UI widgets state that record the user interacts with. Instead, expose it as state of the "settings widget". Thus settings widget provides "raster settings changed" signal, which the client hook to and then query the state through the object API. This makes the full window a bit shorter. This commit is part of work that tries to make the debugger window to be a bit more resizeable, so that it would fit 1900x1200 screen. Review URL: https://codereview.chromium.org/829933002
* Change DebugCanvas API to not encourage memory leaksGravatar kkinnunen2014-12-30
| | | | | | | | | | Pass command strings and offset arrays as out parameters instead of returning new arrays from the functions. This simplifies debugger leak investigations, as the app leaks less by design. Review URL: https://codereview.chromium.org/821663003
* Split SkPicturePlayback out of SkPictureDataGravatar robertphillips2014-07-07
| | | | | | | | | | | | | | | | | | | This splits the playback functionality out of SkPictureData. The old SkPictureData::draw method is pulled out along with its supporting functions as verbatim as possible. Some follow on CLs will be required to: re-enable profiling in the debugger (and remove the vestiges of SkTimedPicture) re-enable display of command offsets in the picture (this should probably wait until we've switched to SkRecord though) Clean up CachedOperationList (maybe fuse with SkPicture::OperationList) Split SkPicturePlayback into a base class and two derived classes Implement parallel version of GatherGPUInfo for SkRecord Landing this is blocked on removing Android's use of the abortPlayback entry point. R=mtklein@google.com, reed@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/377623002
* This adds a checkbox to the debugger to allow seeing the effect pathops has ↵Gravatar commit-bot@chromium.org2014-05-19
| | | | | | | | | | | | | | on the clip. A new tab shows the C code that the pathops generate. Once in place, this CL found a bug in the pathops code where it was not handling empty clip stack elements correctly. The Cl also has the change to SkCanvas to fix this bug. R=robertphillips@google.com, reed@google.com Author: caryclark@google.com Review URL: https://codereview.chromium.org/282283002 git-svn-id: http://skia.googlecode.com/svn/trunk@14774 2bbb7eff-a529-9590-31e7-b0007b416f81
* Debugger improvementsGravatar commit-bot@chromium.org2014-03-25
| | | | | | | | | | | | | | | This CL: improves the 'SaveAs' functionality allows switching between # and offset indexing in the command list minor nit cleanup R=bsalomon@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/211383003 git-svn-id: http://skia.googlecode.com/svn/trunk@13950 2bbb7eff-a529-9590-31e7-b0007b416f81
* The new "mega" viz mode both adds visualization and forces all drawing to ↵Gravatar commit-bot@chromium.org2014-03-03
| | | | | | | | | | | | the visible canvas (by replacing the active saveLayers with saves). R=bsalomon@google.com Author: robertphillips@google.com Review URL: https://codereview.chromium.org/185293007 git-svn-id: http://skia.googlecode.com/svn/trunk@13640 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make the debugger left panel resizable.Gravatar commit-bot@chromium.org2013-11-21
| | | | | | | | | | | | Use QSplitters to allow resizing the left debugger panels. R=robertphillips@google.com Author: fmalita@chromium.org Review URL: https://codereview.chromium.org/79573002 git-svn-id: http://skia.googlecode.com/svn/trunk@12336 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add texture filtering override to debuggerGravatar robertphillips@google.com2013-10-17
| | | | | | | | https://codereview.chromium.org/27716003/ git-svn-id: http://skia.googlecode.com/svn/trunk@11846 2bbb7eff-a529-9590-31e7-b0007b416f81
* Support MSAA in the picture debuggerGravatar commit-bot@chromium.org2013-08-02
| | | | | | | | | | | | | | | | | Add radio buttons for setting the GL sample count to 0 ("off"), 4 or 16. Change the default mode of the GL widget to MSAA4. Previous behavior corresponded to "off". BUG=1459 R=robertphillips@google.com Author: kkinnunen@nvidia.com Review URL: https://chromiumcodereview.appspot.com/21752002 git-svn-id: http://skia.googlecode.com/svn/trunk@10509 2bbb7eff-a529-9590-31e7-b0007b416f81
* Separate showing and hiding the debugger settings and inspector.Gravatar bungeman@google.com2013-07-01
| | | | | | | | | | | | | | | | Internally the settings and inspector in the debugger are separate, but the inspector button toggles both of them. This is unfortunate, because both of these have large minimum vertical sizes. This means that those with vertical resolutions less than ~1024 cannot see the the details tab. This change separates the settings and inspector toggles into two so that the interface is usable on screens of smaller size. R=robertphillips@google.com Review URL: https://codereview.chromium.org/18279002 git-svn-id: http://skia.googlecode.com/svn/trunk@9835 2bbb7eff-a529-9590-31e7-b0007b416f81
* Remove offsetcanvas from debuggerGravatar robertphillips@google.com2013-05-23
| | | | | | | | https://codereview.chromium.org/15746003/ git-svn-id: http://skia.googlecode.com/svn/trunk@9253 2bbb7eff-a529-9590-31e7-b0007b416f81
* Normalize the debugger's profile times by the repeat countGravatar robertphillips@google.com2013-03-07
| | | | | | | | https://codereview.appspot.com/7520043/ git-svn-id: http://skia.googlecode.com/svn/trunk@8025 2bbb7eff-a529-9590-31e7-b0007b416f81
* Allow debugger to be compiled without GaneshGravatar robertphillips@google.com2013-02-13
| | | | | | | | https://codereview.appspot.com/7311084/ git-svn-id: http://skia.googlecode.com/svn/trunk@7710 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add overdraw visualization to the debuggerGravatar robertphillips@google.com2013-02-06
| | | | | | | | https://codereview.appspot.com/7267043/ git-svn-id: http://skia.googlecode.com/svn/trunk@7627 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add command line options to debuggerGravatar robertphillips@google.com2013-01-28
| | | | | | | | https://codereview.appspot.com/7221048/ git-svn-id: http://skia.googlecode.com/svn/trunk@7418 2bbb7eff-a529-9590-31e7-b0007b416f81
* Make debugger profiling honor deleted commandsGravatar robertphillips@google.com2012-12-07
| | | | | | | | https://codereview.appspot.com/6906043/ git-svn-id: http://skia.googlecode.com/svn/trunk@6713 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add drawing of paths to debuggerGravatar robertphillips@google.com2012-11-21
| | | | | | | | https://codereview.appspot.com/6850082/ git-svn-id: http://skia.googlecode.com/svn/trunk@6532 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2012-11-20
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@6500 2bbb7eff-a529-9590-31e7-b0007b416f81
* Breakdown profile time by command type in the overview paneGravatar robertphillips@google.com2012-11-19
| | | | | | | | https://codereview.appspot.com/6851073/ git-svn-id: http://skia.googlecode.com/svn/trunk@6492 2bbb7eff-a529-9590-31e7-b0007b416f81
* Add profiling to debuggerGravatar robertphillips@google.com2012-11-15
| | | | | | | | https://codereview.appspot.com/6817114/ git-svn-id: http://skia.googlecode.com/svn/trunk@6430 2bbb7eff-a529-9590-31e7-b0007b416f81
* Update debugger GUI to have profile button & list widget to optionally ↵Gravatar robertphillips@google.com2012-11-12
| | | | | | | | | | display time https://codereview.appspot.com/6813117/ git-svn-id: http://skia.googlecode.com/svn/trunk@6385 2bbb7eff-a529-9590-31e7-b0007b416f81
* Replaced all instances of std strings and vectors in favor of SkStrings and ↵Gravatar chudy@google.com2012-08-07
| | | | | | | | SkTDArrays within skia code Review URL: https://codereview.appspot.com/6445088 git-svn-id: http://skia.googlecode.com/svn/trunk@4995 2bbb7eff-a529-9590-31e7-b0007b416f81
* Refactoring into a public facing facing SkDebugger class first pass.Gravatar chudy@google.com2012-08-07
| | | | | | Review URL: https://codereview.appspot.com/6450096 git-svn-id: http://skia.googlecode.com/svn/trunk@4986 2bbb7eff-a529-9590-31e7-b0007b416f81
* Moved the ownership of the current clip and current matrix into the debug ↵Gravatar chudy@google.com2012-08-03
| | | | | | | | canvas as part of the upcoming general refactor to everything living in debug canvas. Review URL: https://codereview.appspot.com/6447077 git-svn-id: http://skia.googlecode.com/svn/trunk@4950 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fixed numerous mem errorsGravatar chudy@google.com2012-07-31
| | | | | | Review URL: https://codereview.appspot.com/6441070 git-svn-id: http://skia.googlecode.com/svn/trunk@4849 2bbb7eff-a529-9590-31e7-b0007b416f81
* Skia Debugger can now save modified pictures.Gravatar chudy@google.com2012-07-28
| | | | | | Review URL: https://codereview.appspot.com/6442055 git-svn-id: http://skia.googlecode.com/svn/trunk@4821 2bbb7eff-a529-9590-31e7-b0007b416f81
* Key Shortcuts for Zooming. Refactored some event propogation.Gravatar chudy@google.com2012-07-26
| | | | | | Review URL: https://codereview.appspot.com/6452052 git-svn-id: http://skia.googlecode.com/svn/trunk@4794 2bbb7eff-a529-9590-31e7-b0007b416f81
* GL Widget AlphaGravatar chudy@google.com2012-07-26
| | | | | | Review URL: https://codereview.appspot.com/6422060 git-svn-id: http://skia.googlecode.com/svn/trunk@4789 2bbb7eff-a529-9590-31e7-b0007b416f81
* Quick bug fix that was rendering debugger unusable.Gravatar chudy@google.com2012-07-19
| | | | | | Review URL: https://codereview.appspot.com/6422050 git-svn-id: http://skia.googlecode.com/svn/trunk@4668 2bbb7eff-a529-9590-31e7-b0007b416f81
* Added more features to deletes and breakpointsGravatar chudy@google.com2012-07-17
| | | | | | Review URL: https://codereview.appspot.com/6406050 git-svn-id: http://skia.googlecode.com/svn/trunk@4637 2bbb7eff-a529-9590-31e7-b0007b416f81
* Cleaned up icons, added keyboard shortcuts to everythingGravatar chudy@google.com2012-07-16
| | | | | | | | Keyboard Shortcuts and Menu reordering Review URL: https://codereview.appspot.com/6400051 git-svn-id: http://skia.googlecode.com/svn/trunk@4630 2bbb7eff-a529-9590-31e7-b0007b416f81
* Feature, HitBox selects current commandGravatar chudy@google.com2012-07-12
| | | | | | Review URL: https://codereview.appspot.com/6345091 git-svn-id: http://skia.googlecode.com/svn/trunk@4588 2bbb7eff-a529-9590-31e7-b0007b416f81
* Bug Fix: Delete redraws correctly when redraw command execution is pausedGravatar chudy@google.com2012-07-12
| | | | | | Review URL: https://codereview.appspot.com/6392053 git-svn-id: http://skia.googlecode.com/svn/trunk@4567 2bbb7eff-a529-9590-31e7-b0007b416f81
* Made everything in SkDebuggerGUI live on the stackGravatar chudy@google.com2012-07-10
| | | | | | Review URL: https://codereview.appspot.com/6389043 git-svn-id: http://skia.googlecode.com/svn/trunk@4502 2bbb7eff-a529-9590-31e7-b0007b416f81
* Added stubs for zoom info panel.Gravatar chudy@google.com2012-07-09
| | | | | | Review URL: https://codereview.appspot.com/6350071 git-svn-id: http://skia.googlecode.com/svn/trunk@4493 2bbb7eff-a529-9590-31e7-b0007b416f81
* clt debuggerGravatar chudy@google.com2012-06-29
Review URL: https://codereview.appspot.com/6267043 git-svn-id: http://skia.googlecode.com/svn/trunk@4404 2bbb7eff-a529-9590-31e7-b0007b416f81