Commit 5ef46522 authored by thestig@chromium.org's avatar thestig@chromium.org

Cleanup: Remove an obsolete TODO.

BUG=140571
TEST=none
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/10826155

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149950 0039d316-1c4b-4281-b951-d872f2087c98
parent 1f326d0c
...@@ -416,15 +416,12 @@ ...@@ -416,15 +416,12 @@
'../pdf/pdf.gyp:pdf', '../pdf/pdf.gyp:pdf',
], ],
'conditions': [ 'conditions': [
# TODO(thestig) It would be nice to do this for CrOS as well, # CrOS does this in a separate build step.
# but when I tried that, it failed because dump_syms is built
# for the target arch rather than the host arch.
# http://crbug.com/140571
['OS=="linux" and chromeos==0 and linux_dump_symbols==1', { ['OS=="linux" and chromeos==0 and linux_dump_symbols==1', {
'dependencies': [ 'dependencies': [
'../pdf/pdf.gyp:pdf_linux_symbols', '../pdf/pdf.gyp:pdf_linux_symbols',
], ],
}], # OS=="linux" and linux_dump_symbols==1 }], # OS=="linux" and chromeos==0 and linux_dump_symbols==1
], ],
}], # internal_pdf }], # internal_pdf
], ],
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment