 Chromium Code Reviews
 Chromium Code Reviews Issue 1026823002:
  [S.P.] Don't draw frames of animated images that are offscreen  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk
    
  
    Issue 1026823002:
  [S.P.] Don't draw frames of animated images that are offscreen  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk| OLD | NEW | 
|---|---|
| 1 /* | 1 /* | 
| 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org) | 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org) | 
| 3 * (C) 2000 Antti Koivisto (koivisto@kde.org) | 3 * (C) 2000 Antti Koivisto (koivisto@kde.org) | 
| 4 * (C) 2000 Dirk Mueller (mueller@kde.org) | 4 * (C) 2000 Dirk Mueller (mueller@kde.org) | 
| 5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com) | 5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com) | 
| 6 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2012 Apple Inc. All r ights reserved. | 6 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2012 Apple Inc. All r ights reserved. | 
| 7 * Copyright (C) 2009 Google Inc. All rights reserved. | 7 * Copyright (C) 2009 Google Inc. All rights reserved. | 
| 8 * | 8 * | 
| 9 * This library is free software; you can redistribute it and/or | 9 * This library is free software; you can redistribute it and/or | 
| 10 * modify it under the terms of the GNU Library General Public | 10 * modify it under the terms of the GNU Library General Public | 
| (...skipping 232 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 243 #ifndef NDEBUG | 243 #ifndef NDEBUG | 
| 244 if (paintInvalidationStateIsDirty()) { | 244 if (paintInvalidationStateIsDirty()) { | 
| 245 showLayoutTreeForThis(); | 245 showLayoutTreeForThis(); | 
| 246 ASSERT_NOT_REACHED(); | 246 ASSERT_NOT_REACHED(); | 
| 247 } | 247 } | 
| 248 #endif | 248 #endif | 
| 249 } | 249 } | 
| 250 | 250 | 
| 251 void assertSubtreeClearedPaintInvalidationState() const | 251 void assertSubtreeClearedPaintInvalidationState() const | 
| 252 { | 252 { | 
| 253 if (RuntimeEnabledFeatures::slimmingPaintEnabled()) | |
| 
esprehn
2015/03/25 05:06:36
This means bugs can sneak in, if you use a separat
 | |
| 254 return; | |
| 253 for (const LayoutObject* renderer = this; renderer; renderer = renderer- >nextInPreOrder()) | 255 for (const LayoutObject* renderer = this; renderer; renderer = renderer- >nextInPreOrder()) | 
| 254 renderer->assertRendererClearedPaintInvalidationState(); | 256 renderer->assertRendererClearedPaintInvalidationState(); | 
| 255 } | 257 } | 
| 256 | 258 | 
| 257 #endif | 259 #endif | 
| 258 | 260 | 
| 259 // FIXME: This could be used when changing the size of a renderer without ch ildren to skip some invalidations. | 261 // FIXME: This could be used when changing the size of a renderer without ch ildren to skip some invalidations. | 
| 260 // FIXME: This is incorrect for document element. Remove this when we enable slimming paint. | 262 // FIXME: This is incorrect for document element. Remove this when we enable slimming paint. | 
| 261 bool rendererHasNoBoxEffectObsolete() const | 263 bool rendererHasNoBoxEffectObsolete() const | 
| 262 { | 264 { | 
| (...skipping 1398 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1661 void showTree(const blink::LayoutObject*); | 1663 void showTree(const blink::LayoutObject*); | 
| 1662 void showLineTree(const blink::LayoutObject*); | 1664 void showLineTree(const blink::LayoutObject*); | 
| 1663 void showLayoutTree(const blink::LayoutObject* object1); | 1665 void showLayoutTree(const blink::LayoutObject* object1); | 
| 1664 // We don't make object2 an optional parameter so that showLayoutTree | 1666 // We don't make object2 an optional parameter so that showLayoutTree | 
| 1665 // can be called from gdb easily. | 1667 // can be called from gdb easily. | 
| 1666 void showLayoutTree(const blink::LayoutObject* object1, const blink::LayoutObjec t* object2); | 1668 void showLayoutTree(const blink::LayoutObject* object1, const blink::LayoutObjec t* object2); | 
| 1667 | 1669 | 
| 1668 #endif | 1670 #endif | 
| 1669 | 1671 | 
| 1670 #endif // LayoutObject_h | 1672 #endif // LayoutObject_h | 
| OLD | NEW |