⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



May 9, 2010:

10:07 PM Changeset in webkit [59066] by tkent@chromium.org
  • 4 edits in trunk/LayoutTests

Unreviewed, test expectation update.

Update Chromium/Linux expectation for fast/forms/input-file-re-render.html.
​http://crbug.com/40027

  • platform/chromium-linux/fast/forms/input-file-re-render-expected.checksum:
  • platform/chromium-linux/fast/forms/input-file-re-render-expected.png:
  • platform/chromium/test_expectations.txt:
7:17 PM Changeset in webkit [59065] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Fix accidental repeat addition of emit_op_new_regexp

6:41 PM Changeset in webkit [59064] by oliver@apple.com
  • 14 edits
    2 moves
    1 add
    1 delete in trunk

2010-05-09 Geoffrey Garen <​ggaren@apple.com>

Reviewed by Oliver Hunt.

Reserve a large-ish initial capacity for Lexer::m_buffer16.

SunSpider says 0.3% faster.

m_buffer16 is used when parsing complex strings -- for example, strings
with escape sequences in them. These kinds of strings can be really long,
and we want to avoid repeatedly copying as we grow m_buffer16.

The net memory cost is quite low, since it's proporitional to source
code we already have in memory, and we throw away m_buffer16 right when
we're done parsing.

  • parser/Lexer.cpp: (JSC::Lexer::Lexer): No need to reserve initial capacity in our constructor, since setCode will be called before we're asked to lex anything. (JSC::Lexer::setCode): Reserve enough space to lex half the source code as a complex string without having to copy. (JSC::Lexer::clear): No need to reserve initial capacity here either, since setCode will be called before we're asked to lex anything.

2010-05-09 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

REGRESSION(r57955): RegExp literals should not actually be cached, so r57955 should be rolled out.
​https://bugs.webkit.org/show_bug.cgi?id=38828

Replace incorrect test for caching regexp literals with ones that tests that they are not cached.

  • fast/js/regexp-literals-are-constants-expected.txt: Removed.
  • fast/js/regexp-literals-are-constants.html: Removed.
  • fast/js/regexp-literals-arent-constants-expected.txt: Added.
  • fast/js/regexp-literals-arent-constants.html: Added.
  • fast/js/script-tests/regexp-literals-are-constants.js: Removed.
  • fast/js/script-tests/regexp-literals-arent-constants.js: Added. (test1): (returnRegExpLiteral): (returnConditionalRegExpLiteral):
3:09 PM Changeset in webkit [59063] by loislo@chromium.org
  • 3 edits in trunk/WebCore

2010-05-10 Ilya Tikhonovsky <​loislo@chromium.org>

Not reviewed: rollback the commit r59062

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails):
  • inspector/front-end/inspector.js: (WebInspector.linkifyURLAsNode):
2:38 PM Changeset in webkit [59062] by loislo@chromium.org
  • 3 edits in trunk/WebCore

2010-05-10 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: Cosmetic changes.
1) Inner Function Call event of Timer Fire should be merged into parent even if it is not the only child.
2) Standard tooltips for resource links are clashing with Popover.
3) Recalculate Style caller info are duplicating in Details Popover (Caller and Details rows).
​https://bugs.webkit.org/show_bug.cgi?id=38732

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.FormattedRecord.prototype._generatePopupContent): (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails):
  • inspector/front-end/inspector.js: (WebInspector.linkifyURLAsNode):
2:16 PM Changeset in webkit [59061] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Reserve a large-ish initial capacity for Lexer::m_buffer16.

Reviewed by Oliver Hunt.

SunSpider says 0.3% faster.

m_buffer16 is used when parsing complex strings -- for example, strings
with escape sequences in them. These kinds of strings can be really long,
and we want to avoid repeatedly copying as we grow m_buffer16.

The net memory cost is quite low, since it's proporitional to source
code we already have in memory, and we throw away m_buffer16 right when
we're done parsing.

  • parser/Lexer.cpp:

(JSC::Lexer::Lexer): No need to reserve initial capacity in our constructor,
since setCode will be called before we're asked to lex anything.
(JSC::Lexer::setCode): Reserve enough space to lex half the source code
as a complex string without having to copy.
(JSC::Lexer::clear): No need to reserve initial capacity here either,
since setCode will be called before we're asked to lex anything.

2:11 PM Changeset in webkit [59060] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-05-09 Laszlo Gombos <Laszlo Gombos>

Reviewed by Eric Seidel.

[Qt] Remove YARR configuration rules from Qt make system
​https://bugs.webkit.org/show_bug.cgi?id=38819

Setting YARR based on JIT seeting is now in Platform.h
for all ports. This patch essentially reverses r49238.

1:58 PM Changeset in webkit [59059] by loislo@chromium.org
  • 2 edits in trunk/WebCore

2010-05-08 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: If some record is not in visible area then it's expand element also is not
visible even if it has children in visible area.
​https://bugs.webkit.org/show_bug.cgi?id=38643

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._refreshRecords):
1:34 PM Changeset in webkit [59058] by weinig@apple.com
  • 5 edits in trunk/WebCore

Fix for ​https://bugs.webkit.org/show_bug.cgi?id=38557
r58526 introduced a ~30% regression on Dromaeo JS lib

Reviewed by Darin Adler.

This fix does two things.

  • Don't use QualifiedName as the key to a HashMap, use a RefPtr<QualifiedNameImpl> instead. We should remove the HashTraits for QualifiedName and that will happen in a follow up patch.
  • Only mark cached NodeLists on Documents instead of all Nodes. This is okay since the marking of NodeLists is an optimization to keep NodeList wrappers alive and is not mandated by any spec.
  • bindings/js/JSDocumentCustom.cpp:

(WebCore::JSDocument::markChildren):

  • bindings/js/JSNodeCustom.cpp:

(WebCore::JSNode::markChildren):

  • dom/Node.cpp:

(WebCore::Node::removeCachedTagNodeList):
(WebCore::Node::getElementsByTagNameNS):

  • dom/NodeRareData.h:
9:37 AM Changeset in webkit [59057] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-05-09 Noam Rosenthal <​noam.rosenthal@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Crash in QGraphicsWebViewPrivate::~QGraphicsWebViewPrivate when animation were used
​https://bugs.webkit.org/show_bug.cgi?id=38574

The fix uses a QWeakPointer for rootGraphicsLayer, protecting from a crash in case the layer is deleted before the QGraphicsWebView.

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::QGraphicsWebViewPrivate): (QGraphicsWebViewPrivate::~QGraphicsWebViewPrivate): (QGraphicsWebViewPrivate::setRootGraphicsLayer): (QGraphicsWebViewPrivate::updateCompositingScrollPosition):
4:42 AM Changeset in webkit [59056] by oliver@apple.com
  • 7 edits in trunk/JavaScriptCore

2010-05-09 Oliver Hunt <​oliver@apple.com>

Reviewed by Maciej Stachowiak.

