[Webkit-unassigned] [Bug 28836] [Qt] Add a setting to turn SessionStorage on/off

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Sep 1 08:05:00 PDT 2009


Eric Seidel <eric at webkit.org> changed:

           What    |Removed                     |Added
  Attachment #38809|review?                     |review-
               Flag|                            |

--- Comment #4 from Eric Seidel <eric at webkit.org>  2009-09-01 08:05:00 PDT ---
(From update of attachment 38809)
Not to pick on the naming too much, but testOptionalJSObjects doesn't mention
anything about storage maybe testOptionalStorageJSObjects?

Why does testOptionalJSObjects use two web views?  Seems it doesn't need to.

Seems like we could make a simple inline function to do all of these tests

testStorageFlag(QWebSettings::SessionStorageEnabled, "sessionStorage", false);

static inline testStorageFlag(QString settingName, QString jsObjectName, bool

Why use:
QCOMPARE(webPage1->mainFrame()->evaluateJavaScript("if(window.sessionStorage ==
undefined) {2}").toInt(), 2);
instead of something like this:
QCOMPARE(webPage1->mainFrame()->evaluateJavaScript("window.sessionStorage ==
undefined").toInt(), true);
Then it would be easier to compare with teh bool value passed into your little
helper function.

Turning the "sessionStorage" stirng passed into your helper into the JS string
will require a bit of printf magic. :)

I think this change is fine.  I'm OK to r+ this, but since I expect you'll read
my fantastically convincing arguments above and want to change the test to be
more super awesome, I'll r- this.  If you disagree that the test should be so
awsome, feel free to r? this again as-is.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list