Revert of De-virtualize SkCanvas save/restore.

Revert of De-virtualize SkCanvas save/restore. (https://codereview.chromium.org/194713008/)

Reason for revert:
roll fix

Original issue's description:
> De-virtualize SkCanvas save/restore.
> 
> This moves the state management logic into non-virtual SkCanvas methods,
> and turns the virtuals into protected notifiers.
> 
> R=robertphillips@google.com, reed@google.com
> 
> Committed: https://code.google.com/p/skia/source/detail?r=13747

TBR=reed@google.com,robertphillips@google.com,fmalita@chromium.org
NOTREECHECKS=true
NOTRY=true

[hal.canary, 2014-03-11 21:45:07]

Created Revert of De-virtualize SkCanvas save/restore.

[commit-bot: I haz the power, 2014-03-11 21:45:11]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/halcanary@google.com/195843002/1

[commit-bot: I haz the power, 2014-03-11 21:45:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-11 21:45:40]

Failed to apply patch for src/core/SkCanvas.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file src/core/SkCanvas.cpp
  Hunk #5 FAILED at 890.
  Hunk #6 succeeded at 948 (offset 2 lines).
  1 out of 6 hunks FAILED -- saving rejects to file src/core/SkCanvas.cpp.rej

Patch:       src/core/SkCanvas.cpp
Index: src/core/SkCanvas.cpp
diff --git a/src/core/SkCanvas.cpp b/src/core/SkCanvas.cpp
index
85ef7d687699ed039823766aff3ea6262eaf9e53..0888a01fa459615322717b5896d68bed5e0dc229
100644
--- a/src/core/SkCanvas.cpp
+++ b/src/core/SkCanvas.cpp
@@ -348,7 +348,7 @@
             SkPaint tmp;
             tmp.setImageFilter(fOrigPaint.getImageFilter());
             (void)canvas->internalSaveLayer(bounds, &tmp,
-                                    SkCanvas::kARGB_ClipLayer_SaveFlag, true,
false);
+                                    SkCanvas::kARGB_ClipLayer_SaveFlag, true);
             // we'll clear the imageFilter for the actual draws in next(), so
             // it will only be applied during the restore().
             fDoClearImageFilter = true;
@@ -806,12 +806,7 @@
     return saveCount;
 }
 
-void SkCanvas::onSave(SaveFlags) {
-    // Do nothing. Subclasses may do something.
-}
-
 int SkCanvas::save(SaveFlags flags) {
-    this->onSave(flags);
     // call shared impl
     return this->internalSave(flags);
 }
@@ -868,16 +863,9 @@
     return true;
 }
 
-bool SkCanvas::onSaveLayer(const SkRect*, const SkPaint*, SaveFlags) {
-    // Do nothing. Subclasses may do something.
-    return true;
-}
-
 int SkCanvas::saveLayer(const SkRect* bounds, const SkPaint* paint,
                         SaveFlags flags) {
-    // Overriding classes may return false to signal that we don't need to
create a layer.
-    bool skipLayer = !this->onSaveLayer(bounds, paint, flags);
-    return this->internalSaveLayer(bounds, paint, flags, false, skipLayer);
+    return this->internalSaveLayer(bounds, paint, flags, false);
 }
 
 static SkBaseDevice* createCompatibleDevice(SkCanvas* canvas,
@@ -887,7 +875,7 @@
 }
 
 int SkCanvas::internalSaveLayer(const SkRect* bounds, const SkPaint* paint,
-                                SaveFlags flags, bool justForImageFilter, bool
skipLayer) {
+                                SaveFlags flags, bool justForImageFilter) {
 #ifndef SK_SUPPORT_LEGACY_CLIPTOLAYERFLAG
     flags = (SaveFlags)(flags | kClipToLayer_SaveFlag);
 #endif
@@ -902,9 +890,6 @@
     if (!this->clipRectBounds(bounds, flags, &ir, paint ?
paint->getImageFilter() : NULL)) {
         return count;
     }
-
-    if (skipLayer)
-        return count;
 
     // Kill the imagefilter if our device doesn't allow it
     SkLazyPaint lazyP;
@@ -958,14 +943,9 @@
     }
 }
 
-void SkCanvas::onRestore() {
-    // Do nothing. Subclasses may do something.
-}
-
 void SkCanvas::restore() {
     // check for underflow
     if (fMCStack.count() > 1) {
-        this->onRestore();
         this->internalRestore();
     }
 }