Improve string indexing performance
​https://bugs.webkit.org/show_bug.cgi?id=38814

Add an assembly stub to do indexed loads from strings much
more cheaply than the current stub dispatch logic. We can
do this because we are able to make guarantees about the
register contents when entering the stub so the call overhead
is negligible.

  • jit/JIT.h:
  • jit/JITInlineMethods.h:
  • jit/JITOpcodes.cpp:
  • jit/JITPropertyAccess.cpp: (JSC::JIT::stringGetByValStubGenerator): (JSC::JIT::emitSlow_op_get_by_val):

Moved from JITOpcodes.cpp to keep the slowcase next to
the normal case codegen as we do for everything else.

  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::stringGetByValStubGenerator): (JSC::JIT::emitSlow_op_get_by_val):
  • jit/JSInterfaceJIT.h: (JSC::JSInterfaceJIT::emitFastArithImmToInt):
4:18 AM Changeset in webkit [59055] by mjs@apple.com
  • 3 edits in trunk/JavaScriptCore

2010-05-09 Maciej Stachowiak <​mjs@apple.com>

Fixed version of: "Optimized o[s] where o is a cell and s is a string"
​https://bugs.webkit.org/show_bug.cgi?id=38815


Fixed the previous patch for this from Geoff Garen.


The two problems were a missing exception check and a PropertySlot
initialized improperly, leading to crashes and failures in the case
of getters accessed with bracket syntax.

Previous patch:

Optimized o[s] where o is a cell and s is a string, removing some old
code that wasn't really tuned for the JIT.


SunSpider says 0.8% faster.

  • jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
  • runtime/JSCell.h:
3:24 AM Changeset in webkit [59054] by robert@webkit.org
  • 1 edit
    1 delete in trunk/LayoutTests

2010-05-09 Robert Hogan <​robert@webkit.org>

[Qt] Update expected results for unskipped editing/inserting tests

Unreviewed, remove platform-specific results for editing/inserting/6633727.html

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

  • platform/qt/editing/inserting/6633727-expected.txt: Removed.
3:15 AM Changeset in webkit [59053] by robert@webkit.org
  • 97 edits
    5 adds in trunk/LayoutTests

[Qt] Update expected results for unskipped editing/inserting tests

Unreviewed, add expected results for unskipped tests. Results are taken
from Qt bot.

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

2:52 AM Changeset in webkit [59052] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Unskip editing/inserting tests that fail due to x/y differences

2010-05-03 Robert Hogan <​robert@webkit.org>

Reviewed by Eric Seidel.

[Qt] Unskip editing/inserting tests that fail due to x/y differences

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

Unskip the following tests:

editing/inserting/12882.html
editing/inserting/4840662.html
editing/inserting/4875189-1.html
editing/inserting/4959067.html
editing/inserting/4960120-2.html
editing/inserting/5002441.html
editing/inserting/5058163-2.html
editing/inserting/5156401-2.html
editing/inserting/5418891.html
editing/inserting/5510537.html
editing/inserting/5549929-2.html
editing/inserting/5549929-3.html
editing/inserting/6633727.html
editing/inserting/6703873.html
editing/inserting/editable-inline-element.html
editing/inserting/insert-3654864-fix.html
editing/inserting/insert-3659587-fix.html
editing/inserting/insert-3775316-fix.html
editing/inserting/insert-3778059-fix.html
editing/inserting/insert-3786362-fix.html
editing/inserting/insert-3800346-fix.html
editing/inserting/insert-3851164-fix.html
editing/inserting/insert-3907422-fix.html
editing/inserting/insert-after-delete-001.html
editing/inserting/insert-at-end-01.html
editing/inserting/insert-at-end-02.html
editing/inserting/insert-before-link-1.html
editing/inserting/insert-br-001.html
editing/inserting/insert-br-002.html
editing/inserting/insert-br-003.html
editing/inserting/insert-br-004.html
editing/inserting/insert-br-005.html
editing/inserting/insert-br-006.html
editing/inserting/insert-br-007.html
editing/inserting/insert-br-008.html
editing/inserting/insert-br-009.html
editing/inserting/insert-br-at-tabspan-001.html
editing/inserting/insert-br-at-tabspan-002.html
editing/inserting/insert-br-at-tabspan-003.html
editing/inserting/insert-br-quoted-001.html
editing/inserting/insert-br-quoted-002.html
editing/inserting/insert-br-quoted-003.html
editing/inserting/insert-br-quoted-004.html
editing/inserting/insert-br-quoted-005.html
editing/inserting/insert-br-quoted-006.html
editing/inserting/insert-div-001.html
editing/inserting/insert-div-002.html
editing/inserting/insert-div-003.html
editing/inserting/insert-div-004.html
editing/inserting/insert-div-005.html
editing/inserting/insert-div-006.html
editing/inserting/insert-div-007.html
editing/inserting/insert-div-008.html
editing/inserting/insert-div-009.html
editing/inserting/insert-div-010.html
editing/inserting/insert-div-011.html
editing/inserting/insert-div-012.html
editing/inserting/insert-div-013.html
editing/inserting/insert-div-014.html
editing/inserting/insert-div-015.html
editing/inserting/insert-div-016.html
editing/inserting/insert-div-017.html
editing/inserting/insert-div-018.html
editing/inserting/insert-div-019.html
editing/inserting/insert-div-020.html
editing/inserting/insert-div-021.html
editing/inserting/insert-div-022.html
editing/inserting/insert-div-024.html
editing/inserting/insert-div-025.html
editing/inserting/insert-div-026.html
editing/inserting/insert-div-027.html
editing/inserting/insert-paragraph-01.html
editing/inserting/insert-paragraph-02.html
editing/inserting/insert-paragraph-03.html
editing/inserting/insert-paragraph-04.html
editing/inserting/insert-paragraph-05.html
editing/inserting/insert-space-in-empty-doc.html
editing/inserting/insert-tab-001.html
editing/inserting/insert-tab-002.html
editing/inserting/insert-tab-004.html
editing/inserting/insert-text-at-tabspan-001.html
editing/inserting/insert-text-at-tabspan-002.html
editing/inserting/insert-text-at-tabspan-003.html
editing/inserting/line-break.html
editing/inserting/paragraph-separator-01.html
editing/inserting/paragraph-separator-02.html
editing/inserting/paragraph-separator-in-table-1.html
editing/inserting/paragraph-separator-in-table-2.html
editing/inserting/redo.html
editing/inserting/return-key-with-selection-003.html
editing/inserting/typing-001.html
editing/inserting/typing-002.html
editing/inserting/typing-003.html
editing/inserting/typing-around-br-001.html
editing/inserting/typing-around-image-001.html

editing/inserting/5058163-1.html
editing/inserting/edited-whitespace-1.html
editing/inserting/paragraph-separator-03.html
editing/inserting/return-key-with-selection-001.html
editing/inserting/return-key-with-selection-002.html
editing/inserting/4875189-2.html

  • platform/qt/Skipped:
