Remove E4X code for Firefox 20+ #53 #54
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add commentToText() - Add getTextContent()
|
👍 |
|
マージを待ってます。 |
|
不具合及びこの変更の動作を、Windows 7 Home Premium SP1 64bit上のFirefox 20.0、拡張のバージョンは0.4.34という環境で確認しました。少なくとも、この変更を適用したTomblooは正常に起動し、TumblrへのPostに成功するようです。 E4Xを含むコードは、Firefox 20.0ではSyntaxErrorとなります。これにより、TomblooはE4Xを含むファイルを読み込んだ段階で処理を停止してしまいます。これは、E4X無効化への対応はパッチでは行えない事を意味しています(Tomblooのファイルを読み込んでからパッチを読み込む為)。 また、この変更は基本的にTomblooの全ての動作に影響を与えうるものですが、直接的に影響を受ける機能として以下が該当すると思われます。
この内、Kawa Modelはテストでのみ使用されているようでした。 |
|
👍 |
|
マージが遅れて、大変失礼いたしました。 |
wantora
pushed a commit
to wantora/tombfix-core
that referenced
this pull request
May 13, 2016
wantora
pushed a commit
to wantora/tombfix-core
that referenced
this pull request
May 13, 2016
* Update minVersion from 22.0 to 26.0 * Revise YahooBookmarks Model to#9 * Remove YahooBookmarks's Tag Provider * Update Delicious Model to#56 * Use getLocalStorage() in Delicious.getInfo() to#10 * Drop support for NDrive to#25 * Add "Quote - textarea" Extractor to#41 * Revise "Video - YouTube" Extractor #65 * Revise "Video - Nico Nico Douga" Extractor * Rename "Video - Nico Nico Douga" to "Video - Nicovideo" * Revise "Video - Vimeo" Extractor to#54 * Fix bugs about Tumblr Photoset to#38 * Fix a bug about "Photo - pixiv" Extractor #64 * "Change Account" should be translate * Add "require a browser restart" message in Options Post to#55 * Add Cu * Update DOMStorageManager's ContractID for Firefox 23+ * Update getLocalStorage() for Firefox 23+ * Rename getLocalStorageValue() to getLocalStorageValueBySQL() * Improve Console Logging * Add console * Remove ConsoleService * Revise prototype.js to#12 * Move Array#split from utility.js to prototype.js * Move String.katakana from utility.js to prototype.js * prototype.js: Remove String#repeat, Math.hypot() * utility.js: Add createLink() from Taberareloo 3.0.4 * utility.js: Remove uniq(), createSet() * Add dev scripts to#12 * test: Add getLocalStorage()_about_ssl_delicious_test_patch.js, prototype_test_patch.js
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Firefox 20以降で実施されるE4Xの無効化と削除に対応する為、E4Xが使用されている処理の削除、及び互換性のある処理への置き換えを行なっています。
E4X | MDN
Firefox 20ではabout:configに「javascript.options.xml.chrome」、「javascript.options.xml.content」は存在するのですが、これらを有効にしてもTomblooは動作しませんでした。Firefox 21以降ではこれらは存在しません。
不具合及びこの変更の動作は、Windows 7 Home Premium SP1 64bit上のFirefox Beta 20.0、拡張のバージョンは0.4.34という環境で確認しています。