Remove all unused variables.

Remove all unused variables.

Unused variables were found via pylint.

There are a few types of circumstances where we have unused variables,
so there are a few types of transformations:

 1. variables that are simply unused, e.g.
    a = f()  =>  <delete>

 2. functions that return tuples but only some arguments are used, e.g.
    a, b = f()  =>  a, _ = f()

 3. functions that have side effects and also a return value, e.g.
    a = do_something()  =>  do_something()

 4. exception variables that are unused
    except Exception as e:  =>  except Exception:

Committed: https://crrev.com/69af2dee0ff02ff987b485e1ba297d94c8285427
Cr-Commit-Position: refs/heads/master@{#406887}

[qyearsley, 2016-07-11 16:41:29]

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

[qyearsley, 2016-07-11 16:41:44]

qyearsley@chromium.org changed reviewers:
+ dcheng@chromium.org, dpranke@chromium.org

[commit-bot: I haz the power, 2016-07-11 16:42:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[qyearsley, 2016-07-11 16:43:00]

Description was changed from

==========
Remove all unused variables.

Unused variables were found via pylint.

There are 3 types of circumstances where we have unused variables,
so there are 3 types of transformations:

 1. functions that return tuples but only some arguments are used, e.g.
    a, b = f()  =>  a, _ = f()

 2. functions that have side effects and also a return value, e.g.
    a = do_something()  =>  do_something()

 3. variables that are simply unused, e.g.
    a = f()  =>  <delete>
==========

to

==========
Remove all unused variables.

Unused variables were found via pylint.

There are a few types of circumstances where we have unused variables,
so there are a few types of transformations:

 1. variables that are simply unused, e.g.
    a = f()  =>  <delete>

 2. functions that return tuples but only some arguments are used, e.g.
    a, b = f()  =>  a, _ = f()

 3. functions that have side effects and also a return value, e.g.
    a = do_something()  =>  do_something()

 4. exception variables that are unused
    except Exception as e:  =>  except Exception:
==========

[commit-bot: I haz the power, 2016-07-11 17:57:27]

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

[commit-bot: I haz the power, 2016-07-11 17:57:28]

Dry run: This issue passed the CQ dry run.

[dcheng, 2016-07-12 03:22:38]

non owner LGTM

[qyearsley, 2016-07-20 20:43:56]

On 2016/07/12 at 03:22:38, dcheng wrote:
> non owner LGTM

Dirk, there are a few CLs that unnecessary code (unused variables, a small bit
of dead code, and pass statements) which dcheng@ has stamped -- would you be OK
with me submitting these?

[Dirk Pranke, 2016-07-20 23:40:16]

On 2016/07/20 20:43:56, qyearsley wrote:
> On 2016/07/12 at 03:22:38, dcheng wrote:
> > non owner LGTM
> 
> Dirk, there are a few CLs that unnecessary code (unused variables, a small bit
> of dead code, and pass statements) which dcheng@ has stamped -- would you be
OK
> with me submitting these?

Yes. Technically everyone is an owner of stuff under
//third_party/WebKit/Tools/Scripts :).

I'd like to review stuff when I have time, but don't block on me unless you
really need my input
(and, if you do, make sure you are clear about it when you make me a reviewer so
I don't
accidentally ignore you).

[Dirk Pranke, 2016-07-20 23:45:03]

lgtm

[qyearsley, 2016-07-21 16:05:14]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-07-21 16:05:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-21 17:30:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-21 17:33:30]

Description was changed from

==========
Remove all unused variables.

Unused variables were found via pylint.

There are a few types of circumstances where we have unused variables,
so there are a few types of transformations:

 1. variables that are simply unused, e.g.
    a = f()  =>  <delete>

 2. functions that return tuples but only some arguments are used, e.g.
    a, b = f()  =>  a, _ = f()

 3. functions that have side effects and also a return value, e.g.
    a = do_something()  =>  do_something()

 4. exception variables that are unused
    except Exception as e:  =>  except Exception:
==========

to

==========
Remove all unused variables.

Unused variables were found via pylint.

There are a few types of circumstances where we have unused variables,
so there are a few types of transformations:

 1. variables that are simply unused, e.g.
    a = f()  =>  <delete>

 2. functions that return tuples but only some arguments are used, e.g.
    a, b = f()  =>  a, _ = f()

 3. functions that have side effects and also a return value, e.g.
    a = do_something()  =>  do_something()

 4. exception variables that are unused
    except Exception as e:  =>  except Exception:

Committed: https://crrev.com/69af2dee0ff02ff987b485e1ba297d94c8285427
Cr-Commit-Position: refs/heads/master@{#406887}
==========

[commit-bot: I haz the power, 2016-07-21 17:33:31]

Patchset 1 (id:??) landed as
https://crrev.com/69af2dee0ff02ff987b485e1ba297d94c8285427
Cr-Commit-Position: refs/heads/master@{#406887}