Revert of Python readability review for dbeam@.

(closed, pfeldman did this already)

Revert of Python readability review for dbeam@. (patchset #4 id:280001 of https://codereview.chromium.org/476453002/)

Reason for revert:
Looks like this broke the Linux and Linux 64 clobber bots:

http://build.chromium.org/p/chromium/builders/Linux%20x64/builds/433
http://build.chromium.org/p/chromium/builders/Linux/builds/59857

FAILED: cd ../../remoting; python ../third_party/closure_compiler/checker.py --strict --no-single-file --success-stamp ../out/Release/remoting_key_tester_jscompile.stamp tools/javascript_key_tester/background.js tools/javascript_key_tester/chord_tracker.js tools/javascript_key_tester/event_listeners.js tools/javascript_key_tester/main.js webapp/js_proto/chrome_proto.js webapp/js_proto/chrome_event_proto.js
(ERROR) Exception in thread "main" java.lang.UnsupportedClassVersionError: org/chromium/closure/compiler/Runner : Unsupported major.minor version 51.0
	at java.lang.ClassLoader.defineClass1(Native Method)
	at java.lang.ClassLoader.defineClassCond(ClassLoader.java:631)
	at java.lang.ClassLoader.defineClass(ClassLoader.java:615)
	at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:141)
	at java.net.URLClassLoader.defineClass(URLClassLoader.java:283)
	at java.net.URLClassLoader.access$000(URLClassLoader.java:58)
	at java.net.URLClassLoader$1.run(URLClassLoader.java:197)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.net.URLClassLoader.findClass(URLClassLoader.java:190)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:306)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:301)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:247)
Could not find the main class: org.chromium.closure.compiler.Runner. Program will exit.

Original issue's description:
> Python readability review for dbeam@.
> 
> R=rothwell@google.com
> 
> Committed: https://crrev.com/c5503ce046fdad8c76129c0061f0d6011c2fe92b
> Cr-Commit-Position: refs/heads/master@{#321693}

TBR=rothwell@google.com,thestig@chromium.org,dbeam@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

[Nico, 2015-03-21 21:00:23]

Created Revert of Python readability review for dbeam@.

[commit-bot: I haz the power, 2015-03-21 21:00:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1022293002/1

[commit-bot: I haz the power, 2015-03-21 21:01:25]

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

[commit-bot: I haz the power, 2015-03-21 21:01:25]

Failed to apply the patch.

[commit-bot: I haz the power, 2015-03-21 21:01:31]

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

[Dan Beam, 2015-03-23 19:16:33]

pfeldman@ beat you to this: can we close?

[Nico, 2015-03-23 19:22:41]

On 2015/03/23 19:16:33, Dan Beam wrote:
> pfeldman@ beat you to this: can we close?

Err, sure. (I don't close most of the CLs I don't end up using *shrug*)

[Dan Beam, 2015-03-23 19:23:13]

On 2015/03/23 19:22:41, Nico wrote:
> On 2015/03/23 19:16:33, Dan Beam wrote:
> > pfeldman@ beat you to this: can we close?
> 
> Err, sure. (I don't close most of the CLs I don't end up using *shrug*)

or delete?

[Nico, 2015-03-23 19:24:03]

Never delete CLs, that breaks links.

On Mon, Mar 23, 2015 at 12:23 PM, <dbeam@chromium.org> wrote:

> On 2015/03/23 19:22:41, Nico wrote:
>
>> On 2015/03/23 19:16:33, Dan Beam wrote:
>> > pfeldman@ beat you to this: can we close?
>>
>
>  Err, sure. (I don't close most of the CLs I don't end up using *shrug*)
>>
>
> or delete?
>
> https://codereview.chromium.org/1022293002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.