2:50 AM Changeset in webkit [59051] by dbates@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-09 Daniel Bates <​dbates@rim.com>

Reviewed by Chris Jerdonek.

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

Make the regular expressions for parsing the start of an SVN
and Git header global variables since they are used throughout
VCSUtils.pm.

  • Scripts/VCSUtils.pm:
2:08 AM Changeset in webkit [59050] by dbates@webkit.org
  • 3 edits in trunk

2010-05-07 Daniel Bates <​dbates@rim.com>

Unreviewed, change log fix.

Add back some entries inadvertently deleted in changeset 59000
<​http://trac.webkit.org/changeset/59000> and not restored (as
intended) in changeset 59005 <​http://trac.webkit.org/changeset/59005>.

1:42 AM Changeset in webkit [59049] by mjs@apple.com
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Add back some entries accidentally deleted in r59000 and not restored as intended in r59005.

12:21 AM Changeset in webkit [59048] by Chris Jerdonek
  • 7 edits in trunk/WebKitTools

2010-05-09 Chris Jerdonek <Chris Jerdonek>

Reviewed by Daniel Bates.

Finished moving the header-parsing logic from svn-apply and -unapply
to VCSUtils.pm's parsing methods.

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

  • Scripts/VCSUtils.pm:
    • Added to parseGitDiffHeader() the ability to parse and store whether a file is new or deleted.
    • Also reordered in parseGitDiffHeader() some of the else statements to a more readable ordering.
    • Added to parseSvnDiffHeader() the ability to parse and store whether a file is new.
  • Scripts/svn-apply:
    • Changed handleGitBinaryChange() to use the new "isNew" and "isDeletion" diffHash key-values.
    • Changed patch() to use the new "isNew" diffHash key-value.
  • Scripts/svn-unapply:
    • Changed patch() to use the new "isNew" and "isDeletion" diffHash key-values.
  • Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
    • Added unit tests for new and deleted files.
  • Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
    • Updated the unit tests as necessary.
    • Added a unit test for a deleted file.
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:
    • Updated the unit tests as necessary.

May 8, 2010:

11:50 PM Changeset in webkit [59047] by Chris Jerdonek
  • 4 edits in trunk/BugsSite

2010-05-08 Chris Jerdonek <Chris Jerdonek>

Reviewed by Maciej Stachowiak.

Allow long comment lines to wrap to the window width instead
of to 80 characters.

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

  • Bugzilla/Constants.pm:
    • Introduced a new constant COMMENT_COLS_WRAP to replace COMMENT_COLS inside the wrap_comment() subroutine. The new constant effectively disables text wrapping by increasing the value used in wrap_comment() from 80 characters to a very large value (8000).
  • Bugzilla/Util.pm:
    • Replaced COMMENT_COLS with COMMENT_COLS_WRAP inside wrap_comment().
  • skins/custom/global.css: (.bz_comment_text):
    • Eliminate the width styling on comments to allow long lines to wrap at the window width.

(.bz_comment_text > span.quote):

  • Inherit the "white-space" property. This prevents long quoted lines in comments from causing non-quoted lines to extend beyond the window width.
8:21 PM Changeset in webkit [59046] by tonikitoo@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Antonio Gomes <​tonikitoo@webkit.org>

Unreviewed attempt to fix Windows build.

  • page/SpatialNavigation.h: (WebCore::fudgeFactor):
8:05 PM Changeset in webkit [59045] by tonikitoo@webkit.org
  • 3 edits in trunk/WebCore

Spatial Navigation: create a getter for the "fudgeFactor"
​https://bugs.webkit.org/show_bug.cgi?id=38488

Reviewed by Kenneth Christiansen.
Patch by Antonio Gomes <​tonikitoo@webkit.org>

A couple of places in the Spatial Navigation code make use of a "fudge factor"
to improve precision by working around outline focus metrics and such. Patch adds
a helper method for unify getter operations of this value, instead of having it
declared locally in the various methods it is used.

No behaviour change.

  • page/SpatialNavigation.cpp:

(WebCore::scrollIntoView):
(WebCore::deflateIfOverlapped):

  • page/SpatialNavigation.h:

(WebCore::fudgeFactor):

7:52 PM Changeset in webkit [59044] by Chris Jerdonek
  • 7 edits in trunk/WebKitTools

Added to svn-apply support for git renames and copies with similarity
index less than 100%.

Reviewed by Daniel Bates.

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

  • Scripts/VCSUtils.pm:
    • Added to parseGitDiffHeader() support for renames and similarity index less than 100%.
    • Added to parseDiff() support for processing renames, renames with changes, and copies with changes.
    • Added to parsePatch() the ability to process multiple return values from parseDiff().
  • Scripts/svn-apply:
    • Added to patch() the ability to process diff hashes with the isDeletion key-value set.
  • Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
    • Updated the unit tests as necessary.
    • Added unit tests for rename with similarity index 100%, rename with similarity index < 100%, and rename with a change to the executable bit.
  • Scripts/webkitperl/VCSUtils_unittest/parseGitDiffHeader.pl:
    • Added unit tests for rename with similarity index 100%, rename with similarity index < 100%, and rename with a change to the executable bit.
  • Scripts/webkitperl/VCSUtils_unittest/parsePatch.pl:
    • Updated the unit tests as necessary.
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnDiffHeader.pl:
    • Simplified the carriage-return unit test to more narrowly test only carriage returns.
7:37 PM Changeset in webkit [59043] by Chris Jerdonek
  • 4 edits in trunk/WebKitTools

This revision suppresses the misleading "error: pathspec..." messages
when using svn-apply to add a binary file from a Git diff.

Reviewed by Daniel Bates.

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

When adding a new binary file from a Git diff, svn-apply prints
a misleading error of the form -- "error: pathspec '<filename>' did
not match any file(s) known to git. Did you forget to 'git add'?"
This patch suppresses these messages since they are normal.

  • Scripts/VCSUtils.pm: Added the callSilently() subroutine from runPatchCommand.pl, which executes a Perl function while suppressing STDERR.
  • Scripts/svn-apply: Refactored the Git portion of scmKnowsOfFile() into a gitKnowsOfFile(), and called this new subroutine using callSilently().
  • Scripts/webkitperl/VCSUtils_unittest/runPatchCommand.pl: Removed callSilently() since it was moved to VCSUtils.pm in this patch.
7:02 PM Changeset in webkit [59042] by Laszlo Gombos
  • 2 edits in trunk/JavaScriptCore

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Unreviewed, build fix.

Fix missing terminating ' character warning

The warning has been only seen on the Gtk buildbots.

  • wtf/Platform.h:
5:38 PM Changeset in webkit [59041] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Gavin Barraclough.

Refactor YARR and YARR_JIT default rules in Platform.h
​https://bugs.webkit.org/show_bug.cgi?id=38727

