Граф коммитов

1822 Коммитов

Автор SHA1 Сообщение Дата
epoger@google.com 3095a94d9c Disable Skia on Mac until we resolve http://code.google.com/p/chromium/issues/detail?id=98079
('Unable to see any "TEXT' in chrome browser.')
Review URL: http://codereview.chromium.org/8054015

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102944 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-27 16:30:55 +00:00
thestig@chromium.org b5abd53834 Linux: add a missing check for /usr/local/bin/gold/ld.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/8036008

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102860 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-27 01:57:09 +00:00
joi@chromium.org 7464670db3 Use precompiled headers on Windows only when a flag is set.
I would have preferred if precompiled headers could be enabled by
default. The motivation for this change is primarily that Visual
Studio does the wrong thing on preprocessor flag changes - instead of
rebuilding precompiled headers, it errors out on any file that uses
them, saying that the precompiled header was built with different
flags! This causes a world of hurt on our infrastructure stuff,
particularly on trybots as they may be jumping back and forth between
revisions before and after a flag change.

Prior to this change, only Debug builds used precompiled headers and
Release builds did not. The reason Release builds did not was that
official builds were running out of memory with precompiled headers
enabled. The distinction of Debug vs. Release is no longer necessary
after the flag is added, so removing that extra bit of complexity from
the .gyp files.

BUG=none
TEST=it builds


Review URL: http://codereview.chromium.org/8041022

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102839 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-27 00:15:16 +00:00
michaelbai@google.com 695c3263b9 Upstream: Set build target and evnvironment for Android
BUG=
TEST=

Review URL: http://codereview.chromium.org/8008026

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102807 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-26 21:40:38 +00:00
jschuh@chromium.org c5b1e48cdd Forcing builder to cycle.
Review URL: http://codereview.chromium.org/8043026

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102737 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-26 16:33:51 +00:00
jschuh@chromium.org 54476b5bd1 Update Windows version macros to Win 7
BUG=92941
Review URL: http://codereview.chromium.org/8004004

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102735 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-26 16:06:47 +00:00
cmp@chromium.org 346541bc9d Whitespace commit to start builds.
git-svn-id: http://src.chromium.org/svn/trunk/src/build@102699 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-26 05:47:05 +00:00
rsleevi@chromium.org 8b8e34070f Kick the bots
TBR=bevc
BUG=none
TEST=none

Review URL: http://codereview.chromium.org/8037020

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102696 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-26 04:48:58 +00:00
cmp@chromium.org e8d11ddc0a Whitespace change to start builds.
git-svn-id: http://src.chromium.org/svn/trunk/src/build@102678 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-25 05:46:47 +00:00
sadrul@chromium.org d0ac9c0cb5 touchui: support XInput2 multitouch
Use XI2 multitouch events instead of mouse events as touch event input for touchui build.

Note: XI MT will be supported in X server 1.12 and XI2.2. Please use build switch "use_xi2_mt=<minor version number>" to specify the minimum XI2 minor version. It is useful to test on experimental XI2.1 with MT support (e.g. build with use_xi2_mt=1).

BUG=95150
TEST=(1) build with touchui=1 use_xi2_mt=1 (2) test on ubuntu 11.04 (X server 1.10 and XI2.1 with experimental MT support). (3) manually test if touch works on browser UI and JS touch events.

Review URL: http://codereview.chromium.org/7792094
Patch from Ningxin Hu <ningxin.hu@intel.com>.

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102668 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-25 00:04:35 +00:00
saintlou@chromium.org b35b9c2677 + renamed window_snapshot_x.cc => window_snapshot_gtk.cc
+ added window_snapshot_aura.cc
+ added missing members to icon_loader_aura.cc
+ fixed a couple of minor issues with #includes

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/8028012

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102659 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-24 18:32:49 +00:00
sievers@chromium.org b3ca002e55 Change default for install 32bit libs on 64bit Linux systems to NO.
Review URL: http://codereview.chromium.org/8004007

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102582 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-23 21:35:18 +00:00
thakis@chromium.org 4f6249725f Add curl to build/install-build-deps.sh
tools/clang/scripts/update.sh tries to use curl to download over https (wget has
issues with this). To make sure curl is installed on new bots, add it to
install-build-deps.sh, so that we can use the https download on bots.

BUG=97288
TEST=none


Review URL: http://codereview.chromium.org/7966029

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102566 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-23 20:42:39 +00:00
epoger@google.com 9dfcbb32fd Change default on Mac to use_skia=1
For context, see http://groups.google.com/a/chromium.org/group/chromium-dev/browse_thread/thread/f8d4f43b6e66ab83#
Review URL: http://codereview.chromium.org/8002009

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102532 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-23 18:18:14 +00:00
evan@chromium.org 82db340bc5 linux: convert more dependencies to 'settings'
BUG=96629

Review URL: http://codereview.chromium.org/7970010

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102523 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-23 17:36:06 +00:00
tony@chromium.org 28e023ec89 On Windows, force the output directories to be src/build/{Debug,Release}.
This allows develoeprs to build from any sln file and not have to rebuild the
world.  This is more like how XCode projects are generated.

We choose src/build/{Debug,Release} because this is where all.sln generates
files and this is what the bots use.  This shouldn't require any changes to
the bots.

BUG=95278

Review URL: http://codereview.chromium.org/7835006

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102514 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-23 17:15:32 +00:00
epoger@google.com e386daedd5 whitespace-only test commit
git-svn-id: http://src.chromium.org/svn/trunk/src/build@102495 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-23 14:26:41 +00:00
sadrul@chromium.org eb44487d8e aura: Have aura and aura_demo build and run without gtk on linux.
BUG=97131
TEST=manually

Review URL: http://codereview.chromium.org/7981030

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102403 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-23 00:08:46 +00:00
thakis@chromium.org 3a140855ab mac: Don't build with debug information on the trybots.
Locally, a full build is 50% faster with this.

BUG=none
TEST=mac trybot build time should go down at least 7 minutes.

Review URL: http://codereview.chromium.org/7994008

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102343 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-22 20:23:14 +00:00
davemoore@chromium.org 8b69126dac These are the changes necessary for views to be able to build w/ use_aura implying no gtk.
BUG=97131
TEST=views_unittests

Review URL: http://codereview.chromium.org/7996008

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102335 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-22 19:39:52 +00:00
evan@chromium.org 4b3b87179a Remove 'settings' type from gyp files
It is not actually supported by gyp; the fact that it works is a bug
that I am working on fixing.

BUG=96629

Review URL: http://codereview.chromium.org/7889051

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102306 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-22 17:59:43 +00:00
bradnelson@google.com 0f3b860b98 Temporarily turning off stripping for mac builds on the main waterfall to
debug intermittent crash seen only there.

BUG=None
TEST=None
R=bradchen@chromium.org
Review URL: http://codereview.chromium.org/7976042

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102221 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-22 01:09:01 +00:00
bradnelson@google.com 0912a5375f Adding libssl-dev to the list of packages required for chrome development.
Chrome uses an explicit third_party copy of this library,
however, temporarily the nacl_integration_tests require the system
version to be installed to build sel_ldr.

BUG=http://code.google.com/p/nativeclient/issues/detail?id=2265
TEST=None
R=nfullagar@google.com
Review URL: http://codereview.chromium.org/7977035

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102188 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 23:14:26 +00:00
jamescook@chromium.org 0b31c3206c Aura: Use extension-based file picker from CrOS for pure-views builds.
Removes the dependency on chromeos-specific code in the extension-based file picker.  Compiles, links, and runs, but the dialog doesn't function yet - there's still some work to do to wire up the private JavaScript API calls.  See crbug.com/97424

BUG=chromium-os:20583
TEST=On plain ChromeOS, can still open a file with control-O file picker.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=102113

Review URL: http://codereview.chromium.org/7941014

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102156 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 20:26:19 +00:00
sadrul@chromium.org 5f139a3110 aura: Make 'ui' buildable without gtk.
BUG=97131
TEST=ui_unittests links and runs with use_aura=1

Review URL: http://codereview.chromium.org/7977025

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102131 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 17:52:54 +00:00
jamescook@chromium.org d79a0b0dba Revert 102113 - Aura: Use extension-based file picker from CrOS for pure-views builds.
Removes the dependency on chromeos-specific code in the extension-based file picker.  Compiles and links, but does not yet run - the extension subsystem is not finding the resources for the file_manager component extension.  The Aura build currently won't process keyboard or mouse events, so I don't have a good way to debug this.  For now, added logging, and we'll get back to it when Aura is in better shape.

BUG=chromium-os:20583
TEST=On plain ChromeOS, can still open a file with control-O file picker.

Review URL: http://codereview.chromium.org/7941014

TBR=jamescook@chromium.org
Review URL: http://codereview.chromium.org/7976025

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102116 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 16:44:47 +00:00
jamescook@chromium.org e71037fab5 Aura: Use extension-based file picker from CrOS for pure-views builds.
Removes the dependency on chromeos-specific code in the extension-based file picker.  Compiles and links, but does not yet run - the extension subsystem is not finding the resources for the file_manager component extension.  The Aura build currently won't process keyboard or mouse events, so I don't have a good way to debug this.  For now, added logging, and we'll get back to it when Aura is in better shape.

BUG=chromium-os:20583
TEST=On plain ChromeOS, can still open a file with control-O file picker.

Review URL: http://codereview.chromium.org/7941014

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102113 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 16:27:13 +00:00
sadrul@chromium.org f3bba76ca4 aura: Make GTK a dependency again for chrome/aura etc. to make things buildable.
BUG=97131
TEST=none

Review URL: http://codereview.chromium.org/7978016

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102078 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 03:01:09 +00:00
flackr@chromium.org a16a345354 Add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs (Patch set 1 Reverts revert of http://codereview.chromium.org/7670041/)
This adds the flag --use-more-webui to allow turning on WebUI replacements for native dialogs. This flag is automatically set when --use-pure-views is specified. Modifies replaced native dialogs to be toggled based on flag value. BookmarkEditor has also been refactored to use static constructors to EditDetails to describe what is being edited to make the code more readable.

BUG=96935
TEST=Tested that webui dialogs can be turned on or left off with --use-more-webui.


Review URL: http://codereview.chromium.org/7957002

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102071 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 02:03:38 +00:00
sadrul@chromium.org ef323b60a0 Reland 102005 and 102009: aura: Explicitly disable GTK.
Explicitly disable GTK by setting TOOLKIT_USES_GTK. This is the first in a
series of changes required to have chrome on aura build and link without GTK.
This CL makes base/ build without GTK. Subsequent CLs will do this for other
components.

TBR=evan@chromium.org
BUG=97131
TEST=none

Review URL: http://codereview.chromium.org/7983022

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102058 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-21 00:17:19 +00:00
joi@chromium.org 17492745b4 Relanding precompiled header support for VS 2008, now enabled for
Debug builds only as official builds were failing with out of memory
errors.

This is mostly a revert of http://crrev.com/101836 which was a
revert of the original change, http://crrev.com/101474, except for a
modification to build/win_precompile.gypi to only enable use of
precompiled headers in the Debug configuration.

BUG=none
TEST=it builds

Review URL: http://codereview.chromium.org/7977003

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102029 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-20 21:46:48 +00:00
saintlou@chromium.org 6a24e6d188 Making TOOLKIT_USES_GTK orthogonal to USE_AURA.
BUG=none
TEST=none

Review URL: http://codereview.chromium.org/7973003

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102019 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-20 21:20:33 +00:00
nsylvain@chromium.org 3e5d0ac178 Add a gyp flag to enable dcheck by default in release without
having the pass a flag. This will be used on the try bots.

BUG=96753
Review URL: http://codereview.chromium.org/7719007

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102017 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-20 20:59:01 +00:00
sky@chromium.org 39c9343ddc Revert 102005 - aura: Explicitly disable GTK.
Explicitly disable GTK by setting TOOLKIT_USES_GTK. This is the first in a series of changes required to have chrome on aura build and link without GTK. This CL makes base/ build without GTK. Subsequent CLs will do this for other components.

BUG=97131
TEST=none

Review URL: http://codereview.chromium.org/7904034

TBR=sadrul@chromium.org
Review URL: http://codereview.chromium.org/7978007

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102015 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-20 20:56:51 +00:00
michaelbai@google.com 10e10ee112 Upstream: The Common.gypi change for Android.
BUG=
TEST=

Review URL: http://codereview.chromium.org/7941007

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102007 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-20 20:35:01 +00:00
sadrul@chromium.org be2c499b31 aura: Explicitly disable GTK.
Explicitly disable GTK by setting TOOLKIT_USES_GTK. This is the first in a series of changes required to have chrome on aura build and link without GTK. This CL makes base/ build without GTK. Subsequent CLs will do this for other components.

BUG=97131
TEST=none

Review URL: http://codereview.chromium.org/7904034

git-svn-id: http://src.chromium.org/svn/trunk/src/build@102005 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-20 20:29:52 +00:00
joi@chromium.org a22b81862a Temporary revert of r101474, will re-land tomorrow with a change to disable use of precompiled headers in Release builds as our Canary builds are failing with out of memory messages.
Original change description: Switch on use of precompiled headers for VS 2008.

The precompiled header brings in more definitions of abs() for the base/time_win.cc file, and a hint to the compiler was needed to let VS 2008 build the file correctly.

On VS 2008 only, the mini_installer target did not link correctly with precompiled headers enabled so this change disables them for that project.

BUG=none
TEST=it builds (esp. on Windows), existing tests pass

Review URL: http://codereview.chromium.org/7910002

TBR=laforge@chromium.org
Review URL: http://codereview.chromium.org/7945017

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101836 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-19 21:29:33 +00:00
piman@chromium.org 1724c47105 Fix duplicate rule for LASTCHANGE
BUG=None
TEST=build it on linux (make/ninja), check no warning of duplicate rule


Review URL: http://codereview.chromium.org/7920025

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101793 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-19 19:20:38 +00:00
scherkus@chromium.org 49a36b92da Revert 101785 - Add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs.
This adds the flag --use-more-webui to allow turning on WebUI replacements for native dialogs. This flag is automatically set when --use-pure-views is specified. Modifies replaced native dialogs to be toggled based on flag value. BookmarkEditor has also been refactored to use static constructors to EditDetails to describe what is being edited to make the code more readable.

BUG=96935
TEST=Tested that webui dialogs can be turned on or left off with --use-more-webui.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=101581

Review URL: http://codereview.chromium.org/7670041

TBR=flackr@chromium.org
Review URL: http://codereview.chromium.org/7957001

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101789 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-19 18:49:47 +00:00
flackr@chromium.org 9cacad61ce Add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs.
This adds the flag --use-more-webui to allow turning on WebUI replacements for native dialogs. This flag is automatically set when --use-pure-views is specified. Modifies replaced native dialogs to be toggled based on flag value. BookmarkEditor has also been refactored to use static constructors to EditDetails to describe what is being edited to make the code more readable.

BUG=96935
TEST=Tested that webui dialogs can be turned on or left off with --use-more-webui.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=101581

Review URL: http://codereview.chromium.org/7670041

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101785 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-19 18:33:23 +00:00
nsylvain@chromium.org 91fe2a6840 Dummy change to trigger the bots
git-svn-id: http://src.chromium.org/svn/trunk/src/build@101712 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-18 19:12:41 +00:00
saintlou@chromium.org 300f01fe76 Aura under Linux is enabled with:
'use_aura': 1,

BUG=none
TEST=none

Review URL: http://codereview.chromium.org/7850026

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101704 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-18 17:28:50 +00:00
thakis@chromium.org f35e798c42 cycle bots
TBR=rsleevi

Review URL: http://codereview.chromium.org/7933009

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101660 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-17 21:00:23 +00:00
rsimha@chromium.org 315f1523fb Revert 101581 - Add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs.
Reason for revert: Broke compile on linux touch.

This adds the flag --use-more-webui to allow turning on WebUI replacements for native dialogs. This flag is automatically set when --use-pure-views is specified. Modifies replaced native dialogs to be toggled based on flag value. BookmarkEditor has also been refactored to use static constructors to EditDetails to describe what is being edited to make the code more readable.

BUG=None
TEST=Tested that webui dialogs can be turned on or left off with --use-more-webui.

Review URL: http://codereview.chromium.org/7670041

TBR=flackr@chromium.org
Review URL: http://codereview.chromium.org/7919028

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101585 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-16 22:31:20 +00:00
flackr@chromium.org 27bd099576 Add --use-more-webui runtime flag to toggle WebUI replacements for native dialogs.
This adds the flag --use-more-webui to allow turning on WebUI replacements for native dialogs. This flag is automatically set when --use-pure-views is specified. Modifies replaced native dialogs to be toggled based on flag value. BookmarkEditor has also been refactored to use static constructors to EditDetails to describe what is being edited to make the code more readable.

BUG=None
TEST=Tested that webui dialogs can be turned on or left off with --use-more-webui.

Review URL: http://codereview.chromium.org/7670041

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101581 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-16 22:09:29 +00:00
ben@chromium.org 77a8380c08 Adds some features to the shell:
- desktop background
- stub launcher
- clicking the background opens test windows

BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7903018

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101496 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-16 16:37:30 +00:00
thakis@chromium.org 77909dd432 mac: Turn style plugin on by default.
BUG=none
TEST=none

Review URL: http://codereview.chromium.org/7922006

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101494 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-16 16:11:07 +00:00
joi@chromium.org c912f5404f Switch on use of precompiled headers for VS 2008.
The precompiled header brings in more definitions of abs() for the base/time_win.cc file, and a hint to the compiler was needed to let VS 2008 build the file correctly.

On VS 2008 only, the mini_installer target did not link correctly with precompiled headers enabled so this change disables them for that project.

BUG=none
TEST=it builds (esp. on Windows), existing tests pass

Review URL: http://codereview.chromium.org/7910002

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101474 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-16 09:54:23 +00:00
ben@chromium.org 8fa174cc18 Don't build this gyp file, it's clearly problematic.
git-svn-id: http://src.chromium.org/svn/trunk/src/build@101382 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-15 21:31:56 +00:00
pkasting@chromium.org 36e607a539 Fourth attempt at the following. Unreviewed, this was originally reviewed in http://codereview.chromium.org/7550002 .
Clean up SSL false start blacklist code. Numerous changes, including:
* Handle trailing dots in LastTwoLabels() as in http://codereview.chromium.org/7518035/ . Rename this function to LastTwoComponents() to match the terminology used in the RegistryControlledDomainService and elsewhere in Chrome.
* Since callers are using std::string anyway, make the functions in the header take const std::string& instead of char*. This also allows doing string operations on them.
* Use string operations (like find_last_of()) in place of hand-written algorithms, for brevity, clarity, and safety.
* Avoid "unsigned", which the style guide forbids, and use allowed types like size_t, uint32, or int (depending on the situation).
* Avoid #define and "using".
* Use standard algorithms for similar reasons as using string ops.
* Use file_util functions to significantly abbreviate file reading/writing code.
* Use wmain() (on Windows) in combination with FilePath to avoid issues if the provided pathname has extended characters that don't flatten losslessly to the default codepage (thanks Darin for pointing out this issue).
* Avoid casting where possible. Avoid some casts for printf()-style calls by using a string stream, which also allows for slightly less boilerplate.
* Convert non-error uses of stderr to the chrome-standard VLOG(1).
* Correctly handle hostnames with trailing dots in the input file.
* In general, shorten code where possible.

Because this adds a dependency on base, and ssl_false_start_blacklist_process has the "#host" specifier in net.gyp, bradnelson tells me that base and its dependencies need an explicit "host, target" toolchain list for the Linux builds to work correctly. It would be nice if we could avoid this but I guess gyp would have to be smarter or something.

BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7903016

git-svn-id: http://src.chromium.org/svn/trunk/src/build@101378 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
2011-09-15 21:17:32 +00:00