Get debugger compiling again

debugger/QT/SkDebuggerGUI.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkDebuggerGUI.h"
-#include "PictureRenderer.h"
 #include "SkPictureData.h"
 #include "SkPicturePlayback.h"
 #include "SkPictureRecord.h"
 #include <QListWidgetItem>
 #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)
     , fStatusBar(this)
     , fToolBar(this)
     , fActionOpen(this)
     , fActionBreakpoint(this)
     , fActionProfile(this)
     , fActionCancel(this)
     , fActionClearBreakpoints(this)
@@ skipping 86 matching lines @@
     }
 }
 
 void SkDebuggerGUI::showDeletes() {
     bool deletesActivated = fActionShowDeletes.isChecked();
     for (int row = 0; row < fListWidget.count(); row++) {
         QListWidgetItem *item = fListWidget.item(row);
         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() {
     for (int row = 0; row < fListWidget.count(); row++) {
         fListWidget.item(row)->setHidden(false);
     }
 }
 
 void SkDebuggerGUI::actionClearBreakpoints() {
     for (int row = 0; row < fListWidget.count(); row++) {
@@ skipping 653 matching lines @@
         fCanvasWidget.drawTo(fPausedRow);
     } else {
         fCanvasWidget.drawTo(fListWidget.currentRow());
     }
 }
 
 void SkDebuggerGUI::updateHit(int newHit) {
     fCommandHitBox.setText(QString::number(newHit));
 }