Turn on YARR and YARR JIT by default of JIT is enabled.
Ports can overrule this default decisions as they wish.

  • wtf/Platform.h:
3:59 PM Changeset in webkit [59040] by oliver@apple.com
  • 7 edits
    1 add in trunk/JavaScriptCore

2010-05-08 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Split JSVALUE32_64 code out of JITOpcodes.cpp and into JITOpcodes32_64.cpp
​https://bugs.webkit.org/show_bug.cgi?id=38808

  • GNUmakefile.am:
  • JavaScriptCore.gypi:
  • JavaScriptCore.pro:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • jit/JITOpcodes.cpp:
  • jit/JITOpcodes32_64.cpp: Added. (JSC::JIT::privateCompileCTIMachineTrampolines): (JSC::JIT::emit_op_mov): (JSC::JIT::emit_op_end): (JSC::JIT::emit_op_jmp): (JSC::JIT::emit_op_loop_if_lesseq): (JSC::JIT::emitSlow_op_loop_if_lesseq): (JSC::JIT::emit_op_new_object): (JSC::JIT::emit_op_instanceof): (JSC::JIT::emitSlow_op_instanceof): (JSC::JIT::emit_op_new_func): (JSC::JIT::emit_op_get_global_var): (JSC::JIT::emit_op_put_global_var): (JSC::JIT::emit_op_get_scoped_var): (JSC::JIT::emit_op_put_scoped_var): (JSC::JIT::emit_op_tear_off_activation): (JSC::JIT::emit_op_tear_off_arguments): (JSC::JIT::emit_op_new_array): (JSC::JIT::emit_op_resolve): (JSC::JIT::emit_op_to_primitive): (JSC::JIT::emitSlow_op_to_primitive): (JSC::JIT::emit_op_strcat): (JSC::JIT::emit_op_resolve_base): (JSC::JIT::emit_op_resolve_skip): (JSC::JIT::emit_op_resolve_global): (JSC::JIT::emitSlow_op_resolve_global): (JSC::JIT::emit_op_not): (JSC::JIT::emitSlow_op_not): (JSC::JIT::emit_op_jfalse): (JSC::JIT::emitSlow_op_jfalse): (JSC::JIT::emit_op_jtrue): (JSC::JIT::emitSlow_op_jtrue): (JSC::JIT::emit_op_jeq_null): (JSC::JIT::emit_op_jneq_null): (JSC::JIT::emit_op_jneq_ptr): (JSC::JIT::emit_op_jsr): (JSC::JIT::emit_op_sret): (JSC::JIT::emit_op_eq): (JSC::JIT::emitSlow_op_eq): (JSC::JIT::emit_op_neq): (JSC::JIT::emitSlow_op_neq): (JSC::JIT::compileOpStrictEq): (JSC::JIT::emit_op_stricteq): (JSC::JIT::emitSlow_op_stricteq): (JSC::JIT::emit_op_nstricteq): (JSC::JIT::emitSlow_op_nstricteq): (JSC::JIT::emit_op_eq_null): (JSC::JIT::emit_op_neq_null): (JSC::JIT::emit_op_resolve_with_base): (JSC::JIT::emit_op_new_func_exp): (JSC::JIT::emit_op_throw): (JSC::JIT::emit_op_get_pnames): (JSC::JIT::emit_op_next_pname): (JSC::JIT::emit_op_push_scope): (JSC::JIT::emit_op_pop_scope): (JSC::JIT::emit_op_to_jsnumber): (JSC::JIT::emitSlow_op_to_jsnumber): (JSC::JIT::emit_op_push_new_scope): (JSC::JIT::emit_op_catch): (JSC::JIT::emit_op_jmp_scopes): (JSC::JIT::emit_op_switch_imm): (JSC::JIT::emit_op_switch_char): (JSC::JIT::emit_op_switch_string): (JSC::JIT::emit_op_new_error): (JSC::JIT::emit_op_debug): (JSC::JIT::emit_op_enter): (JSC::JIT::emit_op_enter_with_activation): (JSC::JIT::emit_op_create_arguments): (JSC::JIT::emit_op_init_arguments): (JSC::JIT::emit_op_convert_this): (JSC::JIT::emitSlow_op_convert_this): (JSC::JIT::emit_op_profile_will_call): (JSC::JIT::emit_op_profile_did_call):
1:47 PM Changeset in webkit [59039] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Stuart Morgan <​stuartmorgan@chromium.org>

Reviewed by Darin Adler.

Change the npapi.h header guards to match the npapi-headers repository
copy.

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

  • bridge/npapi.h:
1:03 PM Changeset in webkit [59038] by eric@webkit.org
  • 3 edits in trunk/JavaScriptCore

2010-05-08 Gabor Loki <​loki@webkit.org>

Reviewed by Gavin Barraclough.

Fix halfword loads on ARM
​https://bugs.webkit.org/show_bug.cgi?id=38741

The BaseIndex and ImplicitAddress are contain 32bit wide offset, but
the load16 functions were working with 8 bit data (encoded in the
instruction). If the offset cannot be encoded in an instruction, it
should be stored in a temporary register.

  • assembler/ARMAssembler.h: (JSC::ARMAssembler::getOffsetForHalfwordDataTransfer):
  • assembler/MacroAssemblerARM.h: (JSC::MacroAssemblerARM::load16):
12:51 PM Changeset in webkit [59037] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-05-08 Gabor Loki <​loki@webkit.org>

Reviewed by Gavin Barraclough.

Fix spanning branch instruction on Cortex-A8 with Thumb-2 JIT
​https://bugs.webkit.org/show_bug.cgi?id=38280

If the 32-bit Thumb-2 branch instruction spans two 4KiB regions and
the target of the branch falls within the first region it is
possible for the processor to incorrectly determine the branch
instruction, and it is also possible in some cases for the processor
to enter a deadlock state.

  • assembler/ARMv7Assembler.h: (JSC::ARMv7Assembler::linkJumpAbsolute):
12:36 PM Changeset in webkit [59036] by eric@webkit.org
  • 2 edits
    1 add in trunk/WebKitTools

2010-05-08 Victor Wang <​victorw@chromium.org>

Reviewed by Adam Barth.

[chromium]: Upload test results json files to app engine server
Add an option to run_webkit_tests.py to upload generated
JSON files to app engine server. These JSON files will be used
by chromium layout test falkiness dashboard.

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

  • Scripts/webkitpy/layout_tests/layout_package/test_results_uploader.py: Added.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
12:23 PM Changeset in webkit [59035] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-08 Stuart Morgan <​stuartmorgan@chromium.org>

Reviewed by Darin Fisher.

Resolve most of the whitespace/comment/order differences in npapi.h
and npruntime.h relative to the npapi-headers repository versions.
No changes to the substance of the code.

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

  • bridge/npapi.h: (_NPPrint::): (_NPCocoaEvent::):
  • bridge/npruntime.h:
12:05 PM Changeset in webkit [59034] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Darin Adler.

