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

Unified Diff: chrome/browser/chromeos/login/screens/screen_context.h

Issue 328763003: Added usefull utility methods to Screen and ScreenContext. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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: chrome/browser/chromeos/login/screens/screen_context.h
diff --git a/chrome/browser/chromeos/login/screens/screen_context.h b/chrome/browser/chromeos/login/screens/screen_context.h
index c0dc4460c8331e4ea853977390c3d4f656221556..4878f89824c835c324e6f5e270166008c4b47eb2 100644
--- a/chrome/browser/chromeos/login/screens/screen_context.h
+++ b/chrome/browser/chromeos/login/screens/screen_context.h
@@ -71,7 +71,7 @@ class ScreenContext : public base::NonThreadSafe {
void GetChangesAndReset(base::DictionaryValue* diff);
// Applies changes from |diff| to the context. All keys from |diff|
- // are stored in |keys|.
+ // are stored in |keys|. |keys| is optional and can be NULL.
Denis Kuznetsov (DE-MUC) 2014/06/11 14:29:17 "|Keys| are" ?
dzhioev (left Google) 2014/06/18 14:43:39 Rephrased to "|keys| argument is optional".
Denis Kuznetsov (DE-MUC) 2014/06/19 19:16:36 Nit : capitalize to "Keys"
dzhioev (left Google) 2014/06/20 07:24:16 It is a name of argument, I don't think we should
void ApplyChanges(const base::DictionaryValue& diff,
std::vector<std::string>* keys);

Powered by Google App Engine
This is Rietveld 408576698