Change a couple of TODOs to plain comments.
BUG=none
Review URL: http://codereview.chromium.org/10204018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133741 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/build/precompile.h b/build/precompile.h
index 73dc57f..a3c5193 100644
--- a/build/precompile.h
+++ b/build/precompile.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -9,8 +9,9 @@
//
// Numeric comments beside includes are the number of times they were
// included under src/chrome/browser on 2011/8/20, which was used as a
-// baseline for deciding what to include in the PCH. It may be
-// possible to tweak the speed of the build by commenting out or
+// baseline for deciding what to include in the PCH. Includes without
+// a numeric comment are generally included at least 5 times. It may
+// be possible to tweak the speed of the build by commenting out or
// removing some of the less frequently used headers.
#if defined(BUILD_PRECOMPILE_H_)
@@ -27,9 +28,9 @@
#include <wincrypt.h> // 4
#include <wtypes.h> // 2
-// TODO(joi): Defines in atlbase.h cause conflicts; need to figure out
-// if/how this family of headers can be included in the PCH; several
-// of them are used quite frequently.
+// Defines in atlbase.h cause conflicts; if we could figure out how
+// this family of headers can be included in the PCH, it might speed
+// up the build as several of them are used frequently.
/*
#include <atlbase.h>
#include <atlapp.h>
@@ -42,11 +43,11 @@
#include <atlwin.h> // 2
*/
-// TODO(joi): Objbase.h and other files that rely on it bring in
-// [ #define interface struct ] which can cause problems in a
-// multi-platform build like Chrome's. Tried #undef-ing it, but
-// there are currently 118 targets that break if we do this, so
-// leaving out of the precompiled header for now.
+// Objbase.h and other files that rely on it bring in [ #define
+// interface struct ] which can cause problems in a multi-platform
+// build like Chrome's. #undef-ing it does not work as there are
+// currently 118 targets that break if we do this, so leaving out of
+// the precompiled header for now.
//#include <commctrl.h> // 2
//#include <commdlg.h> // 3
//#include <cryptuiapi.h> // 2
@@ -64,9 +65,8 @@
//#include <shobjidl.h> // 4
//#include <urlhist.h> // 2
-// TODO(joi): Caused other conflicts in addition to the 'interface' issue
-// above, see if they can be resolved.
-//#include <shlobj.h>
+// Caused other conflicts in addition to the 'interface' issue above.
+// #include <shlobj.h>
#include <errno.h>
#include <fcntl.h>