Fix guard around IndexedDatabase
​https://bugs.webkit.org/show_bug.cgi?id=38710

No new tests, there is no new functionality.

  • page/PageGroup.h:
11:51 AM Changeset in webkit [59033] by eric@webkit.org
  • 6 edits
    3 adds in trunk

2010-05-08 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Simon Hausmann.

[Qt] Platform plugin
​https://bugs.webkit.org/show_bug.cgi?id=38438

Select popups delegate private API. To provide select popup delegates
a plugin that implements QWebKitPlatformPlugin must be found
in QCoreApplication::libraryPaths.

  • WebCore.pro:

2010-05-08 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Simon Hausmann.

[Qt] Platform plugin
​https://bugs.webkit.org/show_bug.cgi?id=38438

Select popups delegate private API. To provide select popup delegates
a plugin that implements QWebKitPlatformPlugin must be found
in QCoreApplication::libraryPaths.

  • Api/headers.pri:
  • Api/qwebkitplatformplugin.h: Added. (QWebSelectData::~QWebSelectData): (QWebSelectData::): (QWebSelectMethod::~QWebSelectMethod): (QWebKitPlatformPlugin::~QWebKitPlatformPlugin): (QWebKitPlatformPlugin::):
  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::createSelectPopup):
  • WebCoreSupport/ChromeClientQt.h:
  • WebCoreSupport/QtPlatformPlugin.cpp: Added. (WebCore::SelectData::SelectData): (WebCore::SelectData::itemText): (WebCore::SelectData::itemToolTip): (WebCore::SelectData::itemIsEnabled): (WebCore::SelectData::itemCount): (WebCore::SelectData::itemIsSelected): (WebCore::SelectData::multiple): (WebCore::SelectData::itemType): (WebCore::SelectInputMethodWrapper::SelectInputMethodWrapper): (WebCore::SelectInputMethodWrapper::show): (WebCore::SelectInputMethodWrapper::hide): (WebCore::SelectInputMethodWrapper::selectItem): (WebCore::SelectInputMethodWrapper::didHide): (WebCore::getPluginObject): (WebCore::QtPlatformPlugin::~QtPlatformPlugin): (WebCore::QtPlatformPlugin::plugin): (WebCore::QtPlatformPlugin::createSelectInputMethod):
  • WebCoreSupport/QtPlatformPlugin.h: Added. (WebCore::QtPlatformPlugin::QtPlatformPlugin):
11:40 AM Changeset in webkit [59032] by eric@webkit.org
  • 2 edits in trunk/WebKit

2010-05-08 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Laszlo Gombos.

[EFL] Only compile database stuff if ENABLE_DATABASE is set.
​http://webkit.org/b/38777

  • efl/WebCoreSupport/ChromeClientEfl.cpp:
11:29 AM Changeset in webkit [59031] by eric@webkit.org
  • 4 edits in trunk

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Build the ImageDiff tool for all platforms including Windows and Symbian
​https://bugs.webkit.org/show_bug.cgi?id=38706

  • WebKit.pro:

2010-05-08 Laszlo Gombos <Laszlo Gombos>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Build the ImageDiff tool for all platforms including Windows and Symbian
​https://bugs.webkit.org/show_bug.cgi?id=38706

Use qmath.h instead of math.h to make it portable.

  • DumpRenderTree/qt/ImageDiff.cpp: (main):
11:17 AM Changeset in webkit [59030] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

[EFL] It does not make sense to use RenderThemeEfl::RenderThemeEfl, so
use RenderThemeEfl (GCC 4.5 seems to be pickier about this).
​http://webkit.org/b/38770

  • platform/efl/RenderThemeEfl.cpp: (WebCore::renderThemeEflColorClassSelectionActive): (WebCore::renderThemeEflColorClassSelectionInactive): (WebCore::renderThemeEflColorClassFocusRing): (WebCore::renderThemeEflColorClassButtonText): (WebCore::renderThemeEflColorClassComboText): (WebCore::renderThemeEflColorClassEntryText): (WebCore::renderThemeEflColorClassSearchText): (WebCore::RenderThemeEfl::adjustCheckboxStyle): (WebCore::RenderThemeEfl::adjustRadioStyle): (WebCore::RenderThemeEfl::adjustButtonStyle): (WebCore::RenderThemeEfl::adjustMenuListStyle): (WebCore::RenderThemeEfl::adjustTextFieldStyle): (WebCore::RenderThemeEfl::adjustSearchFieldDecorationStyle): (WebCore::RenderThemeEfl::adjustSearchFieldResultsButtonStyle): (WebCore::RenderThemeEfl::adjustSearchFieldResultsDecorationStyle): (WebCore::RenderThemeEfl::adjustSearchFieldCancelButtonStyle): (WebCore::RenderThemeEfl::adjustSearchFieldStyle):
11:06 AM Changeset in webkit [59029] by eric@webkit.org
  • 4 edits in trunk/WebKit/chromium

2010-05-08 Jens Alfke <​snej@chromium.org>

Reviewed by Darin Fisher.

[chromium] WebFrame::registerPasswordListener shouldn't assert on duplicate listener.
​https://bugs.webkit.org/show_bug.cgi?id=38765

  • public/WebFrame.h: (WebKit::WebFrameImpl::registerPasswordListener): Changed return type to bool. Updated comment.
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::registerPasswordListener): Return false instead of asserting.
  • src/WebFrameImpl.h: (WebKit::WebFrameImpl::registerPasswordListener): Changed return type to bool.
10:53 AM Changeset in webkit [59028] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

ResourceHandleSoup should depend only on GLib, not on GTK+.
​http://webkit.org/b/38763

  • platform/network/soup/ResourceHandleSoup.cpp: Include glib.h instead of gtk/gtk.h.
9:49 AM Changeset in webkit [59027] by eric@webkit.org
  • 13 edits
    1 delete in trunk/WebCore

2010-05-08 Eric Seidel <​eric@webkit.org>

Unreviewed, rolling out r59023.
​http://trac.webkit.org/changeset/59023
​https://bugs.webkit.org/show_bug.cgi?id=37874

Caused Chromium Windows build to fail to link.

  • WebCore.gypi:
  • loader/CachedMetadata.h: Removed.
  • loader/CachedResource.cpp:
  • loader/CachedResource.h:
  • loader/ResourceLoader.h:
  • loader/SubresourceLoader.cpp:
  • loader/SubresourceLoader.h:
  • loader/SubresourceLoaderClient.h:
  • loader/loader.cpp:
  • loader/loader.h:
  • platform/network/ResourceHandle.cpp:
  • platform/network/ResourceHandle.h:
  • platform/network/ResourceHandleClient.h:
6:31 AM Changeset in webkit [59026] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-05-08 Jocelyn Turcotte <​jocelyn.turcotte@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Document a limitation of QWebPage::linkClicked
​https://bugs.webkit.org/show_bug.cgi?id=37694

  • Api/qwebpage.cpp:
