RESOLVED DUPLICATE of bug 109772 37297
[Chromium] need an implementation of LayoutTestController::computedStyleIncludingVisitedInfo
https://bugs.webkit.org/show_bug.cgi?id=37297
Summary [Chromium] need an implementation of LayoutTestController::computedStyleInclu...
Ojan Vafai
Reported 2010-04-08 16:17:50 PDT
http://trac.webkit.org/changeset/57292 I think we need to expose bool WebBindings::getElement(NPObject* range, WebElement* webElement) in order for TestShell's LayoutTestController to be able to get an Element out of the CPPArgumentList. Then it's just a matter of returning RefPtr<CSSComputedStyleDeclaration> style = computedStyle(element, true) as a V8Object.
Attachments
Ojan Vafai
Comment 1 2010-04-09 18:43:30 PDT
Nate, any chance I can convince you to take this on? You're much more familiar with V8 and bindings-related code than I am.
Nate Chapin
Comment 2 2010-04-12 16:00:09 PDT
I can't promise a solution by end of week, but otherwise, yes, you can convince me.
Darin Fisher (:fishd, Google)
Comment 3 2010-04-13 21:31:12 PDT
Instead of getElement, how about getNode since that will be more generically useful? Then, we can use WebNode::isElementNode to determine if a downcast to WebElement is safe.
Hajime Morrita
Comment 4 2011-04-06 20:30:30 PDT
Is it possible to have test-specific Node JS API? It looks like: interface Node ...{ ... [LayoutTest] String computedStyleIncludingVisitedInfo() ... } Using Node or Element with LayoutTestController is tedious especially for Qt because Qt uses QObject reflection to bind LayoutTestController into JS. Other possible way to move part of LayoutTestController inside WebCore. My last attempt didn't get enough attention though? https://bugs.webkit.org/show_bug.cgi?id=42612
Benjamin Poulain
Comment 5 2013-02-14 20:37:15 PST
This could be of interest to you: https://bugs.webkit.org/show_bug.cgi?id=109772 One more thing to fix and that should work for Chromium.
Benjamin Poulain
Comment 6 2013-02-14 20:37:29 PST
*** This bug has been marked as a duplicate of bug 109772 ***
Note You need to log in before you can comment on or make changes to this bug.