Skip to content

Commit

Permalink
fixed about:flags by switching onclick to onchange for select element
Browse files Browse the repository at this point in the history
BUG=107798
TEST=multi-state works on mac in chrome:flags


Review URL: http://codereview.chromium.org/9669036

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126342 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
stephen.searles@gmail.com committed Mar 13, 2012
1 parent 37a0645 commit 6530074
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions AUTHORS
Original file line number Diff line number Diff line change
Expand Up @@ -169,3 +169,4 @@ Yumikiyo Osanai <yumios.art@gmail.com>
Matthew Robertson <matthewrobertson03@gmail.com>
Mao Yujie <yujie.mao@intel.com>
Xinchao He <hexinchao@gmail.com>
Stephen Searles <stephen.searles@gmail.com>
4 changes: 2 additions & 2 deletions chrome/browser/resources/flags.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// Copyright (c) 2011 The Chromium Authors. All rights reserved.
// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

Expand Down Expand Up @@ -47,7 +47,7 @@ function renderTemplate(flagsExperimentsData) {
// Add handlers to dynamically created HTML elements.
var elements = document.getElementsByClassName('experiment-select');
for (var i = 0; i < elements.length; ++i) {
elements[i].onclick = function () {
elements[i].onchange = function () {
handleSelectChoiceExperiment(this, this.selectedIndex);
return false;
};
Expand Down

0 comments on commit 6530074

Please sign in to comment.