6:20 AM Changeset in webkit [59025] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Autocompletion in console is not usable because it is not add to the entered text the common prefix of suggestions.
As example if you entered $0.getE then getElementByClassName will be suggested as gray text.
If you press tab then getElementByClassName be suggested again and lementByClassName becomes selected as black text with selection background.
Really after pressing the TAB the next suggestion should be selected (getElementByTagName) and only flexible part of suggestions should be selected.
In this case it should be TagName because getElementBy is the common part for all getE prefix suggestions.
​https://bugs.webkit.org/show_bug.cgi?id=38753

  • inspector/front-end/TextPrompt.js: (WebInspector.TextPrompt.prototype._completionsReady):
6:09 AM Changeset in webkit [59024] by eric@webkit.org
  • 1 edit
    1 add in trunk/JavaScriptCore

2010-05-08 Adam Barth <​abarth@webkit.org>

Reviewed by Darin Fisher.

[WTFURL] Add a buffer for managing the output of the canonicalizer
​https://bugs.webkit.org/show_bug.cgi?id=38671

The canonicalizer produces a stream of canonicalized characters, which
we will store in a URLBuffer. URLBuffer is quite similar to Vector,
but serves as an abstraction to isolate us from the underlying
character storage. In the case of WebKit, we'll probably implement a
subclass of URLBuffer with Vector. In Chromium, the backing store is a
std::string, which avoids an unnecessary memcpy that would be caused by
using Vector instead.

  • wtf/url/src/URLBuffer.h: Added. (WTF::URLBuffer::URLBuffer): (WTF::URLBuffer::~URLBuffer): (WTF::URLBuffer::at): (WTF::URLBuffer::set): (WTF::URLBuffer::capacity): (WTF::URLBuffer::length): (WTF::URLBuffer::data): (WTF::URLBuffer::setLength): (WTF::URLBuffer::append): (WTF::URLBuffer::grow):
5:58 AM Changeset in webkit [59023] by eric@webkit.org
  • 13 edits
    1 add in trunk/WebCore

2010-05-08 Tony Gentilcore <​tonyg@chromium.org>

Reviewed by Darin Fisher.

Provide mechanism to store cached metadata on a CachedResource.
​https://bugs.webkit.org/show_bug.cgi?id=37874

No new tests because no new functionality.

  • WebCore.gypi:
  • loader/CachedMetadata.h: Added. (WebCore::CachedMetadata::~CachedMetadata): (WebCore::CachedMetadata::data): (WebCore::CachedMetadata::size): (WebCore::CachedMetadata::create): (WebCore::CachedMetadata::deserialize): (WebCore::CachedMetadata::serialize): (WebCore::CachedMetadata::dataTypeID): (WebCore::CachedMetadata::readUnsigned): (WebCore::CachedMetadata::appendUnsigned): (WebCore::CachedMetadata::CachedMetadata):
  • loader/CachedResource.cpp: (WebCore::CachedResource::setSerializedCachedMetadata): (WebCore::CachedResource::setCachedMetadata): (WebCore::CachedResource::cachedMetadata):
  • loader/CachedResource.h:
  • loader/ResourceLoader.h: (WebCore::ResourceLoader::didReceiveCachedMetadata):
  • loader/SubresourceLoader.cpp: (WebCore::SubresourceLoader::didReceiveCachedMetadata):
  • loader/SubresourceLoader.h:
  • loader/SubresourceLoaderClient.h: (WebCore::SubresourceLoaderClient::didReceiveCachedMetadata):
  • loader/loader.cpp: (WebCore::Loader::Host::didReceiveCachedMetadata):
  • loader/loader.h:
  • platform/network/ResourceHandle.cpp: (WebCore::ResourceHandle::cacheMetadata):
  • platform/network/ResourceHandle.h:
  • platform/network/ResourceHandleClient.h: (WebCore::ResourceHandleClient::didReceiveCachedMetadata):
5:39 AM Changeset in webkit [59022] by eric@webkit.org
  • 9 edits in trunk

2010-05-08 Michael Nordman <Michael Nordman>

Reviewed by Dmitry Titov.

Have WorkerScriptLoader provide access to the ResourceResponse for the loaded script
through the WorkerScriptLoaderClient interface.
​https://bugs.webkit.org/show_bug.cgi?id=38605

No new tests. There are no script visible changes.

  • workers/WorkerScriptLoader.cpp: Call the client with the response. (WebCore::WorkerScriptLoader::didReceiveResponse):
  • workers/WorkerScriptLoaderClient.h: Define a new method to receive the response. (WebCore::WorkerScriptLoaderClient::didReceiveResponse):

2010-05-08 Michael Nordman <Michael Nordman>

Reviewed by Dmitry Titov.

Add an additional parameter to the WebSharedWorker::startWorkerContext method
to indicate which appcache the shared worker script resource was loaded from,
and provide that value when calling out to chrome via this interface.
​https://bugs.webkit.org/show_bug.cgi?id=38605

I'm keeping the old method around until patches can land in both repositories
that make the old method obsolete and unneeded. There's a FIXME for that.

Also somewhat related but independent change to propagate the runtime enabled
flag thru to the page settings instance used by the 'shadow page'.

  • public/WebSharedWorker.h: Add the new param (but keep the old interface around for now too) (WebKit::WebSharedWorker::startWorkerContext):
  • src/SharedWorkerRepository.cpp: Provide the value when calling out to chrome. (WebCore::SharedWorkerScriptLoader::notifyFinished):
  • src/WebSharedWorkerImpl.cpp: (WebKit::WebSharedWorkerImpl::startWorkerContext):
  • src/WebSharedWorkerImpl.h:
  • src/WebWorkerBase.cpp: Propagate runtime enabled flag thru to the page settings. (WebKit::WebWorkerBase::initializeLoader):
5:13 AM Changeset in webkit [59021] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

2010-05-08 Robert Hogan <​robert@webkit.org>

Unreviewed Qt bot fix: test unskipped accidentally in r59020.

  • platform/qt/Skipped: Uncommented http/tests/xmlhttprequest/failed-auth.html again.
4:50 AM Changeset in webkit [59020] by robert@webkit.org
  • 7 edits
    1 add in trunk

[Qt] Fix http/tests/xmlhttprequest/cross-origin-no-authorization.html

2010-05-08 Robert Hogan <​robert@roberthogan.net>

Reviewed by Simon Hausmann.

[Qt] Fix http/tests/xmlhttprequest/cross-origin-no-authorization.html

and http/tests/xmlhttprequest/cross-origin-authorization.html

QHttpNetworkRequest adds Authorization and Cookie headers to XHRs
without knowing if this is valid behaviour or not. In order to allow
Qt to decide whether Cookie/Authorization headers should be added
to an XHR QtWebKit needs to use an attribute added to QNetworkRequest.
These new attributes are: QNetworkRequest::CookieLoadControlAttribute,
QNetworkRequest::CookieSaveControlAttribute,and
QNetworkRequest::AuthenticationReuseControlAttribute.

