[keys] Cleanup: removing unused parameters in elements.cc and keys.cc

[keys] Cleanup: removing unused parameters in elements.cc and keys.cc

- Cleanup CL to remove unused parameters in elements.cc. Additionally use fields
  from the KeyAccumulator wherever possible.
- Make method names in KeyAccumulator more consistent

BUG=

Committed: https://crrev.com/e801d554aa30b034b191246fb02a1856165899e3
Cr-Commit-Position: refs/heads/master@{#36073}

[Camillo Bruni, 2016-05-03 13:14:39]

Description was changed from

==========
[keys] removing unused parameters

Cleanup CL to remove unused parameters in elements.cc. Additionally use fields
from the KeyAccumulator wherever possible.

BUG=
==========

to

==========
[keys] removing unused parameters

- Cleanup CL to remove unused parameters in elements.cc. Additionally use fields
  from the KeyAccumulator wherever possible. 
- Make method names in KeyAccumulator more consistent

BUG=
==========

[Camillo Bruni, 2016-05-03 13:15:04]

Description was changed from

==========
[keys] removing unused parameters

- Cleanup CL to remove unused parameters in elements.cc. Additionally use fields
  from the KeyAccumulator wherever possible. 
- Make method names in KeyAccumulator more consistent

BUG=
==========

to

==========
[keys] Cleanup: removing unused parameters in elements.cc and keys.cc

- Cleanup CL to remove unused parameters in elements.cc. Additionally use fields
  from the KeyAccumulator wherever possible. 
- Make method names in KeyAccumulator more consistent

BUG=
==========

[Camillo Bruni, 2016-05-04 14:30:35]

The CQ bit was checked by cbruni@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-04 14:30:40]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944703002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944703002/40001

[commit-bot: I haz the power, 2016-05-04 14:32:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-04 14:32:58]

Dry run: Try jobs failed on following builders:
  v8_android_arm_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux64_avx2_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/5282)
  v8_linux_arm64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_arm_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_gcc_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_gcc_compile_rel/bu...)
  v8_linux_mips64el_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_nodcheck_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_linux_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/5271)
  v8_mac_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/1168)

[Camillo Bruni, 2016-05-06 09:24:23]

The CQ bit was checked by cbruni@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-06 09:24:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944703002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944703002/100001

[commit-bot: I haz the power, 2016-05-06 09:53:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-06 09:53:58]

Dry run: This issue passed the CQ dry run.

[Camillo Bruni, 2016-05-06 09:54:53]

cbruni@chromium.org changed reviewers:
+ ishell@chromium.org

[Camillo Bruni, 2016-05-06 09:54:54]

PTAL

[Igor Sheludko, 2016-05-06 10:09:40]

lgtm

[Camillo Bruni, 2016-05-06 10:48:37]

The CQ bit was checked by cbruni@chromium.org

[commit-bot: I haz the power, 2016-05-06 10:48:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1944703002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1944703002/100001

[commit-bot: I haz the power, 2016-05-06 10:50:47]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-05-06 10:51:46]

Description was changed from

==========
[keys] Cleanup: removing unused parameters in elements.cc and keys.cc

- Cleanup CL to remove unused parameters in elements.cc. Additionally use fields
  from the KeyAccumulator wherever possible. 
- Make method names in KeyAccumulator more consistent

BUG=
==========

to

==========
[keys] Cleanup: removing unused parameters in elements.cc and keys.cc

- Cleanup CL to remove unused parameters in elements.cc. Additionally use fields
  from the KeyAccumulator wherever possible.
- Make method names in KeyAccumulator more consistent

BUG=

Committed: https://crrev.com/e801d554aa30b034b191246fb02a1856165899e3
Cr-Commit-Position: refs/heads/master@{#36073}
==========

[commit-bot: I haz the power, 2016-05-06 10:51:47]

Patchset 6 (id:??) landed as
https://crrev.com/e801d554aa30b034b191246fb02a1856165899e3
Cr-Commit-Position: refs/heads/master@{#36073}