Summary: | [Qt] Add option to turn on disk caching in QtTestBrowser | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Keith Kyzivat <kamaji> | ||||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Enhancement | CC: | cshu, eric, laszlo.gombos, ossy, webkit.review.bot | ||||||||
Priority: | P5 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 65083 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Keith Kyzivat
2011-07-21 19:52:41 PDT
Created attachment 101694 [details]
Patch
Comment on attachment 101694 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=101694&action=review > Tools/QtTestBrowser/launcherwindow.cpp:808 > + qDebug() << "Using disk cache location: " << cacheLocation; > + cache->setCacheDirectory(cacheLocation); > + } > + page()->networkAccessManager()->setCache(cache); > + qDebug() << "disk cache" << (enable ? "enabled" : "disabled"); Please remove qDebug() before committing. Those tend to clutter after a while. Created attachment 101734 [details]
patch
Comment on attachment 101734 [details] patch Clearing flags on attachment: 101734 Committed r91635: <http://trac.webkit.org/changeset/91635> All reviewed patches have been landed. Closing bug. Reopen, because it was rolled out by https://trac.webkit.org/changeset/91649 I rolled out this patch, because it broke the Qt minimal buildbot. Before you ask me, why I didn't try to fix it instead of rolling out: Because I won't do it instead of the author on a sunday night and the author must watch the buildbots on build.webkit.org and fix his patch if he break something. You can find the build log here: http://build.webkit.org/builders/Qt%20Linux%20Release%20minimal/builds/29415 Totally understandable Ossy. I'll fix it as soon as I get a chance to do a minimal build. How can I prevent this sort of situation from happening in the future? Is there an early warning bot that runs the minimal build configuration that I'm not aware of? (In reply to comment #8) > Totally understandable Ossy. I'll fix it as soon as I get a chance to do a minimal build. > > How can I prevent this sort of situation from happening in the future? Is there an early warning bot that runs the minimal build configuration that I'm not aware of? You can find the configuration of the minial bot here: http://build.webkit.org/buildslaves/szeged-linux-2 Unfortunately we don't have EWS bot for minimal build now, but it is a good idea. Eric, how can we add a new EWS platform for qt-minimal if we would like to do it? Created attachment 101906 [details]
Patch
patch updated to work with Qt minimal build
Comment on attachment 101906 [details] Patch Clearing flags on attachment: 101906 Committed r91806: <http://trac.webkit.org/changeset/91806> All reviewed patches have been landed. Closing bug. |