Bug 70230 – CSSParser: Remove unused variable 'propertyComponents'

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Andreas Kling]

SSIA.

[Andreas Kling]

Created attachment 111250 [details]
Proposed patch

[WebKit Review Bot]

Attachment 111250 [details] 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.

[WebKit Review Bot]

Comment on attachment 111250 [details]
Proposed patch

Clearing flags on attachment: 111250

Committed r97687: <http://trac.webkit.org/changeset/97687>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.