Roll out dominantScript, getGenericFontForScript patches to unfork a bunch of...

Roll out dominantScript, getGenericFontForScript patches to unfork a bunch of files, WebKit side.

These patches are now tracked upstream:
http://bugs.webkit.org/show_bug.cgi?id=20797
http://bugs.webkit.org/show_bug.cgi?id=10874
http://bugs.webkit.org/show_bug.cgi?id=18085

R=jshin
BUG=9768
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=13156

[jungshik at Google, 2009-04-02 21:20:58]

lgtm with the following unrelated changes removed (for an easier tracking
later). 

Also, can you add the following references to webkit bugs to the CL description
(again to help code archaelogists :-))?

http://bugs.webkit.org/show_bug.cgi?id=20797
http://bugs.webkit.org/show_bug.cgi?id=10874
http://bugs.webkit.org/show_bug.cgi?id=18085

And, make sure to rebaseline layout tests :-)

http://codereview.chromium.org/56198/diff/1/12
File third_party/WebKit/WebCore/css/CSSStyleSelector.cpp (left):

http://codereview.chromium.org/56198/diff/1/12#oldcode1516
Line 1516: style->setOverflowY(style->overflowY() == OVISIBLE ? OAUTO :
style->overflowY());
At least, this should be in a separate CL. This part is not related to the rest
of roll-out/unforking.

http://codereview.chromium.org/56198/diff/1/2
File third_party/WebKit/WebCore/dom/Document.cpp (left):

http://codereview.chromium.org/56198/diff/1/2#oldcode163
Line 163: #define WTF_PLATFORM_CHROMIUM 1
Again, this change is not related to the rest of changes. So, it'd better be in
a different CL, I think.

http://codereview.chromium.org/56198/diff/1/2
File third_party/WebKit/WebCore/dom/Document.cpp (right):

http://codereview.chromium.org/56198/diff/1/2#newcode4275
Line 4275: if (equalIgnoringCase(dnsPrefetchControl, "on") &&
!m_haveExplicitlyDisabledDNSPrefetch) {
same here.

[dglazkov, 2009-04-06 16:12:56]

Thanks for reviewing this! Done and committed.

http://codereview.chromium.org/56198/diff/1/12
File third_party/WebKit/WebCore/css/CSSStyleSelector.cpp (left):

http://codereview.chromium.org/56198/diff/1/12#oldcode1516
Line 1516: style->setOverflowY(style->overflowY() == OVISIBLE ? OAUTO :
style->overflowY());
On 2009/04/02 21:20:58, Jungshik Shin wrote:
> At least, this should be in a separate CL. This part is not related to the
rest
> of roll-out/unforking. 
> 

Done.

http://codereview.chromium.org/56198/diff/1/2
File third_party/WebKit/WebCore/dom/Document.cpp (left):

http://codereview.chromium.org/56198/diff/1/2#oldcode163
Line 163: #define WTF_PLATFORM_CHROMIUM 1
On 2009/04/02 21:20:58, Jungshik Shin wrote:
> Again, this change is not related to the rest of changes. So, it'd better be
in
> a different CL, I think.

Done.

http://codereview.chromium.org/56198/diff/1/2
File third_party/WebKit/WebCore/dom/Document.cpp (right):

http://codereview.chromium.org/56198/diff/1/2#newcode4275
Line 4275: if (equalIgnoringCase(dnsPrefetchControl, "on") &&
!m_haveExplicitlyDisabledDNSPrefetch) {
On 2009/04/02 21:20:58, Jungshik Shin wrote:
> same here. 

Done.