Remove SPECIAL_KEYS and GclChange.Changelist() to simplify the code.

tests/presubmit_unittest.py

Index: tests/presubmit_unittest.py
diff --git a/tests/presubmit_unittest.py b/tests/presubmit_unittest.py
index f50a0c59bdf17b54eddaf3f9e24007109783d8f9..e6d3d5ed895cfc0762c9b85ee43a220b26452a4c 100755
--- a/tests/presubmit_unittest.py
+++ b/tests/presubmit_unittest.py
@@ -116,7 +116,7 @@ class PresubmitUnittest(PresubmitTestsBase):
     members = [
       'AffectedFile', 'DoPresubmitChecks', 'GclChange', 'InputApi',
       'ListRelevantPresubmitFiles', 'Main', 'NotImplementedException',
-      'OutputApi', 'ParseFiles', 'PresubmitExecuter', 'SPECIAL_KEYS',
+      'OutputApi', 'ParseFiles', 'PresubmitExecuter',
       'ScanSubDirs', 'SvnAffectedFile', 'cPickle', 'cStringIO', 'exceptions',
       'fnmatch', 'gcl', 'gclient', 'glob', 'marshal', 'normpath', 'optparse',
       'os', 'pickle', 'presubmit_canned_checks', 're', 'subprocess', 'sys',
@@ -162,13 +162,11 @@ class PresubmitUnittest(PresubmitTestsBase):
     change = presubmit.GclChange(ci)
 
     self.failUnless(change.Change() == 'mychange')
-    self.failUnless(change.Changelist() == 'mychange')
     self.failUnless(change.DescriptionText() ==
                     'Hello there\nthis is a change\nand some more regular text')
     self.failUnless(change.FullDescriptionText() ==
                     '\n'.join(description_lines))
 
-    self.failUnless(change.BugIDs == '123')
     self.failUnless(change.BUG == '123')
     self.failUnless(change.STORY == 'http://foo/')