QtWebKit will set QNetworkRequest::AuthenticationReuseControlAttribute to false
unless withCredentials is set to true in the XHR.

QtWebkit will set CookieLoad/SaveControlAttribute to false unless withCredentials
is set to true in the XHR.

Qt will pass the values onto QHttpNetworkRequest and this will permit
the Qt network access processing to decide whether or not to add either
or both of the Cookie/Authorisation headers, and whether to save cookies
returned from such requests. By default the attribute
will always be true so unless QtWebKit sets it to false normal
header processing is unaffected.

There are platform-specific results for cross-origin-no-authorization.html:

  1. Qt's DRT does not capture the final 'DONE'.
  2. QNetworkReplyHandler.cpp's ignoreHttpError() means that async requests do not execute didFail() when they receive a 401 HTTP response. This results in async tests passing with '401 Authorisation required' rather than 'received error event'.

The Qt part of these changes was merged at: ​http://gitorious.org/qt/qt/merge_requests/592

In order to properly support the tests, Qt's DRT needs to use one
NetworkAccessManager for all pages. This allows it to use cached
credentials where appropriate.

The tests now pass when run individually but there seems to be a problem with
leaking the results of requests across tests when run with the others in
http/tests. This will be addressed in a separate patch.

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

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::WebPage): (WebCore::DumpRenderTree::DumpRenderTree):
  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • platform/network/qt/ResourceRequestQt.cpp: (WebCore::ResourceRequest::toNetworkRequest):
  • platform/qt/Skipped: Group together the tests that are waiting for 4.7 to be unskipped.
  • platform/qt/http/tests/xmlhttprequest/cross-origin-no-authorization-expected.txt: Added.
4:48 AM Changeset in webkit [59019] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInsector: Record expand UI element should be extracted from TimelineRecordGraphRow as separate class for future reuse.
​https://bugs.webkit.org/show_bug.cgi?id=38726

4:08 AM Changeset in webkit [59018] by Nikolas Zimmermann
  • 2 edits in trunk/LayoutTests

2010-05-08 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Forgot to rebase fast/dom/prototype-inheritance-2.html after the <vkern> patches.

  • platform/win/fast/dom/prototype-inheritance-2-expected.txt:
4:01 AM Changeset in webkit [59017] by robert@webkit.org
  • 4 edits in trunk

[Qt] unskip http/tests/plugins/npapi-response-headers.html

2010-05-04 Robert Hogan <​robert@webkit.org>

Reviewed by Eric Seidel.

[Qt] unskip http/tests/plugins/npapi-response-headers.html

Turns out this failed because run-webkit-tests was eating the first occurrence
of 'Content-Type: text/plain' in the test output as a header. Strange but true.
So do as Chromium does and preface all text dumps with the
'Content-Type: text/plain' header.

Take this opportunity to unskip plugins/private-browsing-mode.html too, which
has passed since r55433. See ​https://bugs.webkit.org/show_bug.cgi?id=33180
and ​https://bugs.webkit.org/show_bug.cgi?id=35576.

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

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::dump):
  • platform/qt/Skipped:
3:55 AM Changeset in webkit [59016] by robert@webkit.org
  • 4 edits in trunk

[Qt] Fix encoding of status message in Plugin API

2010-05-05 Robert Hogan <​robert@webkit.org>

Reviewed by Alexey Proskuryakov.

[Qt] Fix encoding of status message in Plugin API

Spotted by Alexey Proskuryakov.

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

  • platform/qt/Skipped: Unskip plugins/set-status.html
  • plugins/PluginView.cpp: (WebCore::PluginView::status):
3:11 AM Changeset in webkit [59015] by krit@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Dirk Schulze <​krit@webkit.org>

Unreviewed win build fix.

JS bindings were missing in DerivedSources.cpp

  • DerivedSources.cpp:
3:01 AM Changeset in webkit [59014] by Nikolas Zimmermann
  • 4 edits in trunk/LayoutTests

2010-05-08 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Rebaseline Win specific results after exposing JSSVGVKernElement/JSSVGHKernElement.

  • platform/win/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/win/fast/dom/prototype-inheritance-expected.txt:
  • platform/win/fast/js/global-constructors-expected.txt:
2:56 AM Changeset in webkit [59013] by krit@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Dirk Schulze <​krit@webkit.org>

Uneviewed fix of Win bot

Added JS bindings to Win build system.

  • WebCore.vcproj/WebCore.vcproj:
2:54 AM Changeset in webkit [59012] by Nikolas Zimmermann
  • 4 edits in trunk/LayoutTests

2010-05-08 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Rebaseline GTK specific results after exposing JSSVGVKernElement/JSSVGHKernElement.

  • platform/gtk/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/gtk/fast/dom/prototype-inheritance-expected.txt:
  • platform/gtk/fast/js/global-constructors-expected.txt:
2:28 AM Changeset in webkit [59011] by krit@webkit.org
  • 2 edits in trunk/WebCore

2010-05-08 Dirk Schulze <​krit@webkit.org>

Unreviewed try to fix Win bot.

SVGFont's VKern implementation missing
​https://bugs.webkit.org/show_bug.cgi?id=38663

  • bindings/scripts/CodeGeneratorJS.pm:
2:06 AM Changeset in webkit [59010] by krit@webkit.org
  • 31 edits
    7 adds in trunk

2010-05-08 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGFont's VKern implementation missing
​https://bugs.webkit.org/show_bug.cgi?id=38663

Implementation of vkern, needed by SVGFont. Also added DOM and JS bindings
for vkern and hkern. Made some clean-up in SVGParserUtilities and SVGFontElement.
SVGHKernElement and SVGVKernElement share most of the code now.

Test: svg/text/text-vkern.svg

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • Android.mk:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/DOMWindow.idl:
  • rendering/SVGRootInlineBox.cpp: (WebCore::applySVGKerning): (WebCore::SVGRootInlineBox::buildLayoutInformationForTextBox):
  • svg/SVGAllInOne.cpp:
  • svg/SVGFontElement.cpp: (WebCore::SVGFontElement::invalidateGlyphCache): (WebCore::SVGFontElement::ensureGlyphCache): (WebCore::matches): (WebCore::kerningForPairOfStringsAndGlyphs): (WebCore::SVGFontElement::horizontalKerningForPairOfStringsAndGlyphs): (WebCore::SVGFontElement::verticalKerningForPairOfStringsAndGlyphs):
  • svg/SVGFontElement.h: (WebCore::SVGKerningPair::SVGKerningPair): (WebCore::SVGFontElement::rendererIsNeeded):
  • svg/SVGHKernElement.cpp: (WebCore::SVGHKernElement::buildHorizontalKerningPair):
  • svg/SVGHKernElement.h:
  • svg/SVGParserUtilities.cpp: (WebCore::parseGlyphName): (WebCore::parseKerningUnicodeString):
  • svg/SVGParserUtilities.h:
  • svg/SVGVKernElement.cpp: Added. (WebCore::SVGVKernElement::SVGVKernElement): (WebCore::SVGVKernElement::~SVGVKernElement): (WebCore::SVGVKernElement::insertedIntoDocument): (WebCore::SVGVKernElement::removedFromDocument): (WebCore::SVGVKernElement::buildVerticalKerningPair):
  • svg/SVGVKernElement.h: Added. (WebCore::SVGVKernElement::rendererIsNeeded):
  • svg/SVGVKernElement.idl: Added.
  • svg/svgtags.in:

