Bug 25059 - Remove references to dominantScript, getGenericFontForScript
Summary: Remove references to dominantScript, getGenericFontForScript
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Dimitri Glazkov (Google)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-06 11:21 PDT by Dimitri Glazkov (Google)
Modified: 2009-04-06 16:13 PDT (History)
1 user (show)

See Also:


Attachments
2009-04-06 Simon Fraser <simon.fraser@apple.com> (9.59 KB, patch)
2009-04-06 11:25 PDT, Dimitri Glazkov (Google)
no flags Details | Formatted Diff | Diff
WebCore: (5.43 KB, patch)
2009-04-06 11:25 PDT, Dimitri Glazkov (Google)
no flags Details | Formatted Diff | Diff
2009-04-06 Mike Belshe <mike@belshe.com> (5.05 KB, patch)
2009-04-06 11:25 PDT, Dimitri Glazkov (Google)
no flags Details | Formatted Diff | Diff
Remove references to dominantScript, getGenericFontForScript, v1 (10.13 KB, patch)
2009-04-06 11:25 PDT, Dimitri Glazkov (Google)
fishd: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitri Glazkov (Google) 2009-04-06 11:21:32 PDT
These forking patches were rolled out downstream in Chromium tree. See:

http://codereview.chromium.org/56198.
Comment 1 Dimitri Glazkov (Google) 2009-04-06 11:25:18 PDT
Created attachment 29281 [details]
2009-04-06  Simon Fraser  <simon.fraser@apple.com>


        Reviewed by Darin Adler

        https://bugs.webkit.org/show_bug.cgi?id=25052

        Pull the call to updateGraphicsLayerGeometry() out of updateGraphicsLayers(),
        because we need to call it at the end of rebuildCompositingLayerTree() once
        we've determined which descendant layers are composited, otherwise
        calculateCompositedBounds() can give the wrong answer.

        Now that updateLayerCompositingState() doesn't end up calling updateGraphicsLayerGeometry(),
        call that explicitly from styleChanged(), if we know a layer update is not pending,
        and similarly from updateAfterLayout().

        Test: compositing/overflow/overflow-positioning.html

        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::styleChanged):
        * rendering/RenderLayerBacking.cpp:
        (WebCore::RenderLayerBacking::updateAfterLayout):
        (WebCore::RenderLayerBacking::updateGraphicsLayers):
        (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
        * rendering/RenderLayerCompositor.cpp:
        (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@42246 268f45cc-cd09-0410-ab3c-d52691b4dbfc
---
 LayoutTests/ChangeLog                              |   11 +++
 .../compositing/overflow/overflow-positioning.html |   86 ++++++++++++++++++++
 .../overflow-positioning-expected.checksum         |    1 +
 .../overflow/overflow-positioning-expected.png     |  Bin 0 -> 16400 bytes
 .../overflow/overflow-positioning-expected.txt     |   14 +++
 WebCore/ChangeLog                                  |   26 ++++++
 WebCore/rendering/RenderLayer.cpp                  |    2 +
 WebCore/rendering/RenderLayerBacking.cpp           |   14 +---
 WebCore/rendering/RenderLayerCompositor.cpp        |    7 +-
 9 files changed, 149 insertions(+), 12 deletions(-)
Comment 2 Dimitri Glazkov (Google) 2009-04-06 11:25:20 PDT
Created attachment 29282 [details]
WebCore:


2009-04-06  Dave Moore  <davemoore@google.com>

        Reviewed by Dimitri Glazkov.

        https://bugs.webkit.org/show_bug.cgi?id=25031
        Make the V8 element collections check for named properties *before*
        checking if there are any elements with the same name. Otherwise
        it is both incorrect and slow.

        Test: fast/dom/HTMLSelectElement/length-not-overridden.html

        Both of these interceptors were attempting to find an element in the
        collection that had a name or id of the property name before checking
        for a JS property with that name.
        * bindings/v8/V8Collection.h:
        (WebCore::collectionNamedPropertyGetter):
        (WebCore::nodeCollectionNamedPropertyGetter):

LayoutTests:

2009-04-06  Dave Moore  <davemoore@google.com>

        Reviewed by Dimitri Glazkov.

        https://bugs.webkit.org/show_bug.cgi?id=25031
        Test for ensuring that named elements don't override properties on collections

        * fast/dom/HTMLSelectElement/length-not-overridden-expected.txt: Added.
        * fast/dom/HTMLSelectElement/length-not-overridden.html: Added.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@42247 268f45cc-cd09-0410-ab3c-d52691b4dbfc
---
 LayoutTests/ChangeLog                              |   10 +++++
 .../length-not-overridden-expected.txt             |    4 ++
 .../HTMLSelectElement/length-not-overridden.html   |   37 ++++++++++++++++++++
 WebCore/ChangeLog                                  |   18 +++++++++
 WebCore/bindings/v8/V8Collection.h                 |   18 +++++++++
 5 files changed, 87 insertions(+), 0 deletions(-)
Comment 3 Dimitri Glazkov (Google) 2009-04-06 11:25:23 PDT
Created attachment 29283 [details]
2009-04-06  Mike Belshe  <mike@belshe.com>


        Reviewed by Dimitri Glazkov.

        https://bugs.webkit.org/show_bug.cgi?id=24978
        Use custom implementations of a few Node methods so that
        we don't over-aggressively create JS wrappers here.

        * bindings/v8/custom/V8NodeCustom.cpp: Added custom implementations.
        * dom/Node.idl: Removed JSC-specifier, making impl custom for V8 as well.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@42248 268f45cc-cd09-0410-ab3c-d52691b4dbfc
---
 WebCore/ChangeLog                           |   11 ++++
 WebCore/bindings/v8/custom/V8NodeCustom.cpp |   74 +++++++++++++++++++++++++++
 WebCore/dom/Node.idl                        |    8 ++--
 3 files changed, 89 insertions(+), 4 deletions(-)
Comment 4 Dimitri Glazkov (Google) 2009-04-06 11:25:25 PDT
Created attachment 29284 [details]
Remove references to dominantScript, getGenericFontForScript, v1

 WebCore/ChangeLog                                  |   23 +++++++++++++++++++
 .../graphics/chromium/FontCacheChromiumWin.cpp     |   15 ------------
 .../platform/graphics/chromium/FontCacheLinux.cpp  |    7 -----
 WebCore/rendering/RenderThemeChromiumLinux.cpp     |    6 ++--
 WebCore/rendering/RenderThemeChromiumLinux.h       |    2 +-
 WebCore/rendering/RenderThemeChromiumMac.h         |    2 +-
 WebCore/rendering/RenderThemeChromiumMac.mm        |    3 +-
 WebCore/rendering/RenderThemeChromiumWin.cpp       |   24 +++----------------
 WebCore/rendering/RenderThemeChromiumWin.h         |    2 +-
 9 files changed, 34 insertions(+), 50 deletions(-)
Comment 5 Dimitri Glazkov (Google) 2009-04-06 11:35:23 PDT
Sorry for the git-send-bugzilla spam :)
Comment 6 Dimitri Glazkov (Google) 2009-04-06 16:13:41 PDT
Landed as http://trac.webkit.org/changeset/42260.