Get debugger compiling again

debugger/QT/SkDebuggerGUI.cpp

Index: debugger/QT/SkDebuggerGUI.cpp
diff --git a/debugger/QT/SkDebuggerGUI.cpp b/debugger/QT/SkDebuggerGUI.cpp
index 155fb63901ee91a3118bc7c8d837ace4bbac67ef..340c664b77d84543d76653bc931f9a7a1faace47 100644
--- a/debugger/QT/SkDebuggerGUI.cpp
+++ b/debugger/QT/SkDebuggerGUI.cpp
@@ -6,7 +6,6 @@
  */
 
 #include "SkDebuggerGUI.h"
-#include "PictureRenderer.h"
 #include "SkPictureData.h"
 #include "SkPicturePlayback.h"
 #include "SkPictureRecord.h"
@@ -14,17 +13,6 @@
 #include <QtGui>
 #include "sk_tool_utils.h"
 
-#if defined(SK_BUILD_FOR_WIN32)
-    #include "SysTimer_windows.h"
-#elif defined(SK_BUILD_FOR_MAC)
-    #include "SysTimer_mach.h"
-#elif defined(SK_BUILD_FOR_UNIX) || defined(SK_BUILD_FOR_ANDROID)
-    #include "SysTimer_posix.h"
-#else
-    #include "SysTimer_c.h"
-#endif
-
-
 SkDebuggerGUI::SkDebuggerGUI(QWidget *parent) :
         QMainWindow(parent)
     , fCentralSplitter(this)
@@ -131,59 +119,9 @@ void SkDebuggerGUI::showDeletes() {
         item->setHidden(fDebugger.isCommandVisible(row) && deletesActivated);
     }
 }
-// This is a simplification of PictureBenchmark's run with the addition of
-// clearing of the times after the first pass (in resetTimes)
-void SkDebuggerGUI::run(const SkPicture* pict,
-                        sk_tools::PictureRenderer* renderer,
-                        int repeats) {
-    SkASSERT(pict);
-    if (nullptr == pict) {
-        return;
-    }
-
-    SkASSERT(renderer != nullptr);
-    if (nullptr == renderer) {
-        return;
-    }
-
-    renderer->init(pict, nullptr, nullptr, nullptr, false, false);
-
-    renderer->setup();
-    renderer->render();
-    renderer->resetState(true);    // flush, swapBuffers and Finish
-
-    for (int i = 0; i < repeats; ++i) {
-        renderer->setup();
-        renderer->render();
-        renderer->resetState(false);  // flush & swapBuffers, but don't Finish
-    }
-    renderer->resetState(true);    // flush, swapBuffers and Finish
-
-    renderer->end();
-}
 
 void SkDebuggerGUI::actionProfile() {

[djsollen, 2015/10/26 13:17:48]

Can we just remove the profiling option altogether if it doesn't work?  No
reason to have a button in the UI that doesn't do anything.

[robertphillips, 2015/10/26 16:01:07]

Done.

-    // In order to profile we pass the command offsets (that were read-in
-    // in loadPicture by the SkOffsetPicture) to an SkTimedPlaybackPicture.
-    // The SkTimedPlaybackPicture in turn passes the offsets to an
-    // SkTimedPicturePlayback object which uses them to track the performance
-    // of individual commands.
-    if (fFileName.isEmpty()) {
-        return;
-    }
-
-    SkFILEStream inputStream;
 
-    inputStream.setPath(fFileName.c_str());
-    if (!inputStream.isValid()) {
-        return;
-    }
-
-    SkAutoTUnref<SkPicture> picture(SkPicture::CreateFromStream(&inputStream,
-                                        &SkImageDecoder::DecodeMemory)); // , fSkipCommands));
-    if (nullptr == picture.get()) {
-        return;
-    }
 }
 
 void SkDebuggerGUI::actionCancel() {