Link to DirectWrite directly, instead of calling LoadLibrary

Link to DirectWrite directly, instead of calling LoadLibrary

BUG=

Committed: https://crrev.com/7561063aa78a31c73506f666c9eeccf45fe3dc07
Cr-Commit-Position: refs/heads/master@{#403344}

[Ilya Kulshin, 2016-06-11 01:10:47]

Patchset #6 (id:100001) has been deleted

[Ilya Kulshin, 2016-06-24 21:15:01]

Description was changed from

==========
Link to DirectWrite directly, instead of calling LoadLibrary

BUG=
==========

to

==========
Link to DirectWrite directly, instead of calling LoadLibrary

TODO: remove new calls to LoadLibrary introduced by
https://codereview.chromium.org/2054273002/

BUG=
==========

[Ilya Kulshin, 2016-06-28 22:57:18]

kulshin@chromium.org changed reviewers:
+ scottmg@chromium.org

[Ilya Kulshin, 2016-06-28 22:57:18]

ptal

Still need to wait for https://codereview.chromium.org/2054273002/ to land, but
would like to get a start on this review.

[scottmg, 2016-06-28 23:22:45]

lgtm

https://codereview.chromium.org/2029343002/diff/120001/content/child/dwrite_f...
File content/child/dwrite_font_proxy/dwrite_font_proxy_win_unittest.cc (left):

https://codereview.chromium.org/2029343002/diff/120001/content/child/dwrite_f...
content/child/dwrite_font_proxy/dwrite_font_proxy_win_unittest.cc:46: if
(!factory)
Were all these early outs because the dll wasn't loaded? Or because the creation
failed? (I guess we'll find out...?)

[Ilya Kulshin, 2016-06-28 23:42:50]

https://codereview.chromium.org/2029343002/diff/120001/content/child/dwrite_f...
File content/child/dwrite_font_proxy/dwrite_font_proxy_win_unittest.cc (left):

https://codereview.chromium.org/2029343002/diff/120001/content/child/dwrite_f...
content/child/dwrite_font_proxy/dwrite_font_proxy_win_unittest.cc:46: if
(!factory)
On 2016/06/28 23:22:45, scottmg wrote:
> Were all these early outs because the dll wasn't loaded? Or because the
creation
> failed? (I guess we'll find out...?)

These were there because we still needed XP and Vista support when this code was
first written, and dwrite does not exist in some of those versions of Windows.
Rather than have the unittests crash, I had them abort, and never got around to
removing these checks until now.

[Ilya Kulshin, 2016-06-30 00:47:44]

Description was changed from

==========
Link to DirectWrite directly, instead of calling LoadLibrary

TODO: remove new calls to LoadLibrary introduced by
https://codereview.chromium.org/2054273002/

BUG=
==========

to

==========
Link to DirectWrite directly, instead of calling LoadLibrary

BUG=
==========

[Ilya Kulshin, 2016-06-30 04:22:19]

Patchset #7 (id:140001) has been deleted

[Ilya Kulshin, 2016-06-30 16:44:37]

kulshin@chromium.org changed reviewers:
+ jam@chromium.org

[Ilya Kulshin, 2016-06-30 16:44:38]

+jam@ - could you review ui/gfx/* and give owners approval for
content/*/BUILD.gn

scottmg@ - one change since your l-g-t-m - in font_fallback_win.cc I deleted the
global IDWriteFactory and am now asking for a new one every time, since the
concern about calling LoadLibrary has been addressed.

[scottmg, 2016-06-30 17:02:46]

changes lgtm

[jam, 2016-06-30 22:24:01]

lgtm

[Ilya Kulshin, 2016-06-30 23:09:56]

The CQ bit was checked by kulshin@chromium.org

[commit-bot: I haz the power, 2016-06-30 23:10:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 23:42:06]

Description was changed from

==========
Link to DirectWrite directly, instead of calling LoadLibrary

BUG=
==========

to

==========
Link to DirectWrite directly, instead of calling LoadLibrary

BUG=
==========

[commit-bot: I haz the power, 2016-06-30 23:42:07]

Committed patchset #8 (id:180001)

[commit-bot: I haz the power, 2016-06-30 23:42:10]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-06-30 23:45:06]

Description was changed from

==========
Link to DirectWrite directly, instead of calling LoadLibrary

BUG=
==========

to

==========
Link to DirectWrite directly, instead of calling LoadLibrary

BUG=

Committed: https://crrev.com/7561063aa78a31c73506f666c9eeccf45fe3dc07
Cr-Commit-Position: refs/heads/master@{#403344}
==========

[commit-bot: I haz the power, 2016-06-30 23:45:07]

Patchset 8 (id:??) landed as
https://crrev.com/7561063aa78a31c73506f666c9eeccf45fe3dc07
Cr-Commit-Position: refs/heads/master@{#403344}

[Ilya Kulshin, 2016-07-01 01:00:32]

A revert of this CL (patchset #8 id:180001) has been created in
https://codereview.chromium.org/2117683002/ by kulshin@chromium.org.

The reason for reverting is: Breaks aura_demo target.