Simplify code; no need to pass issue arg to GetApprovingReviewers().

Simplify code; no need to pass issue arg to GetApprovingReviewers().

The class already knows the issue number.

R=iannucci@chromium.org
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=213083

[iannucci, 2013-07-23 01:57:11]

lgtm

[commit-bot: I haz the power, 2013-07-23 12:57:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/19976002/1

[commit-bot: I haz the power, 2013-07-23 13:00:43]

Presubmit check for 19976002-1 failed and returned exit status 1.

Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit ERRORS **
tests/checkout_test.py failed
...........EE......
======================================================================
ERROR: testAutoProps (__main__.SvnCheckout)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/checkout_test.py", line 255, in setUp
    self.enabled = self.FAKE_REPOS.set_up_svn()
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
301, in set_up_svn
    self.set_up()
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
211, in set_up
    self.cleanup_dirt()
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
226, in cleanup_dirt
    if not self.tear_down_svn():
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
248, in tear_down_svn
    wait_for_port_to_free(self.host, self.svn_port)
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
161, in wait_for_port_to_free
    assert False, '%d is still bound' % port
AssertionError: 10000 is still bound

======================================================================
ERROR: testException (__main__.SvnCheckout)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/checkout_test.py", line 255, in setUp
    self.enabled = self.FAKE_REPOS.set_up_svn()
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
301, in set_up_svn
    self.set_up()
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
211, in set_up
    self.cleanup_dirt()
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
226, in cleanup_dirt
    if not self.tear_down_svn():
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
248, in tear_down_svn
    wait_for_port_to_free(self.host, self.svn_port)
  File
"/b/commit-queue/workdir/tools/depot_tools/testing_support/fake_repos.py", line
161, in wait_for_port_to_free
    assert False, '%d is still bound' % port
AssertionError: 10000 is still bound

----------------------------------------------------------------------
Ran 19 tests in 98.998s

FAILED (errors=2)


Presubmit checks took 165.7s to calculate.

[commit-bot: I haz the power, 2013-07-23 13:06:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/19976002/1

[commit-bot: I haz the power, 2013-07-23 13:08:49]

Change committed as 213083