Summary: | Enable web intents flag for chromium build | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Greg Billock <gbillock> | ||||
Component: | New Bugs | Assignee: | Greg Billock <gbillock> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 75123 | ||||||
Attachments: |
|
Description
Greg Billock
2012-05-17 11:48:04 PDT
Created attachment 142517 [details]
Patch
Can we unskip some tests now that this feature is enabled during testing? (In reply to comment #2) > Can we unskip some tests now that this feature is enabled during testing? They're not skipped in chromium now, but the tests should now actually be running the full test in the suite instead of skipping the good stuff. (The intent tag test is still skipped.) > They're not skipped in chromium now, but the tests should now actually be running the full test in the suite instead of skipping the good stuff. (The intent tag test is still skipped.)
What does that mean? Do we need to update any -expected.txt files? I'm confused.
(In reply to comment #4) > > They're not skipped in chromium now, but the tests should now actually be running the full test in the suite instead of skipping the good stuff. (The intent tag test is still skipped.) > > What does that mean? Do we need to update any -expected.txt files? I'm confused. I don't think so (the bot passes fine). My local webkit checkout builds and runs the tests. Am I doing what I think I'm doing with this flag? My expectation was that the webkit chromium builds use it, but the flags can be (and are)set differently in the chromium-initiated builds, from the chromium gypi files. (And that this change merely aligns them.) Is that not the right understanding? The tests were definitely running before this cl (hence the red tests in cr-linux on the delivery change). IOW, my impression is that the chromium code would no longer need to pass in enable_web_intents=1, but that it in fact does pass that in right now. So they're now more aligned, but there's still a shut-off valve on both sides. Is that right? Or does this change also (inadvertently) disable the enable_web_intents shut-off valve on the Chromium side? There shouldn't be any misalignment. We use the same build flags for both. It sounds like the next step after this patch is to delete the enable_web_intents variable. (In reply to comment #6) > There shouldn't be any misalignment. We use the same build flags for both. It sounds like the next step after this patch is to delete the enable_web_intents variable. Right. That sounds good to me. Comment on attachment 142517 [details] Patch Clearing flags on attachment: 142517 Committed r117615: <http://trac.webkit.org/changeset/117615> All reviewed patches have been landed. Closing bug. |