Disable subpixel positioning if DirectWrite isn't initialized

ui/gfx/render_text_unittest.cc

 // 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.
 
 #include "ui/gfx/render_text.h"
 
 #include <algorithm>
 
 #include "base/format_macros.h"
 #include "base/i18n/break_iterator.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/break_list.h"
 #include "ui/gfx/canvas.h"
+#include "ui/gfx/color_utils.h"
 #include "ui/gfx/font.h"
 #include "ui/gfx/render_text_harfbuzz.h"
 
 #if defined(OS_WIN)
 #include "base/win/windows_version.h"
 #include "ui/gfx/platform_font_win.h"
 #include "ui/gfx/render_text_win.h"
 #endif
 
 #if defined(OS_LINUX) && !defined(USE_OZONE)
@@ skipping 2365 matching lines @@
     sk_canvas->clear(SK_ColorWHITE);
     render_text->SetText(WideToUTF16(kTestStrings[i]));
     render_text->SetStyle(BOLD, true);
     render_text->Draw(canvas.get());
     int width = render_text->GetStringSize().width();
     ASSERT_LT(width, 300);
     const uint32* buffer = static_cast<const uint32*>(
         sk_canvas->peekPixels(NULL, NULL));
     ASSERT_NE(nullptr, buffer);
     for (int y = 0; y < 50; ++y) {
-      EXPECT_EQ(SK_ColorWHITE, buffer[width + y * 300])
+      const SkColor color = buffer[width + y * 300];
+      EXPECT_LT(230U, color_utils::GetLuminanceForColor(color))

[msw, 2014/12/10 23:55:50]

Add a comment explaining the lax condition here.

[msw, 2014/12/11 23:30:12]

Should this perhaps just allow the first column past the expected width to have
anti-aliased content? Optionally: "// Allow [a single column of] anti-aliased
pixels past the expected width."

[ckocagil, 2014/12/12 09:52:11]

Done.

           << "String: " << kTestStrings[i];
     }
   }
 }
 
 }  // namespace gfx