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

Unified Diff: third_party/WebKit/Source/platform/graphics/paint/README.md

Issue 2022563002: Remove display item scope (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/graphics/paint/README.md
diff --git a/third_party/WebKit/Source/platform/graphics/paint/README.md b/third_party/WebKit/Source/platform/graphics/paint/README.md
index 28d916d9e3f5d8aaef29c0a7359e4b018d7a2352..0bb62a6874e819ca63f9b6abfebed79040d7a9e2 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/README.md
+++ b/third_party/WebKit/Source/platform/graphics/paint/README.md
@@ -118,11 +118,6 @@ item is identified by an ID consisting of:
* an opaque pointer to the *display item client* that produced it
* a type (from the `DisplayItem::Type` enum)
-* a scope number
-
-*** aside
-TODO(jbroman): Explain scope numbers.
-***
In practice, display item clients are generally subclasses of `LayoutObject`,
but can be other Blink objects which get painted, such as inline boxes and drag
@@ -130,7 +125,8 @@ images.
*** note
It is illegal for there to be two drawings with the same ID in a display item
-list.
+list, except for drawings that are marked uncacheable
+(see [DisplayItemCacheSkipper](DisplayItemCacheSkipper.h)).
***
Generally, clients of this code should use stack-allocated recorder classes to

Powered by Google App Engine
This is Rietveld 408576698