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

Unified Diff: tools/ipc_fuzzer/fuzzer/fuzzer.cc

Issue 2449853004: Getting rid of DelegatedFrameData (Closed)
Patch Set: nit Created 4 years, 1 month 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: tools/ipc_fuzzer/fuzzer/fuzzer.cc
diff --git a/tools/ipc_fuzzer/fuzzer/fuzzer.cc b/tools/ipc_fuzzer/fuzzer/fuzzer.cc
index 643cda501298f2b910f1531c771197e0b26e52ef..d7ebc4f1b6db03fa52c5f1c091754a584d241c7f 100644
--- a/tools/ipc_fuzzer/fuzzer/fuzzer.cc
+++ b/tools/ipc_fuzzer/fuzzer/fuzzer.cc
@@ -683,8 +683,9 @@ struct FuzzTraits<cc::CompositorFrame> {
switch (RandInRange(2)) {
case 0: {
- p->delegated_frame_data.reset(new cc::DelegatedFrameData());
- if (!FuzzParam(p->delegated_frame_data.get(), fuzzer))
+ if (!FuzzParam(&p->resource_list, fuzzer))
+ return false;
+ if (!FuzzParam(&p->render_pass_list, fuzzer))
return false;
return true;
}
@@ -695,17 +696,6 @@ struct FuzzTraits<cc::CompositorFrame> {
}
};
-template <>
-struct FuzzTraits<cc::DelegatedFrameData> {
- static bool Fuzz(cc::DelegatedFrameData* p, Fuzzer* fuzzer) {
- if (!FuzzParam(&p->resource_list, fuzzer))
- return false;
- if (!FuzzParam(&p->render_pass_list, fuzzer))
- return false;
- return true;
- }
-};
-
template <class A>
struct FuzzTraits<cc::ListContainer<A>> {
static bool Fuzz(cc::ListContainer<A>* p, Fuzzer* fuzzer) {

Powered by Google App Engine
This is Rietveld 408576698