| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
INSTANT_NEW_TAB_PAGE_WITH_OMNIBOX_AS_STARTING_FOCUS, NEW_TAB_PAGE, and OBSOLETE_INSTANT_NEW_TAB_PAGE enums.
This is based the suggestion at https://codereview.chromium.org/23621037/diff/972001/chrome/browser/autocomplete/search_provider.cc#newcode941
BUG=
Review URL: https://codereview.chromium.org/65373002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@233988 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
them.
As described on http://crbug.com/266955#c7
Also adding AtExitManager to gcapi_test.exe; this is required to support MasterPreferences's LazyInstance used by BrowserDistribution, used by google_update's ReadExperimentLabels().
Introducing GCAPITestRegistryOverrider as a class to be added as a member to GCAPI test fixtures that require registry overriding; extracted from the existing GCAPIReactivationTest fixture.
Move Windows-specific variations_util.cc code to experiment_labels_win.cc
BUG=266955
TEST=gcapi_test.exe
Review URL: https://codereview.chromium.org/23579003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@233493 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Renames previous constants to make it clearer they are for Dev/Beta.
BUG=251830
NOTRY=true
R=asvitkine@chromium.org
Review URL: https://codereview.chromium.org/56513005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232848 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/52793002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232067 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add a new X-Chrome-UMA-Log-SHA1 header that is transmitted
along with each UMA log upload. The header contains the SHA1
(as hex) of the payload prior to compression.
The hash is computed at the time that the log is closed, rather than
just before being transmitted - to minimize the chance of hashing
an already corrupted log.
Does not store the hash with logs when they are persisted temporarily
to local state for later upload, since log are already hashed when stored
(and discarded on load if the hash does not match). The SHA1 hash is
re-computed when logs are loaded from local state.
BUG=308066
Review URL: https://codereview.chromium.org/26646003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229594 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=297184
Review URL: https://codereview.chromium.org/25364002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228126 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also updates proto file comments and marks bucket_index as
deprecated.
BUG=288750
TEST=New unit test.
Review URL: https://codereview.chromium.org/25611005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@227115 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=251830
Review URL: https://codereview.chromium.org/25108003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226094 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
logging system.
BUG=77656
R=asvitkine@chromium.org, jochen@chromium.org
Review URL: https://codereview.chromium.org/23995025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225186 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=286554
R=asvitkine
Review URL: https://chromiumcodereview.appspot.com/23902024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222930 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This field was never used for anything and actually takes up a
lot of space in the records.
BUG=288750
Review URL: https://chromiumcodereview.appspot.com/23526035
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222442 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=285440
Review URL: https://chromiumcodereview.appspot.com/23534051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222179 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Moving this code out of chrome/ paves the way for server-controlled system
field trials on ChromeOS. This allows building a new ChromeOS daemon binary
that depends on the new variations target, without needing to depend on all of
chrome/.
This CL moves several files from chrome/common/metrics and
chrome/browser/metrics into a separate component/variations target and
updates all the include paths referencing these. It does not do additional
cleanup, such as changing namespaces - this is left for a follow-up cleanup
CL.
BUG=266007
TEST=Existing unit tests.
R=joi@chromium.org, jwd@chromium.org, sky@chromium.org
Review URL: https://codereview.chromium.org/23097007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@220339 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/23587005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219941 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Followed up previous code review; see linked bug.
Once you approve of these comments, I'll revise them in the other
two places (google-internal protocol file and autocomplete_input.h).
BUG=264938
Review URL: https://chromiumcodereview.appspot.com/22912009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@219926 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
R=asvitkine@chromium.org
Review URL: https://codereview.chromium.org/22303005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218850 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This way, variation_ids.h can stay under chrome/ when a subset of this
code will move to a new target that will also be used by the ChromeOS
daemon process.
BUG=266007
TEST=Existing unit tests.
Review URL: https://chromiumcodereview.appspot.com/22980009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218102 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This experiment, which runs as part of the bundled omnibox field trial,
uses the field trial parameters to demote results of particular types
(e.g., HISTORY_TITLE results), possibly depending on context (e.g.,
search results page doing search term replacement).
It also improves the testing framework for field trial parameters,
allowing them to be cleared between consecutive TEST_F()s.
Tested via unit tests and by setting up my local variations server with
some parameters and making sure results can be demoted or
omitted.
BUG=264066
Review URL: https://chromiumcodereview.appspot.com/22031002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216633 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
R=asvitkine@chromium.org
BUG=237218
Review URL: https://chromiumcodereview.appspot.com/22481004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216576 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
emitted only for Instant search results pages. Now it should be emitted for any search results page.
These fixes include:
- Pull out checking for search results page to its own function, IsSearchResultsPageFromDefaultSearchProvider(url) on the TemplateURLService
- Add IsSearchResultsPage() to OmniboxCurrentPageDelegate to make checking within omnibox code easier
- Add a test for TempalteURL::IsSearchURL which is the heart of the above functions
- Move checking for search results pages from metrics_service.cc to uma_browsing_activity_observer.cc which is a more logical location.
- Listen to notification for NAV_ENTRY_COMMITTED rather than LOAD_STARTED to capture more events. This includes adding a control RecordAction for NAV_ENTRY_COMMITTED so we can differentiate SRP from non-SRP.
Incorporating some of the changes from https://codereview.chromium.org/20043003/ which utilize this functionality. These changes include:
- Add a page classification category for SEARCH_RESULT_PAGE_NO_SEARCH_TERM_REPLACEMENT
- Update the ClassifyPage function using the changes above to determine if search replacement is being done.
- Update the omnibox_event.proto to account for the additional category as well.
Also updates the name of the metrics since the definitions have changed.
BUG=263644,264067
Review URL: https://chromiumcodereview.appspot.com/21395002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216469 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Keep track of whether omnibox was visibly or invisibly focused when user first
started typing and record that state in OmniboxLog.
BUG=264069
R=isherman@chromium.org, mpearson@chromium.org, pkasting@chromium.org
Review URL: https://codereview.chromium.org/20587003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216067 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
the verbatim suggestion when it is at position 0.
Also flips ShouldPreferRemoteNTPOnStartup to true by default, and modifies the pseudo-flags defined in the field trial names of InstantExtended field trials to be able to be queried successfully even when overall InstantExtended is disabled. (This is necessary to allow an experiment for suppressing verbatim outside of InstantExtended.)
BUG=252823
Review URL: https://chromiumcodereview.appspot.com/17114003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215762 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This ensures the trial is reported in metrics.
BUG=none
Review URL: https://chromiumcodereview.appspot.com/21959007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215670 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Moves id and param association to variations_associated_data.cc,
while keeping the rest of the functionality in variations_util.cc.
No functional changes.
BUG=266007
TEST=Existing unit tests.
Review URL: https://chromiumcodereview.appspot.com/21401002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215644 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reland of https://codereview.chromium.org/21127007/
The original CL changed a std::string to a const char* in
a test, which resulted in saving a ref to a temporary that
was later deref'd, which is now fixed in this CL.
BUG=266007
TEST=Existing unit tests.
TBR=thakis@chromium.org,isherman@chromium.org
NOTRY=true
Review URL: https://chromiumcodereview.appspot.com/21893002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215291 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> Move CachingPermutedEntropyProvider to its own file.
>
> No functional changes.
>
> BUG=266007
> TEST=Existing unit tests.
>
> Review URL: https://chromiumcodereview.appspot.com/21127007
TBR=asvitkine@chromium.org
Review URL: https://codereview.chromium.org/21542004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215112 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
No functional changes.
BUG=266007
TEST=Existing unit tests.
Review URL: https://chromiumcodereview.appspot.com/21127007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@215106 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a pre-cursor to moving CachingPermutedEntropyProvider
to a separate file. Done in a separate CL to make it easier
to review.
BUG=266007
TEST=Existing unit tests.
Review URL: https://chromiumcodereview.appspot.com/21248003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214708 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Everyone in this field trial is always in group_01. It's useful in field trial based
dashboard views if you actually just want to look at all data.
R=rkaplow@chromium.org
BUG=None
Review URL: https://chromiumcodereview.appspot.com/19185005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214444 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changes the Field Trial API to require users to pass ONE_TIME_RANDOMIZED
or SESSION_RANDOMIZED at the time the field trial is created.
This removes a source of human errors, where developers would sometimes
previously forget to call UseOneTimeRandomization() in their CLs.
It's a small startup performance win, since one time randomized trials will
no longer call RandDouble() that will be then replaced by a value from the
entropy provider. (This was showing up in profiles when I was measuring
metrics startup perf on mobile.)
BUG=262971
TEST=Existing unit tests.
TBR=isherman@chromium.org, thakis@chromium.org
Review URL: https://codereview.chromium.org/20777005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214352 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
Review URL: https://chromiumcodereview.appspot.com/19751005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213537 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
Review URL: https://chromiumcodereview.appspot.com/19497006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213246 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
More precisely, reduce whether the user was on a search result page
doing search term replacement.
This has been approved by chrome-privacy.
Tested by adding a LOG(INFO) line in metrics_log to see what the page
classification emitted was.
BUG=251484
Review URL: https://chromiumcodereview.appspot.com/19810002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212895 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently the Instant Extended new tab page is classified as OTHER
because it doesn't have the URL chrome://newtab. This adds a new
way classification for the instant new tab page.
BUG=
Review URL: https://chromiumcodereview.appspot.com/19019008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212491 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This is particularly useful to help determine which CPUs fail to tick
monotonically in HighResNow() on Windows.
BUG=254211
Review URL: https://chromiumcodereview.appspot.com/17770005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212323 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=109818
TEST=none
R=jar@chromium.org
Review URL: https://chromiumcodereview.appspot.com/19707002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212319 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=109818
TEST=none
R=jar@chromium.org
Review URL: https://chromiumcodereview.appspot.com/19684005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212309 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
chrome/common, chrome/nacl/, chrome/renderer/, chrome/service/.
BUG=260807
TEST=none
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/19579004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212174 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=109818
TEST=none
Review URL: https://chromiumcodereview.appspot.com/19278006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212051 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Reduce granularity of uma_enabled_date to 1 hour.
BUG=254551
Review URL: https://chromiumcodereview.appspot.com/18503011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211190 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, some variations have abused group names to pass
params anyway. This change adds first class support for this.
BUG=257088
TEST=New unit tests.
Review URL: https://chromiumcodereview.appspot.com/18178010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210586 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=229660
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://chromiumcodereview.appspot.com/18523004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210197 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
chrome/renderer/, chrome/service/.
BUG=254986
TEST=none
TBR=thestig@chromium.org
Review URL: https://codereview.chromium.org/18147002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209157 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This maps much better to percentages people actually use for
experiments (e.g. 1%) without causing an uneven distribution
and thus a source of non-uniformity.
Re-generates the low entropy source values for users who have
it outside the new range.
BUG=252283
Review URL: https://chromiumcodereview.appspot.com/17499002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207865 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=180475
Review URL: https://chromiumcodereview.appspot.com/16834003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206208 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
Review URL: https://chromiumcodereview.appspot.com/16821005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206063 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=247723
TEST=none
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/16579006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205372 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
chrome/common/, chrome/installer.
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/16026013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204902 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Taken from:
https://gerrit.chromium.org/gerrit/48671
https://gerrit.chromium.org/gerrit/48673
BUG=238327
TEST=build successfully
Change-Id: I6a64d49658e6ab26619688ab50c2beaed66336bc
Signed-off-by: Simon Que <sque@chromium.org>
Review URL: https://chromiumcodereview.appspot.com/14957007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204489 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> removed unused variation ids
>
> BUG=
>
> Review URL: https://chromiumcodereview.appspot.com/14232021
TBR=harrym@chromium.org
Review URL: https://codereview.chromium.org/14566023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@200389 0039d316-1c4b-4281-b951-d872f2087c98
|