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

Side by Side Diff: components/update_client/update_response.h

Issue 2252093002: Add support for Omaha cohorts to the component updater. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Don't serialize the attrs when they are empty. Created 4 years, 4 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 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 #ifndef COMPONENTS_UPDATE_CLIENT_UPDATE_RESPONSE_H_ 5 #ifndef COMPONENTS_UPDATE_CLIENT_UPDATE_RESPONSE_H_
6 #define COMPONENTS_UPDATE_CLIENT_UPDATE_RESPONSE_H_ 6 #define COMPONENTS_UPDATE_CLIENT_UPDATE_RESPONSE_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 ~Result(); 92 ~Result();
93 93
94 std::string extension_id; 94 std::string extension_id;
95 95
96 // The list of fallback urls, for full and diff updates respectively. 96 // The list of fallback urls, for full and diff updates respectively.
97 // These urls are base urls; they don't include the filename. 97 // These urls are base urls; they don't include the filename.
98 std::vector<GURL> crx_urls; 98 std::vector<GURL> crx_urls;
99 std::vector<GURL> crx_diffurls; 99 std::vector<GURL> crx_diffurls;
100 100
101 Manifest manifest; 101 Manifest manifest;
102
103 bool set_cohort;
Sorin Jianu 2016/08/18 21:15:15 What is the meaning of the booleans?
Sorin Jianu 2016/08/18 21:15:15 are these initialized anywhere?
waffles 2016/08/18 22:29:23 Done.
waffles 2016/08/18 22:29:23 Done.
104 bool set_cohort_hint;
105 bool set_cohort_name;
106 std::string cohort;
107 std::string cohort_hint;
108 std::string cohort_name;
102 }; 109 };
103 110
104 static const int kNoDaystart = -1; 111 static const int kNoDaystart = -1;
105 struct Results { 112 struct Results {
106 Results(); 113 Results();
107 Results(const Results& other); 114 Results(const Results& other);
108 ~Results(); 115 ~Results();
109 116
110 // This will be >= 0, or kNoDaystart if the <daystart> tag was not present. 117 // This will be >= 0, or kNoDaystart if the <daystart> tag was not present.
111 int daystart_elapsed_seconds; 118 int daystart_elapsed_seconds;
(...skipping 20 matching lines...) Expand all
132 139
133 // Adds parse error details to |errors_| string. 140 // Adds parse error details to |errors_| string.
134 void ParseError(const char* details, ...); 141 void ParseError(const char* details, ...);
135 142
136 DISALLOW_COPY_AND_ASSIGN(UpdateResponse); 143 DISALLOW_COPY_AND_ASSIGN(UpdateResponse);
137 }; 144 };
138 145
139 } // namespace update_client 146 } // namespace update_client
140 147
141 #endif // COMPONENTS_UPDATE_CLIENT_UPDATE_RESPONSE_H_ 148 #endif // COMPONENTS_UPDATE_CLIENT_UPDATE_RESPONSE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698