LGTM ======================================================================== http://mondrian.corp.google.com/file/8228214///depot/googleclient/gears/opensource/gears/test/testcases/cgi/location_provider.py?a=4 File //depot/googleclient/gears/opensource/gears/test/testcases/cgi/location_provider.py (snapshot 4) ------------------------------------ Line 10: This script is used to simulate test three cases: simulate/test ------------------------------------ Line 75: outer_key: The name of the dictionary, expected to contain the probably should use similar variable names - outer_key/inner_key or outer_key_name/inner_key_name ------------------------------------ Line 101: 'mac_address', alignment ========================================================================
-- To respond, reply to this email or visit http://mondrian.corp.google.com/8228214
