70230 2011-10-17 05:39:32 -0700 CSSParser: Remove unused variable 'propertyComponents' 2011-10-17 18:06:38 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 kling kling webkit.review.bot oldest_to_newest 485078 0 kling 2011-10-17 05:39:32 -0700 SSIA. 485082 1 111250 kling 2011-10-17 05:42:08 -0700 Created attachment 111250 Proposed patch 485084 2 webkit.review.bot 2011-10-17 05:45:13 -0700 Attachment 111250 did not pass style-queue: Failed to run "['Tools/Scripts/update-webkit', '--chromium']" exit_code: 2 Last 3072 characters of output: cessed: True deps_parsed: True requirements: ('./', 'chromium_deps', 'third_party') name: third_party/yasm/source/patched-yasm url: From('chromium_deps', 'src/third_party/yasm/source/patched-yasm') should_process: True requirements: ('./', 'chromium_deps', 'third_party') name: tools/clang url: http://src.chromium.org/svn/trunk/src/tools/clang@105777 parsed_url: http://src.chromium.org/svn/trunk/src/tools/clang@105777 should_process: True processed: True deps_parsed: True requirements: ('./',) name: tools/generate_stubs url: http://src.chromium.org/svn/trunk/src/tools/generate_stubs@105777 parsed_url: http://src.chromium.org/svn/trunk/src/tools/generate_stubs@105777 should_process: True processed: True deps_parsed: True requirements: ('./',) name: tools/grit url: From('chromium_deps', 'src/tools/grit') parsed_url: http://grit-i18n.googlecode.com/svn/trunk@6 should_process: True processed: True deps_parsed: True requirements: ('./', 'chromium_deps') name: tools/gritsettings url: http://src.chromium.org/svn/trunk/src/tools/gritsettings@105777 parsed_url: http://src.chromium.org/svn/trunk/src/tools/gritsettings@105777 should_process: True processed: True deps_parsed: True requirements: ('./',) name: tools/gyp url: From('chromium_deps', 'src/tools/gyp') parsed_url: http://gyp.googlecode.com/svn/trunk@1074 should_process: True processed: True deps_parsed: True requirements: ('./', 'chromium_deps') name: scons url: http://src.chromium.org/svn/trunk/src/third_party/scons@44099 requirements: ('tools/gyp',) name: tools/win url: http://src.chromium.org/svn/trunk/src/tools/win@105777 parsed_url: http://src.chromium.org/svn/trunk/src/tools/win@105777 should_process: True processed: True deps_parsed: True requirements: ('./',) name: tools/xdisplaycheck url: http://src.chromium.org/svn/trunk/src/tools/xdisplaycheck@105777 parsed_url: http://src.chromium.org/svn/trunk/src/tools/xdisplaycheck@105777 should_process: True processed: True deps_parsed: True requirements: ('./',) name: ui url: http://src.chromium.org/svn/trunk/src/ui@105777 parsed_url: http://src.chromium.org/svn/trunk/src/ui@105777 should_process: True processed: True deps_parsed: True requirements: ('./',) name: v8 url: From('chromium_deps', 'src/v8') should_process: True requirements: ('./', 'chromium_deps') name: webkit url: http://src.chromium.org/svn/trunk/src/webkit@105777 parsed_url: http://src.chromium.org/svn/trunk/src/webkit@105777 should_process: True processed: True deps_parsed: True requirements: ('./',) Error: 'depot_tools/gclient sync' failed 3 tries and returned 256 at Tools/Scripts/update-webkit-chromium line 107. Re-trying 'depot_tools/gclient sync' No such file or directory at Tools/Scripts/update-webkit line 104. If any of these errors are false positives, please file a bug against check-webkit-style. 485692 3 111250 webkit.review.bot 2011-10-17 18:06:34 -0700 Comment on attachment 111250 Proposed patch Clearing flags on attachment: 111250 Committed r97687: <http://trac.webkit.org/changeset/97687> 485693 4 webkit.review.bot 2011-10-17 18:06:38 -0700 All reviewed patches have been landed. Closing bug. 111250 2011-10-17 05:42:08 -0700 2011-10-17 18:06:34 -0700 Proposed patch bug-70230.diff text/plain 1202 kling SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk3NjA3KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTEtMTAtMTcgIEFuZHJlYXMg S2xpbmcgIDxrbGluZ0B3ZWJraXQub3JnPgorCisgICAgICAgIENTU1BhcnNlcjogUmVtb3ZlIHVu dXNlZCB2YXJpYWJsZSAncHJvcGVydHlDb21wb25lbnRzJworICAgICAgICBodHRwczovL2J1Z3Mu d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzAyMzAKKworICAgICAgICBSZXZpZXdlZCBieSBO T0JPRFkgKE9PUFMhKS4KKworICAgICAgICAoV2ViQ29yZTo6Q1NTUGFyc2VyOjptYXJrUHJvcGVy dHlFbmQpOgorCiAyMDExLTEwLTE3ICBBbGV4YW5kZXIgUGF2bG92ICA8YXBhdmxvdkBjaHJvbWl1 bS5vcmc+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogRWRpdGluZyBDU1Mgc2VsZWN0b3IgZG9l c24ndCBmaXJlIG9uUmVzb3VyY2VDb250ZW50Q29tbWl0dGVkCkluZGV4OiBTb3VyY2UvV2ViQ29y ZS9jc3MvQ1NTUGFyc2VyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9jc3MvQ1NT UGFyc2VyLmNwcAkocmV2aXNpb24gOTc2MDQpCisrKyBTb3VyY2UvV2ViQ29yZS9jc3MvQ1NTUGFy c2VyLmNwcAkod29ya2luZyBjb3B5KQpAQCAtNzQ2NCw3ICs3NDY0LDYgQEAKICAgICAgICAgU3Ry aW5nIHByb3BlcnR5U3RyaW5nID0gU3RyaW5nKG1fZGF0YSArIHN0YXJ0LCBlbmQgLSBzdGFydCku c3RyaXBXaGl0ZVNwYWNlKCk7CiAgICAgICAgIGlmIChwcm9wZXJ0eVN0cmluZy5lbmRzV2l0aCgi OyIsIHRydWUpKQogICAgICAgICAgICAgcHJvcGVydHlTdHJpbmcgPSBwcm9wZXJ0eVN0cmluZy5s ZWZ0KHByb3BlcnR5U3RyaW5nLmxlbmd0aCgpIC0gMSk7Ci0gICAgICAgIFZlY3RvcjxTdHJpbmc+ IHByb3BlcnR5Q29tcG9uZW50czsKICAgICAgICAgc2l6ZV90IGNvbG9uSW5kZXggPSBwcm9wZXJ0 eVN0cmluZy5maW5kKCI6Iik7CiAgICAgICAgIEFTU0VSVChjb2xvbkluZGV4ICE9IG5vdEZvdW5k KTsKIAo=