Bug 97116

Summary: [Chromium] Move notifyJSOutOfMemory out of PlatformSupport
Product: WebKit Reporter: Mark Pilgrim (Google) <pilgrim>
Component: WebKit Misc.Assignee: Mark Pilgrim (Google) <pilgrim>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, fishd, haraken, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 82948    
Attachments:
Description Flags
Patch none

Mark Pilgrim (Google)
Reported 2012-09-19 10:15:00 PDT
[Chromium] Move notifyJSOutOfMemory out of PlatformSupport
Attachments
Patch (6.32 KB, patch)
2012-09-19 10:15 PDT, Mark Pilgrim (Google) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Pilgrim (Google)
Comment 1 2012-09-19 10:15:45 PDT
Created attachment 164751 [details] Patch
Mark Pilgrim (Google)
Comment 2 2012-09-19 10:17:19 PDT
Routing call from WebCore/bindings/v8/V8Binding through FrameLoaderClient instead of PlatformSupport.
Adam Barth
Comment 3 2012-09-19 10:19:57 PDT
Comment on attachment 164751 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=164751&action=review > Source/WebKit/chromium/src/FrameLoaderClientImpl.h:64 > + // Script in the page tried to allocate too much memory. I'd skip this comment.
WebKit Review Bot
Comment 4 2012-09-19 10:53:25 PDT
Comment on attachment 164751 [details] Patch Clearing flags on attachment: 164751 Committed r129020: <http://trac.webkit.org/changeset/129020>
WebKit Review Bot
Comment 5 2012-09-19 10:53:29 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.