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

Side by Side Diff: chrome/browser/about_flags.cc

Issue 100453005: Update the location of extract_actions.py in comments. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: change LOG(INFO) to VLOG(1) Created 7 years 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/chromeos/external_metrics.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "chrome/browser/about_flags.h" 5 #include "chrome/browser/about_flags.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <iterator> 8 #include <iterator>
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 308 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 cc::switches::kDisableMapImage, ""} 319 cc::switches::kDisableMapImage, ""}
320 }; 320 };
321 321
322 // RECORDING USER METRICS FOR FLAGS: 322 // RECORDING USER METRICS FOR FLAGS:
323 // ----------------------------------------------------------------------------- 323 // -----------------------------------------------------------------------------
324 // The first line of the experiment is the internal name. If you'd like to 324 // The first line of the experiment is the internal name. If you'd like to
325 // gather statistics about the usage of your flag, you should append a marker 325 // gather statistics about the usage of your flag, you should append a marker
326 // comment to the end of the feature name, like so: 326 // comment to the end of the feature name, like so:
327 // "my-special-feature", // FLAGS:RECORD_UMA 327 // "my-special-feature", // FLAGS:RECORD_UMA
328 // 328 //
329 // After doing that, run //chrome/tools/extract_actions.py (see instructions at 329 // After doing that, run //tools/metrics/actions/extract_actions.py (see
330 // the top of that file for details) to update the chromeactions.txt file, which 330 // instructions at the top of that file for details) to update the
331 // will enable UMA to record your feature flag. 331 // chromeactions.txt file, which will enable UMA to record your feature flag.
332 // 332 //
333 // After your feature has shipped under a flag, you can locate the metrics under 333 // After your feature has shipped under a flag, you can locate the metrics under
334 // the action name AboutFlags_internal-action-name. Actions are recorded once 334 // the action name AboutFlags_internal-action-name. Actions are recorded once
335 // per startup, so you should divide this number by AboutFlags_StartupTick to 335 // per startup, so you should divide this number by AboutFlags_StartupTick to
336 // get a sense of usage. Note that this will not be the same as number of users 336 // get a sense of usage. Note that this will not be the same as number of users
337 // with a given feature enabled because users can quit and relaunch the 337 // with a given feature enabled because users can quit and relaunch the
338 // application multiple times over a given time interval. The dashboard also 338 // application multiple times over a given time interval. The dashboard also
339 // shows you how many (metrics reporting) users have enabled the flag over the 339 // shows you how many (metrics reporting) users have enabled the flag over the
340 // last seven days. However, note that this is not the same as the number of 340 // last seven days. However, note that this is not the same as the number of
341 // users who have the flag enabled, since enabling the flag happens once, 341 // users who have the flag enabled, since enabling the flag happens once,
(...skipping 2010 matching lines...) Expand 10 before | Expand all | Expand 10 after
2352 } 2352 }
2353 2353
2354 const Experiment* GetExperiments(size_t* count) { 2354 const Experiment* GetExperiments(size_t* count) {
2355 *count = num_experiments; 2355 *count = num_experiments;
2356 return experiments; 2356 return experiments;
2357 } 2357 }
2358 2358
2359 } // namespace testing 2359 } // namespace testing
2360 2360
2361 } // namespace about_flags 2361 } // namespace about_flags
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/chromeos/external_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698