Add fields to timeline.proto (issue1100001)

2 views

Skip to first unread message

dbat...@google.com

unread,

Nov 22, 2013, 11:32:33 AM11/22/13

to mdst...@google.com, page-speed...@googlegroups.com, re...@page-speed-codereview.appspotmail.com

Reviewers: mdsteele,

Please review this at http://page-speed-codereview.appspot.com/1100001/

Affected files:
M pagespeed/proto/timeline.proto

Index: pagespeed/proto/timeline.proto
===================================================================
--- pagespeed/proto/timeline.proto (revision 2561)
+++ pagespeed/proto/timeline.proto (working copy)
@@ -167,6 +167,12 @@

// Marks
optional bool is_main_frame = 33;
+
+ // DecodeImage
+ optional string image_type = 34;
+
+ // ResizeImage
+ optional bool cached = 35;
}

// The type field is required, but we mark it 'optional' because
otherwise

mdst...@google.com

unread,

Nov 22, 2013, 11:44:44 AM11/22/13

to dbat...@google.com, page-speed...@googlegroups.com, re...@page-speed-codereview.appspotmail.com