WinDDK ATL and MSVC express compatability

chrome/browser/ui/views/autocomplete/autocomplete_result_view.cc

Index: chrome/browser/ui/views/autocomplete/autocomplete_result_view.cc
===================================================================
--- chrome/browser/ui/views/autocomplete/autocomplete_result_view.cc	(revision 78547)
+++ chrome/browser/ui/views/autocomplete/autocomplete_result_view.cc	(working copy)
@@ -4,6 +4,11 @@
 
 #include "chrome/browser/ui/views/autocomplete/autocomplete_result_view.h"
 
+#if defined(OS_WIN)

[Mark Mentovai, 2011/03/17 20:43:42]

OK. If you ever need to do an early OS-specific #include as you say you need to
do here, you need to make sure of the following:

1. #include "build/build_config.h" first, otherwise OS_WIN may not be defined.
In this case, it probably works out fine because autocomplete_result_view.h
ultimately winds up including build_config.h, but if someone changes
build_config.h or some file on which it depends, that may cease to be the case.

2. Provide a quick comment along the lines of

// These headers are intentionally #included before any others.

[RN, 2011/03/18 09:34:24]

Makes sense. Done.

+#include <atlbase.h>
+#include <atlwin.h>
+#endif
+
 #include "base/i18n/bidi_line_iterator.h"
 #include "chrome/browser/ui/views/autocomplete/autocomplete_result_view_model.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"