2010-05-08 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGFont's VKern implementation missing
​https://bugs.webkit.org/show_bug.cgi?id=38663

The changes in DOMWindow.idl and the implementation of the DOM
and JS bindings of VKern and HKern made some updates necessary.
Added new test case for SVGFont's VKern.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/Window/window-property-descriptors-expected.txt:
  • fast/dom/prototype-inheritance-2-expected.txt:
  • fast/dom/prototype-inheritance-expected.txt:
  • fast/js/global-constructors-expected.txt:
  • platform/mac/svg/text/text-vkern-expected.checksum: Added.
  • platform/mac/svg/text/text-vkern-expected.png: Added.
  • platform/mac/svg/text/text-vkern-expected.txt: Added.
  • svg/custom/global-constructors-expected.txt:
  • svg/text/text-vkern.svg: Added.

2010-05-08 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVGFont's VKern implementation missing
​https://bugs.webkit.org/show_bug.cgi?id=38663

  • projects/svg/status.xml:
1:44 AM Changeset in webkit [59009] by jer.noble@apple.com
  • 2 edits in trunk/WebCore

Unreviewed, just fixing windows build.

  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.h: Add #if USE(ACCELERATED_COMPOSITING) guards around GraphicsLayer areas of the code. Include GraphicsLayer.h directly.
  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp: As above.
  • platform/graphics/win/QTMovieVisualContext.cpp: Remove the #include of d3d9types.h.
1:26 AM Changeset in webkit [59008] by jer.noble@apple.com
  • 2 edits in trunk/WebCore/platform/graphics/win

Fix the windows build (by adding #if USE(ACCELERATED_COMPOSITING) guards).

12:38 AM Changeset in webkit [59007] by jer.noble@apple.com
  • 1 edit in trunk/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.h

Fix the windows build.

12:24 AM Changeset in webkit [59006] by eric@webkit.org
  • 6 edits in trunk/WebKitTools

2010-05-08 Eric Seidel <​eric@webkit.org>

Reviewed by Daniel Bates.

test-webkitpy fails under cygwin
​https://bugs.webkit.org/show_bug.cgi?id=38718

  • Scripts/webkitpy/common/system/executive.py:
    • Add _KILL_PROCESS_KILLED_PROCESS_EXIT_CODE and _KILL_ALL_KILLED_PROCESS_EXIT_CODE to store the expected exit codes of processes killed by kill_process and kill_all. These two constants are only used by the unit tests but are stored in executive.py so they can be right next to the platform ifs.
    • Remove unnecessary str() conversion, run_command does that for us.
    • Make os.kill retry on cygwin on EAGAIN. It's unclear why CYGWIN throws EAGAIN, but it only does so sometimes. 3 may not be enough retries, but we'll try it to start with.
    • Add _windows_image_name to automatically convert "yes" to "yes.exe" for use with taskkill.exe /im. Various callers to kill_all could be updated to remove the .exe, but that can be done in another patch.
    • Use taskkill.exe for killall on cygwin.
  • Scripts/webkitpy/common/system/executive_unittest.py:
    • Use the new *_KILLED_PROCESS_EXIT_CODE constants which are correctly set to 0 on windows/cygwin systems where taskkill.exe is used.
    • Test _windows_image_name
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
    • Add FIXME about including mac-snowleopard in baseline_search_path.
  • Scripts/webkitpy/layout_tests/port/webkit.py:
    • Make default_configuration actually read from the Configuration file.
  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py: Added.
    • Test default_configuration
  • Scripts/webkitpy/layout_tests/port/win.py:
    • Need a basic baseline_search_path if --platform dryrun is to work.
12:14 AM Changeset in webkit [59005] by eric@webkit.org
  • 25 edits
    3 adds
    1 delete in trunk

2010-05-08 Eric Seidel <​eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
​https://bugs.webkit.org/show_bug.cgi?id=38798

  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitResolve):
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_resolve_global):

2010-05-08 Eric Seidel <​eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
​https://bugs.webkit.org/show_bug.cgi?id=38798

  • fast/js/global-resolve-through-eval-expected.txt: Added.
  • fast/js/global-resolve-through-eval.html: Added.
  • fast/js/script-tests/global-resolve-through-eval.js: Added. (accessGlobal): (accessLocal):
  • platform/win/Skipped:

2010-05-08 Eric Seidel <​eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
​https://bugs.webkit.org/show_bug.cgi?id=38798

Test: fast/js/global-resolve-through-eval.html

  • bindings/js/JSCallbackData.h: (WebCore::JSCallbackData::JSCallbackData): (WebCore::JSCallbackData::~JSCallbackData): (WebCore::DeleteCallbackDataTask::create): (WebCore::DeleteCallbackDataTask::performTask): (WebCore::DeleteCallbackDataTask::isCleanupTask): (WebCore::DeleteCallbackDataTask::DeleteCallbackDataTask):
  • bindings/js/JSCustomVoidCallback.cpp: (WebCore::JSCustomVoidCallback::JSCustomVoidCallback): (WebCore::JSCustomVoidCallback::~JSCustomVoidCallback):
  • bindings/js/JSCustomVoidCallback.h:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/test/JS/JSTestInterface.cpp: (WebCore::):
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::):
  • workers/WorkerThread.cpp: (WebCore::WorkerThreadShutdownFinishTask::performTask): (WebCore::WorkerThreadShutdownStartTask::performTask):

2010-05-08 Eric Seidel <​eric@webkit.org>

Unreviewed, just reverting commit.

REGRESSION(59000): r59000 contained all sorts of changes it should not have, needs revert.
​https://bugs.webkit.org/show_bug.cgi?id=38798

  • Scripts/webkitpy/common/system/executive.py:
  • Scripts/webkitpy/common/system/executive_unittest.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
  • Scripts/webkitpy/layout_tests/port/webkit_unittest.py: Removed.
  • Scripts/webkitpy/layout_tests/port/win.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py:
  • Scripts/webkitpy/tool/commands/download.py:
  • Scripts/webkitpy/tool/commands/download_unittest.py:
12:08 AM Changeset in webkit [59004] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

[Qt] Unreviewed, QtWebKit versioning for the trunk.

The next version is going to be 2.1, based off the trunk.

  • Api/qwebkitglobal.h:
Note: See TracTimeline for information about the timeline view.