diff options
| author | abodenha@chromium.org <abodenha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-14 18:14:57 +0000 |
|---|---|---|
| committer | abodenha@chromium.org <abodenha@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-14 18:14:57 +0000 |
| commit | 606d1080b6d03151a32da36586b4d22b394a8666 (patch) | |
| tree | f7103aa06ebc97102feb9bab565593a58a2839c4 /chrome/common/form_field_data.cc | |
| parent | 3f460adc7484c33bd8d8513a219c5f0d0beac50f (diff) | |
| download | chromium_src-606d1080b6d03151a32da36586b4d22b394a8666.zip chromium_src-606d1080b6d03151a32da36586b4d22b394a8666.tar.gz chromium_src-606d1080b6d03151a32da36586b4d22b394a8666.tar.bz2 | |
Revert 176189
Performance regression re-occurred.
> Add support for radiobuttons and checkboxes.
>
> This is attempt no 2 to submit this CL. earlier attempt at
> https://chromiumcodereview.appspot.com/11415221/ (which was rolled back due to perf regression)
>
> Changes from last attempt:
> * In chrome/browser/autofill/form_structure.cc: Read switch value into local variable before being used multiple times.
> * In chrome/renderer/autofill/form_autofill_util.cc: Extracted form_control_type strings in WebString constants for reuse.
>
> Due to inconsistency in the timing reported by the page cycler tests on my local setup and unavailablity of good environment to perform these tests, the only data I have now is runtimes were approximatly same during *one* set of runs with and without my change. I have observed ~4ms difference in timing without any code changes, while I am chasing ~2 ms.
>
> If perf regresssion is observed again on the perf machines, I will revert it back.
>
>
> BUG=157636
>
> Review URL: https://chromiumcodereview.appspot.com/11788025
TBR=ramankk@chromium.org
Review URL: https://codereview.chromium.org/11884020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176685 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common/form_field_data.cc')
| -rw-r--r-- | chrome/common/form_field_data.cc | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/chrome/common/form_field_data.cc b/chrome/common/form_field_data.cc index 0a2205a..8ff3ce1 100644 --- a/chrome/common/form_field_data.cc +++ b/chrome/common/form_field_data.cc @@ -10,8 +10,6 @@ FormFieldData::FormFieldData() : max_length(0), is_autofilled(false), - is_checked(false), - is_checkable(false), is_focusable(false), should_autocomplete(false) { } @@ -56,10 +54,6 @@ std::ostream& operator<<(std::ostream& os, const FormFieldData& field) { << " " << (field.is_autofilled ? "true" : "false") << " " - << (field.is_checked ? "true" : "false") - << " " - << (field.is_checkable ? "true" : "false") - << " " << (field.is_focusable ? "true" : "false") << " " << (field.should_autocomplete ? "true" : "false"); |