remove drawSprite from canvas

src/core/SkPicturePlayback.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkCanvas.h"
 #include "SkPatchUtils.h"
 #include "SkPictureData.h"
@@ skipping 349 matching lines @@
         case DRAW_RECT: {
             const SkPaint& paint = *fPictureData->getPaint(reader);
             canvas->drawRect(reader->skipT<SkRect>(), paint);
         } break;
         case DRAW_RRECT: {
             const SkPaint& paint = *fPictureData->getPaint(reader);
             SkRRect rrect;
             reader->readRRect(&rrect);
             canvas->drawRRect(rrect, paint);
         } break;
-        case DRAW_SPRITE: {

[mtklein, 2015/12/16 14:06:11]

Does this not break .skp compatibility?  What happens if we read an .skp that
contains a DRAW_SPRITE?

Perhaps we should just comment out the canvas->drawSprite(...) call here, so
that everything keeps parsing?

[reed1, 2015/12/16 14:52:36]

Done.

-            const SkPaint* paint = fPictureData->getPaint(reader);
-            const SkBitmap bitmap = shallow_copy(fPictureData->getBitmap(reader));
-            int left = reader->readInt();
-            int top = reader->readInt();
-            canvas->drawSprite(bitmap, left, top, paint);
-        } break;
         case DRAW_TEXT: {
             const SkPaint& paint = *fPictureData->getPaint(reader);
             TextContainer text;
             get_text(reader, &text);
             SkScalar x = reader->readScalar();
             SkScalar y = reader->readScalar();
             canvas->drawText(text.text(), text.length(), x, y, paint);
         } break;
         case DRAW_TEXT_BLOB: {
             const SkPaint& paint = *fPictureData->getPaint(reader);
@@ skipping 92 matching lines @@
         case TRANSLATE: {
             SkScalar dx = reader->readScalar();
             SkScalar dy = reader->readScalar();
             canvas->translate(dx, dy);
         } break;
         default:
             SkASSERTF(false, "Unknown draw type: %d", op);
     }
 }