Bug 83600 - Need WebKit2 API to notify whether history loads are in the page cache
Summary: Need WebKit2 API to notify whether history loads are in the page cache
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Brady Eidson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2012-04-10 12:00 PDT by Brady Eidson
Modified: 2012-04-13 10:52 PDT (History)
3 users (show)

See Also:


Attachments
Patch v1 - Fix plus API test (29.72 KB, patch)
2012-04-10 12:22 PDT, Brady Eidson
webkit-ews: commit-queue-
Details | Formatted Diff | Diff
Patch v2 - Now with more Qt build fixes (30.46 KB, patch)
2012-04-10 13:35 PDT, Brady Eidson
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brady Eidson 2012-04-10 12:00:52 PDT
Need WebKit2 API to notify whether history loads are in the page cache

This should be in the bundle and leverage FrameLoaderClient::shouldGoToHistoryItem

In radar as <rdar://problem/11176921>
Comment 1 Brady Eidson 2012-04-10 12:22:34 PDT
Created attachment 136506 [details]
Patch v1 - Fix plus API test
Comment 2 WebKit Review Bot 2012-04-10 12:26:59 PDT
Attachment 136506 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1
Tools/TestWebKitAPI/Tests/WebKit2/ShouldGoToBackForwardListItem_Bundle.cpp:39:  The parameter name "bundle" adds no information, so it should be removed.  [readability/parameter_name] [5]
Tools/TestWebKitAPI/Tests/WebKit2/ShouldGoToBackForwardListItem_Bundle.cpp:39:  The parameter name "page" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 2 in 19 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Early Warning System Bot 2012-04-10 13:02:01 PDT
Comment on attachment 136506 [details]
Patch v1 - Fix plus API test

Attachment 136506 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/12376930
Comment 4 Brady Eidson 2012-04-10 13:35:20 PDT
Created attachment 136525 [details]
Patch v2 - Now with more Qt build fixes
Comment 5 WebKit Review Bot 2012-04-10 13:37:25 PDT
Attachment 136525 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1
Tools/TestWebKitAPI/Tests/WebKit2/ShouldGoToBackForwardListItem_Bundle.cpp:39:  The parameter name "bundle" adds no information, so it should be removed.  [readability/parameter_name] [5]
Tools/TestWebKitAPI/Tests/WebKit2/ShouldGoToBackForwardListItem_Bundle.cpp:39:  The parameter name "page" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 2 in 20 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 6 Darin Adler 2012-04-12 14:33:32 PDT
Comment on attachment 136525 [details]
Patch v2 - Now with more Qt build fixes

View in context: https://bugs.webkit.org/attachment.cgi?id=136525&action=review

> Source/WebKit2/UIProcess/API/C/WKPage.h:74
> -typedef void (*WKPageWillGoToBackForwardListItemCallback)(WKPageRef page, WKBackForwardListItemRef item, const void *clientInfo);
> +typedef void (*WKPageWillGoToBackForwardListItemCallback)(WKPageRef page, WKBackForwardListItemRef item, WKTypeRef userData, const void *clientInfo);

Are there any existing clients that would now have binary incompatibility? Or is this already something new?
Comment 7 Brady Eidson 2012-04-12 15:31:30 PDT
(In reply to comment #6)
> (From update of attachment 136525 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=136525&action=review
> 
> > Source/WebKit2/UIProcess/API/C/WKPage.h:74
> > -typedef void (*WKPageWillGoToBackForwardListItemCallback)(WKPageRef page, WKBackForwardListItemRef item, const void *clientInfo);
> > +typedef void (*WKPageWillGoToBackForwardListItemCallback)(WKPageRef page, WKBackForwardListItemRef item, WKTypeRef userData, const void *clientInfo);
> 
> Are there any existing clients that would now have binary incompatibility? Or is this already something new?

Less than two months old, added in http://trac.webkit.org/changeset/108570

Thanks for the review!
Comment 8 Brady Eidson 2012-04-13 10:52:46 PDT
http://trac.webkit.org/changeset/114143