REOPENED 136579
Move CallFrame and Register inlines functions out of JSScope.h 
https://bugs.webkit.org/show_bug.cgi?id=136579
Summary Move CallFrame and Register inlines functions out of JSScope.h
Mark Lam
Reported 2014-09-05 10:58:08 PDT
This include fixing up some files to #include JSCInlines.h to pick up these inline functions. I also added JSCellInlines.h to JSCInlines.h since it is included from many of the affected .cpp files.
Attachments
the patch (25.93 KB, patch)
2014-09-05 11:06 PDT, Mark Lam 		mark.lam: review-
DetailsFormatted DiffDiff
patch 2: +WebCore build fixes. (38.04 KB, patch)
2014-09-05 22:25 PDT, Mark Lam 		no flags
DetailsFormatted DiffDiff
patch 3: one more WebCore build fix (38.03 KB, patch)
2014-09-08 15:25 PDT, Mark Lam 		ggaren: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2014-09-05 11:06:22 PDT
Created attachment 237700 [details] the patch
Geoffrey Garen
Comment 2 2014-09-05 11:58:22 PDT
Comment on attachment 237700 [details] the patch r=me
Mark Lam
Comment 3 2014-09-05 18:26:10 PDT
Comment on attachment 237700 [details] the patch WebCore side needed a lot of work with #includes. I’ll upload an updated patch with WebCore changes once I’m done rebuilding the world.
Mark Lam
Comment 4 2014-09-05 22:25:05 PDT
Created attachment 237729 [details] patch 2: +WebCore build fixes.
Mark Lam
Comment 5 2014-09-08 15:25:17 PDT
Created attachment 237815 [details] patch 3: one more WebCore build fix
Geoffrey Garen
Comment 6 2014-09-08 15:52:08 PDT
Comment on attachment 237815 [details] patch 3: one more WebCore build fix r=me
Mark Lam
Comment 7 2014-09-08 16:00:08 PDT
Thanks for the review. Landed in r173402: <http://trac.webkit.org/r173402>.
WebKit Commit Bot
Comment 8 2014-09-08 17:17:07 PDT
Re-opened since this is blocked by bug 136649
Note You need to log in before you can comment on or make changes to this bug.