Delete PrintContext::spoolAllPagesWithBoundaries

Source/web/WebLocalFrameImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 284 matching lines @@
 
     virtual ~ChromePrintContext() { }
 
     virtual void begin(float width, float height)
     {
         ASSERT(!m_printedPageWidth);
         m_printedPageWidth = width;
         PrintContext::begin(m_printedPageWidth, height);
     }
 
-    virtual void end()
-    {
-        PrintContext::end();

[Vitaly Buka (NO REVIEWS), 2014/08/13 19:37:07]

it's incorrect to remove this calls
base class has no virtual

Example:
ChromePrintContext* context = new ChromePluginPrintContext()
context->end()  // PrintContext::end() is called, not
ChromePluginPrintContext::end()

-    }
-
     virtual float getPageShrink(int pageNumber) const
     {
         IntRect pageRect = m_pageRects[pageNumber];
         return m_printedPageWidth / pageRect.width();
     }
 
     // Spools the printed page, a subrect of frame(). Skip the scale step.
     // NativeTheme doesn't play well with scaling. Scaling is done browser side
     // instead. Returns the scale to be applied.
     // On Linux, we don't have the problem with NativeTheme, hence we let WebKit
@@ skipping 54 matching lines @@
             float scale = getPageShrink(pageIndex);
             graphicsContext.scale(scale, scale);
 #endif
             spoolPage(graphicsContext, pageIndex);
             graphicsContext.restore();
 
             currentHeight += pageSizeInPixels.height() + 1;
         }
     }
 
-    virtual void computePageRects(const FloatRect& printRect, float headerHeight, float footerHeight, float userScaleFactor, float& outPageHeight)
-    {

[Vitaly Buka (NO REVIEWS), 2014/08/13 19:37:07]

same

-        PrintContext::computePageRects(printRect, headerHeight, footerHeight, userScaleFactor, outPageHeight);
-    }
-
-    virtual int pageCount() const
-    {
-        return PrintContext::pageCount();
-    }
-
 private:
     // Set when printing.
     float m_printedPageWidth;
 };
 
 // Simple class to override some of PrintContext behavior. This is used when
 // the frame hosts a plugin that supports custom printing. In this case, we
 // want to delegate all printing related calls to the plugin.
 class ChromePluginPrintContext FINAL : public ChromePrintContext {
 public:
@@ skipping 1471 matching lines @@
 
 void WebLocalFrameImpl::invalidateAll() const
 {
     ASSERT(frame() && frame()->view());
     FrameView* view = frame()->view();
     view->invalidateRect(view->frameRect());
     invalidateScrollbar();
 }
 
 } // namespace blink