PDF Plugin: Swallow Invalidate calls before getting initial size

pdf/paint_manager.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "pdf/paint_manager.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "ppapi/c/pp_errors.h"
@@ skipping 79 matching lines @@
   has_pending_resize_ = true;
   pending_size_ = new_size;
   pending_device_scale_ = device_scale;
 
   view_size_changed_waiting_for_paint_ = true;
 
   Invalidate();
 }
 
 void PaintManager::Invalidate() {
-  // You must call SetSize before using.
-  DCHECK(!graphics_.is_null() || has_pending_resize_);
+  if (graphics_.is_null() && !has_pending_resize_)
+    return;
 
   EnsureCallbackPending();
   aggregator_.InvalidateRect(pp::Rect(GetEffectiveSize()));
 }
 
 void PaintManager::InvalidateRect(const pp::Rect& rect) {
   DCHECK(!in_paint_);
 
-  // You must call SetSize before using.
-  DCHECK(!graphics_.is_null() || has_pending_resize_);
+  if (graphics_.is_null() && !has_pending_resize_)
+    return;
 
   // Clip the rect to the device area.
   pp::Rect clipped_rect = rect.Intersect(pp::Rect(GetEffectiveSize()));
   if (clipped_rect.IsEmpty())
     return;  // Nothing to do.
 
   EnsureCallbackPending();
   aggregator_.InvalidateRect(clipped_rect);
 }
 
 void PaintManager::ScrollRect(const pp::Rect& clip_rect,
                               const pp::Point& amount) {
   DCHECK(!in_paint_);
 
-  // You must call SetSize before using.
-  DCHECK(!graphics_.is_null() || has_pending_resize_);
+  if (graphics_.is_null() && !has_pending_resize_)
+    return;
 
   EnsureCallbackPending();
 
   aggregator_.ScrollRect(clip_rect, amount);
 }
 
 pp::Size PaintManager::GetEffectiveSize() const {
   return has_pending_resize_ ? pending_size_ : plugin_size_;
 }
 
@@ skipping 155 matching lines @@
   DCHECK(manual_callback_pending_);
   manual_callback_pending_ = false;
 
   // Just because we have a manual callback doesn't mean there are actually any
   // invalid regions. Even though we only schedule this callback when something
   // is pending, a Flush callback could have come in before this callback was
   // executed and that could have cleared the queue.
   if (aggregator_.HasPendingUpdate())
     DoPaint();
 }