Bug 84385

Summary: Default to null value for HistoryItem::m_pageScaleFactor
Product: WebKit Reporter: Alexandre Elias <aelias>
Component: New BugsAssignee: Alexandre Elias <aelias>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, aelias, bdakin, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 84408    
Bug Blocks: 66687    
Attachments:
Description Flags
Patch none

Alexandre Elias
Reported 2012-04-19 14:36:26 PDT
Default to null value for HistoryItem::m_pageScaleFactor
Attachments
Patch (3.99 KB, patch)
2012-04-19 14:46 PDT, Alexandre Elias 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexandre Elias
Comment 1 2012-04-19 14:46:00 PDT
Created attachment 137980 [details] Patch
WebKit Review Bot
Comment 2 2012-04-19 16:30:45 PDT
Comment on attachment 137980 [details] Patch Clearing flags on attachment: 137980 Committed r114690: <http://trac.webkit.org/changeset/114690>
WebKit Review Bot
Comment 3 2012-04-19 16:30:49 PDT
All reviewed patches have been landed. Closing bug.
Adam Barth
Comment 4 2012-05-09 14:50:31 PDT
I'm told this was reverted.
Adam Barth
Comment 5 2012-05-09 15:02:58 PDT
Apparently this patch is blocked on <https://chromiumcodereview.appspot.com/10153008/>, which is now in the Chromium commit-queue.
Adam Barth
Comment 6 2012-05-10 00:20:58 PDT
https://chromiumcodereview.appspot.com/10153008/ landed as Chromium@136194. I think that means we're ready to land this patch again.
WebKit Review Bot
Comment 7 2012-05-10 00:38:08 PDT
Comment on attachment 137980 [details] Patch Clearing flags on attachment: 137980 Committed r116617: <http://trac.webkit.org/changeset/116617>
WebKit Review Bot
Comment 8 2012-05-10 00:38:13 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.