[Webkit-unassigned] [Bug 47607] meta tag parser needs to support ; as separator due to Android having made that popular

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 14 10:38:03 PDT 2010


https://bugs.webkit.org/show_bug.cgi?id=47607





--- Comment #3 from Kenneth Rohde Christiansen <kenneth at webkit.org>  2010-10-14 10:38:02 PST ---
(In reply to comment #2)
> (From update of attachment 70644 [details])
> I'm not saying that I'm necessarily against this change, but it needs a better explanation. Does this work in IE? In Firefox? Opera Mobile? Are these pages (which ones?) broken on iPhone? What do relevant specs say?

Facebook recently changed and I do not have an iPhone with me right now to test, but the site might be doing UA sniffing. On the other hand, I have ran into this problem at least twice before.

> What does Android version history say about reasons for such deviation from webkit.org?

The webkit.org code is upstreamed by RIM, presumable from the iPhone code dump, after Android started shipping this code. I do not know the original reason for this change.

> I'm also unsure about what the scope of this change is. ChangeLog says "meta tag parser", but it seems that code changes only affect meta viewport. In the latter is the case, perhaps the question about IE and Firefox isn't valid, but other ones still stand.

I'm not sure whether any shipped Opera mobile supports viewport meta tag or not, but at least Firefox support is pretty preliminary and IE does not support it.

> R- because of all the questions, and the lack of test case or an explanation why one isn't possible.

I just pushed the patch to get feedback. I believe that some of the viewport meta tests that we are not passing might be due to this issue. These tests come from Opera.

-- 
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