Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Side by Side Diff: third_party/WebKit/Source/platform/graphics/paint/PaintController.cpp

Issue 2820233002: Always report maybe-clipped when painting fragmented content (Closed)
Patch Set: none Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « third_party/WebKit/Source/core/paint/PaintLayerPainter.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "platform/graphics/paint/PaintController.h" 5 #include "platform/graphics/paint/PaintController.h"
6 6
7 #include "platform/graphics/GraphicsLayer.h" 7 #include "platform/graphics/GraphicsLayer.h"
8 #include "platform/graphics/paint/DrawingDisplayItem.h" 8 #include "platform/graphics/paint/DrawingDisplayItem.h"
9 #include "platform/instrumentation/tracing/TraceEvent.h" 9 #include "platform/instrumentation/tracing/TraceEvent.h"
10 #include "platform/wtf/AutoReset.h" 10 #include "platform/wtf/AutoReset.h"
(...skipping 434 matching lines...) Expand 10 before | Expand all | Expand 10 after
445 AutoReset<size_t> subsequence_begin_index( 445 AutoReset<size_t> subsequence_begin_index(
446 &current_cached_subsequence_begin_index_in_new_list_, 446 &current_cached_subsequence_begin_index_in_new_list_,
447 new_display_item_list_.size()); 447 new_display_item_list_.size());
448 DisplayItem* cached_item = 448 DisplayItem* cached_item =
449 &current_paint_artifact_.GetDisplayItemList()[begin_index]; 449 &current_paint_artifact_.GetDisplayItemList()[begin_index];
450 450
451 if (RuntimeEnabledFeatures::paintUnderInvalidationCheckingEnabled()) { 451 if (RuntimeEnabledFeatures::paintUnderInvalidationCheckingEnabled()) {
452 DCHECK(!IsCheckingUnderInvalidation()); 452 DCHECK(!IsCheckingUnderInvalidation());
453 under_invalidation_checking_begin_ = begin_index; 453 under_invalidation_checking_begin_ = begin_index;
454 under_invalidation_message_prefix_ = 454 under_invalidation_message_prefix_ =
455 "(In cached subsequence of " + cached_item->Client().DebugName() + ")"; 455 "(In cached subsequence starting with " +
456 cached_item->Client().DebugName() + ")";
456 } 457 }
457 458
458 Vector<PaintChunk>::const_iterator cached_chunk; 459 Vector<PaintChunk>::const_iterator cached_chunk;
459 if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) { 460 if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
460 cached_chunk = 461 cached_chunk =
461 current_paint_artifact_.FindChunkByDisplayItemIndex(begin_index); 462 current_paint_artifact_.FindChunkByDisplayItemIndex(begin_index);
462 DCHECK(cached_chunk != current_paint_artifact_.PaintChunks().end()); 463 DCHECK(cached_chunk != current_paint_artifact_.PaintChunks().end());
463 UpdateCurrentPaintChunkProperties( 464 UpdateCurrentPaintChunkProperties(
464 cached_chunk->id ? &*cached_chunk->id : nullptr, 465 cached_chunk->id ? &*cached_chunk->id : nullptr,
465 cached_chunk->properties); 466 cached_chunk->properties);
(...skipping 466 matching lines...) Expand 10 before | Expand all | Expand 10 after
932 show_paint_records 933 show_paint_records
933 ? (DisplayItemList::JsonOptions::kShowPaintRecords | 934 ? (DisplayItemList::JsonOptions::kShowPaintRecords |
934 DisplayItemList::JsonOptions::kShowClientDebugName) 935 DisplayItemList::JsonOptions::kShowClientDebugName)
935 : DisplayItemList::JsonOptions::kShowClientDebugName) 936 : DisplayItemList::JsonOptions::kShowClientDebugName)
936 ->ToPrettyJSONString() 937 ->ToPrettyJSONString()
937 .Utf8() 938 .Utf8()
938 .Data()); 939 .Data());
939 } 940 }
940 941
941 } // namespace blink 942 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/paint/PaintLayerPainter.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698