diff options
author | craig.schlenter@chromium.org <craig.schlenter@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-07 19:01:41 +0000 |
---|---|---|
committer | craig.schlenter@chromium.org <craig.schlenter@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-07 19:01:41 +0000 |
commit | 7e93e2f52a534ff2fb1434ae7bca88dc8bd8e4df (patch) | |
tree | cbc5f50dbf28f24d71383466e6a6cd6d4aef6aec /chrome/chrome.gyp | |
parent | 3ee8e02bec65d80ed1ae652470af1c97f1575559 (diff) | |
download | chromium_src-7e93e2f52a534ff2fb1434ae7bca88dc8bd8e4df.zip chromium_src-7e93e2f52a534ff2fb1434ae7bca88dc8bd8e4df.tar.gz chromium_src-7e93e2f52a534ff2fb1434ae7bca88dc8bd8e4df.tar.bz2 |
Fix some gcc 4.4 issues when compiling with toolkit_views.
Most of these squash harmless compiler warnings but the custom_button.cc
change fixes a real problem with the accelerator key bitmask calculation
(+ preceeds << in terms of operator precedence)
The change to tab_strip.cc is ugly but it fixes the following error:
chrome/browser/views/tabs/tab_strip.cc: In member function
‘void TabStrip::StartRemoveTabAnimation(int, TabContents*)’:
chrome/browser/views/tabs/tab_strip.cc:201: error: assuming
signed overflow does not occur when assuming that
(X - c) > X is always false
As an added bonus, this makes the Linux shared build of toolkit_views
work too.
Review URL: http://codereview.chromium.org/199025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@25602 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/chrome.gyp')
-rw-r--r-- | chrome/chrome.gyp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/chrome/chrome.gyp b/chrome/chrome.gyp index 5aa5234..629321c 100644 --- a/chrome/chrome.gyp +++ b/chrome/chrome.gyp @@ -2532,8 +2532,6 @@ ['include', '^browser/gtk/import_lock_dialog_gtk.h'], ['include', '^browser/gtk/keyword_editor_view.cc'], ['include', '^browser/gtk/keyword_editor_view.h'], - ['include', '^browser/gtk/list_store_favicon_loader.cc'], - ['include', '^browser/gtk/list_store_favicon_loader.h'], ['include', '^browser/gtk/menu_gtk.cc'], ['include', '^browser/gtk/menu_gtk.h'], ['include', '^browser/gtk/tab_contents_drag_source.cc'], @@ -2550,6 +2548,11 @@ ['exclude', '^browser/tab_contents/tab_contents_view_gtk.h'], ['exclude', '^browser/tab_contents/render_view_context_menu_gtk.cc'], ['exclude', '^browser/tab_contents/render_view_context_menu_gtk.h'], + # This compiles but has Linux shared build linking issues due to + # missing rsa_private_key functions. + ['exclude', '^browser/extensions/extension_creator.cc'], + # This compiles but it needs extension_creator. + ['exclude', '^browser/extensions/pack_extension_job.cc'], ], }], ['chromeos==1',{ |