Bug 194046 - testapi's `testBytecodeCache` does not need to run the code twice
Summary: testapi's `testBytecodeCache` does not need to run the code twice
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tadeu Zagallo
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-01-30 13:59 PST by Tadeu Zagallo
Modified: 2019-01-30 19:09 PST (History)
7 users (show)

See Also:


Attachments
Patch (2.80 KB, patch)
2019-01-30 14:00 PST, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch for landing (2.80 KB, patch)
2019-01-30 16:33 PST, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch for landing (2.80 KB, patch)
2019-01-30 17:56 PST, Tadeu Zagallo
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tadeu Zagallo 2019-01-30 13:59:34 PST
Since we populate the cache eagerly (unlike the stress tests) we don't need to run the code twice.
Comment 1 Tadeu Zagallo 2019-01-30 14:00:49 PST
Created attachment 360618 [details]
Patch
Comment 2 Mark Lam 2019-01-30 14:11:37 PST
Comment on attachment 360618 [details]
Patch

r=me
Comment 3 WebKit Commit Bot 2019-01-30 15:37:55 PST
Comment on attachment 360618 [details]
Patch

Rejecting attachment 360618 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 360618, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as commit-queue@webkit.org...
Fetching: https://bugs.webkit.org/attachment.cgi?id=360618&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=194046&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Updating working directory
Processing patch 360618 from bug 194046.
Fetching: https://bugs.webkit.org/attachment.cgi?id=360618
Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	Source/JavaScriptCore/API/tests/testapi.mm
	M	Source/JavaScriptCore/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/Source/JavaScriptCore/ChangeLog' is out of date
W: 55440ac93da99bc994e82a2e1ae6a5a468e8cc55 and refs/remotes/origin/master differ, using rebase:
:040000 040000 a12a397647e5b95fae2d59cef74aedbf9cb08859 c53753ff86e001b637539b9f8f1254059d22aef7 M	Source
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	Source/JavaScriptCore/API/tests/testapi.mm
	M	Source/JavaScriptCore/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/Source/JavaScriptCore/ChangeLog' is out of date
W: 55440ac93da99bc994e82a2e1ae6a5a468e8cc55 and refs/remotes/origin/master differ, using rebase:
:040000 040000 a12a397647e5b95fae2d59cef74aedbf9cb08859 c53753ff86e001b637539b9f8f1254059d22aef7 M	Source
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
Current branch master is up to date.

Full output: https://webkit-queues.webkit.org/results/10958001
Comment 4 Tadeu Zagallo 2019-01-30 16:33:47 PST
Created attachment 360645 [details]
Patch for landing
Comment 5 WebKit Commit Bot 2019-01-30 17:36:19 PST
Comment on attachment 360645 [details]
Patch for landing

Rejecting attachment 360645 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'validate-changelog', '--check-oops', '--non-interactive', 360645, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Source/JavaScriptCore/ChangeLog contains OOPS!.

Full output: https://webkit-queues.webkit.org/results/10960789
Comment 6 Tadeu Zagallo 2019-01-30 17:56:02 PST
Created attachment 360661 [details]
Patch for landing
Comment 7 WebKit Commit Bot 2019-01-30 19:08:01 PST
Comment on attachment 360661 [details]
Patch for landing

Clearing flags on attachment: 360661

Committed r240753: <https://trac.webkit.org/changeset/240753>
Comment 8 WebKit Commit Bot 2019-01-30 19:08:03 PST
All reviewed patches have been landed.  Closing bug.
Comment 9 Radar WebKit Bug Importer 2019-01-30 19:09:29 PST
<rdar://problem/47691091>