Bug 154304 - Navigator.geolocation should not be marked a [Replaceable] and should be on the prototype
Summary: Navigator.geolocation should not be marked a [Replaceable] and should be on t...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Bindings (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL: https://dev.w3.org/geo/api/spec-sourc...
Keywords: InRadar, WebExposed
Depends on:
Blocks:
 
Reported: 2016-02-16 12:49 PST by Chris Dumez
Modified: 2016-02-16 17:11 PST (History)
13 users (show)

See Also:


Attachments
Patch (19.04 KB, patch)
2016-02-16 15:25 PST, Chris Dumez
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews101 for mac-yosemite (766.17 KB, application/zip)
2016-02-16 16:18 PST, Build Bot
no flags Details
Patch (26.81 KB, patch)
2016-02-16 16:20 PST, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2016-02-16 12:49:34 PST
Navigator.geolocation should not be marked a [Replaceable]:
https://dev.w3.org/geo/api/spec-source.html#geolocation_interface

This was done as a workaround for an Amazon App bug in Bug 133559 but has the following unwanted behavior now that our bindings are behaving in a more standard way:
navigator.geolocation = 1
console.log(navigator.geolocation) -> prints 1 instead of the Geolocation object.

We need to test if this workaround is still needed by the Amazon app and find a better workaround if it is.
Comment 1 Chris Dumez 2016-02-16 14:17:43 PST
Moving the Navigator properties to the prototype and dropping the [Replaceable] for Navigator.geolocation, I have verified that rdar://problem/16332749 no longer reproduces in the latest Amazon iOS app.
Comment 2 Radar WebKit Bug Importer 2016-02-16 14:32:17 PST
<rdar://problem/24685092>
Comment 3 Chris Dumez 2016-02-16 14:45:26 PST
Firefox and Chrome both don't allow replacing Navigator.geolocation. Both throw an exception when trying to set Navigator.geolocation in strict mode.
Comment 4 Chris Dumez 2016-02-16 15:25:53 PST
Created attachment 271495 [details]
Patch
Comment 5 Build Bot 2016-02-16 16:18:27 PST
Comment on attachment 271495 [details]
Patch

Attachment 271495 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/842221

New failing tests:
fast/dom/Geolocation/enabled.html
js/dom/delete-syntax.html
Comment 6 Build Bot 2016-02-16 16:18:34 PST
Created attachment 271503 [details]
Archive of layout-test-results from ews101 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 7 Chris Dumez 2016-02-16 16:20:32 PST
Created attachment 271504 [details]
Patch
Comment 8 Chris Dumez 2016-02-16 17:11:26 PST
Comment on attachment 271504 [details]
Patch

Clearing flags on attachment: 271504

Committed r196673: <http://trac.webkit.org/changeset/196673>
Comment 9 Chris Dumez 2016-02-16 17:11:32 PST
All reviewed patches have been landed.  Closing bug.