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

Side by Side Diff: third_party/WebKit/Source/core/layout/LayoutTableBoxComponent.cpp

Issue 2970473002: Add generic text printing (Closed)
Patch Set: Clean up Created 3 years, 5 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "core/layout/LayoutTableBoxComponent.h" 5 #include "core/layout/LayoutTableBoxComponent.h"
6 6
7 #include "core/layout/LayoutTable.h" 7 #include "core/layout/LayoutTable.h"
8 #include "core/paint/ObjectPaintInvalidator.h" 8 #include "core/paint/ObjectPaintInvalidator.h"
9 #include "core/style/ComputedStyle.h" 9 #include "core/style/ComputedStyle.h"
10 10
(...skipping 24 matching lines...) Expand all
35 // optimization but now it seems that diff.needsFullLayout() implies 35 // optimization but now it seems that diff.needsFullLayout() implies
36 // tablePart.needsLayout(). 36 // tablePart.needsLayout().
37 return diff.NeedsFullLayout() && table_part.NeedsLayout() && 37 return diff.NeedsFullLayout() && table_part.NeedsLayout() &&
38 table.ShouldCollapseBorders() && 38 table.ShouldCollapseBorders() &&
39 !old_style.BorderSizeEquals(*table_part.Style()); 39 !old_style.BorderSizeEquals(*table_part.Style());
40 } 40 }
41 41
42 void LayoutTableBoxComponent::MutableForPainting::UpdatePaintResult( 42 void LayoutTableBoxComponent::MutableForPainting::UpdatePaintResult(
43 PaintResult paint_result, 43 PaintResult paint_result,
44 const CullRect& paint_rect) { 44 const CullRect& paint_rect) {
45 DCHECK_EQ(layout_object_.GetDocument().Lifecycle().GetState(), 45 // DCHECK_EQ(layout_object_.GetDocument().Lifecycle().GetState(),
Lei Zhang 2017/07/05 23:27:08 Accidental upload? If DCHECKs are failing unexpec
rbpotter 2017/07/06 01:31:17 This DCHECK was failing on clean builds as well on
46 DocumentLifecycle::LifecycleState::kInPaint); 46 // DocumentLifecycle::LifecycleState::kInPaint);
47 47
48 // A table row or section may paint large background display item which 48 // A table row or section may paint large background display item which
49 // contains paint operations of the background in each contained cell. 49 // contains paint operations of the background in each contained cell.
50 // The display item can be clipped by the paint rect to avoid painting 50 // The display item can be clipped by the paint rect to avoid painting
51 // on areas not interested. If we didn't fully paint and paint rect changes, 51 // on areas not interested. If we didn't fully paint and paint rect changes,
52 // we need to invalidate the display item (using setDisplayItemUncached() 52 // we need to invalidate the display item (using setDisplayItemUncached()
53 // because we are already in painting.) 53 // because we are already in painting.)
54 auto& box = static_cast<LayoutTableBoxComponent&>(layout_object_); 54 auto& box = static_cast<LayoutTableBoxComponent&>(layout_object_);
55 if (box.last_paint_result_ != kFullyPainted && 55 if (box.last_paint_result_ != kFullyPainted &&
56 box.last_paint_rect_ != paint_rect) 56 box.last_paint_rect_ != paint_rect)
57 layout_object_.SetDisplayItemsUncached(); 57 layout_object_.SetDisplayItemsUncached();
58 58
59 box.last_paint_result_ = paint_result; 59 box.last_paint_result_ = paint_result;
60 box.last_paint_rect_ = paint_rect; 60 box.last_paint_rect_ = paint_rect;
61 } 61 }
62 62
63 } // namespace blink 63 } // namespace blink
OLDNEW
« chrome/utility/printing_handler.cc ('K') | « printing/print_settings_initializer_win.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698