💡スマート家電
引越ししたので家電をいくつか進捗した。 せっかくなので、リモートで制御できるものを買った。
🤖ルンバ
床がちらかる前にルンバを買った。iPhoneアプリで制御できるなかでは一番安いモデルにした。
【Amazon.co.jp限定】アイロボット ルンバ691 wifi対応 複数床面対応 自動充電 ロボット掃除機 R691060
- 出版社/メーカー: iRobot (アイロボット)
- メディア: ホーム&キッチン
- この商品を含むブログを見る
動かしたらいきなりケーブルを巻き込んで停止した。ケーブルボックスを導入したら、なんとかなった。
ルンバちゃんがベースステーションのコンセントに突進していって自殺みたいになってる
— mzp (@mzp) August 15, 2018
@blackenedgold OAタップ & ケーブルボックスありがとうございます! これで、ルンバちゃんがケーブルを食べることがなくなります!! pic.twitter.com/i7mXrTPuhI
— mzp (@mzp) August 17, 2018
ルンバがケーブルを巻き込むのでケーブルボックスを追加購入した。なんかペット飼ってる気分になってきた
— mzp (@mzp) September 3, 2018
いまも週一くらいで停止している。かわいい。
💨エアコン
エアコンは部屋についてたので、NatureRemo miniを購入した。
スマートリモコン Nature Remo mini【Amazon Echo/Google Home対応】
- 出版社/メーカー: Nature, Inc.
- メディア: エレクトロニクス
- この商品を含むブログを見る
あまり調べずに買ったが、温度調整のUIが格好よくていい。温度計がついてるのもおもしろい。
💡照明
照明も部屋に備え付けだったが、調べたらリモコン操作に対応していた。
リモコンの信号をNatureRemo miniに覚えさせたいけどどうしたらいいのかなーと調べたが、どうやら実物が必須らしかった。Amazonで購入して初期設定したあとはしまってある。
スマートリモコンに覚えされるために照明の純正リモコン買ったけど、これは正しい道を歩んでるんだろうか
— mzp (@mzp) August 27, 2018
スマートリモコンがパターンを記憶したので、エアコンと照明のリモコンをしまった。なんか自炊したあとの物理本の処理に困る話と似てる
— mzp (@mzp) August 28, 2018
⚡️HomeBridge
それぞれ別のアプリから制御する必要があって不便だった。 iPhoneのHomeアプリから一括制御するためにnfarina/homebridgeを導入した。
部屋はひとつしかないがベッドルームということにしている。
HomeBridgeはRaspberryPiZeroWで動かしている。Raspberry Pi Zero W ケースキット - SWITCH-SCIENCEを買ったが、SDカードがついてないことを見落してて追加購入した。
家電をhomekitから扱えるようにするぞ!!ということで買ったRaspberryPi ZeroWが届いたが、micro SDカードがなかったので、おしまいです
— mzp (@mzp) September 3, 2018
セットアップ以下のサイトを参考にした。
- USB OTGを使ったRapsberry Pi Zero WH のセットアップ - Qiita
- Raspberry Pi 3 にhomebridgeを導入してsiriから家電操作できるようにした際の覚書 - Qiita
モジュールは以下のものをいれた。
💕感想
一番の利点はいつも手元にあるiOSデバイスから制御できるようになった点である。リモコンはどこに置いたか忘れがちだが、iPhoneかiPadかmacOSのどれかが手元にあることは多い。
Siri経由で操作できるけどそこまで便利ではない。
スマフォで家電制御するやつ、音声で制御できるとかよりも、どのiPhoneからでも電気を消せるのが便利な気がする
— mzp (@mzp) September 9, 2018
あとは以下のものをスマート家電化したいができてない。
- スマートロック。Qrioがよさそうに見えるが。
- コーヒーメーカー。自動応答するコーヒーメーカーは夢がある。RFC 2324 Hyper Text Coffee Pot Control Protocolをしゃべってほしい。
- 洗濯機・炊飯器。残り時間をアプリから確認したい。
☀️ダイナミックデスクトップ壁紙
Mojaveのダイナミックデスクトップでは時刻によって壁紙が変化する。 これを用いてミクさんが部屋の中を歩きまわる壁紙を作成した。
(livetune feat. 初音ミク「Redial」Music Video - YouTubeより)
🖥ダイナミックデスクトップ
macOS Mojaveの紹介ページに記載されているとおり、Mojaveでは時間の経過に応じて壁紙が変化する。
WWDCのキーノートでも、生活にあわせて変化する壁紙のデモがあった。
🛠仕組み
ダイナミックデスクトップ用の壁紙は他の壁紙と同様 /Library/Desktop Pictures
に配置されている。 異様にファイルサイズがでかい。
プレビューで開くと16枚の画像が確認できる。HEIFでは一つのファイルに複数の画像をまとめられるので、その機能を利用している。
それに加えてメタデータに時刻と画像の対応、より正確には太陽の位置と画像の対応を格納している。 ここに関しては、以下の記事が詳しい。
- macOS Mojave dynamic wallpaper – ITNEXT
- macOS Mojave dynamic wallpapers (II) – ITNEXT
- macOS Mojave wallpaper (III) – ITNEXT
💕 作る
WallpaperEngineを使った以下の壁紙の再現を目指す。動きまわってるのがかわいい。
ついったでオススメが流れてきて衝動買いしたWallpaper Engineいいぞ!
— にゃいぼん aka へらぶな (@nyaibon) 2017年1月4日
壁紙が動くだけで謎の未来感に感動している!steamで買えるぞ! pic.twitter.com/hHRSTRAuFg
livetune feat. 初音ミク「Redial」Music Video - YouTubeをダウンロードしてフレームごとに分割する。
youtube-dl 'https://www.youtube.com/watch?v=243vPl8HdVk' ffmpeg -i 'livetune feat. 初音ミク「Redial」Music Video-243vPl8HdVk.mkv' -f image2 %d.png
ここからいい感じの16枚を選ぶ。
複数の画像を束ねた上でメタデータを書き込むのは、mczachurski/wallpapper: Console application for creating dynamic wallpapers for macOS Mojaveで行なう。
brew tap mczachurski/wallpapper brew install wallpapper
macOS Mojave dynamic wallpaper – ITNEXTを参考に定義ファイルを雑に作る。
info.json
[ { "altitude": -0.3427528387535028, "azimuth": 270.9334057827345, "fileName": "1.png", "isPrimary": true, "isForLight": true, "isForDark": false }, { "altitude": -10.239758644725045, "azimuth": 81.77588714480999, "fileName": "2.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": -4.247734408075456, "azimuth": 86.33545030477751, "fileName": "3.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 1.3890866331008431, "azimuth": 90.81267037496195, "fileName": "4.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 7.167168970526129, "azimuth": 95.30740958876589, "fileName": "5.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 13.08619419164163, "azimuth": 99.92062963268938, "fileName": "6.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 40.41563946490428, "azimuth": 129.18652208191958, "fileName": "7.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 53.43347266172774, "azimuth": 182.2330942549791, "fileName": "8.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 38.793128200638634, "azimuth": 233.5515919580959, "fileName": "9.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 11.089423171265878, "azimuth": 261.87159046576664, "fileName": "10.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": 5.1845753236736245, "azimuth": 266.4432737071051, "fileName": "11.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": -6.248309374122789, "azimuth": 275.44204536695247, "fileName": "12.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": -12.20770735214888, "azimuth": 280.07031589401174, "fileName": "13.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": -39.48933951993012, "azimuth": 309.41857318745144, "fileName": "14.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": -52.75318137879935, "azimuth": 2.1750965538675473, "fileName": "15.png", "isPrimary": false, "isForLight": false, "isForDark": false }, { "altitude": -38.04743388682423, "azimuth": 53.50908581251309, "fileName": "16.png", "isPrimary": false, "isForLight": false, "isForDark": true } ]
このファイルを用いてダイナミックデスクトップ用の壁紙を生成する。
wallpapper -o ~/Pictures/redial.heic -i info.json *.png
システム環境設定から設定する。なぜかサムネ画像がおかしい。
時刻を変更して動作を確認する。
✨感想
動画を壁紙にする場合と比較して変化がおだやかなので、そこまで気がちらない。よい。
こちらを覗き込む画像は夜に表示されるようにしている。 なので、出社前は空の部屋で、家かえってきてPCをつけるとミクさんと目があうので、体験としてよい。
もうちょっと応用したいが、変化してたのしい壁紙のアイデアが足りない。WallpaperEngineのときもそういう話をしていた。
「wallpaper engineめっちゃいいじゃん」「作るか」みたいな話をしたけど、結局、壁紙が動画になってるのが最高なんじゃなくて、ミクさんがかわいいだけだわ、という結論になった。
— mzp (@mzp) January 7, 2017
♨️銭湯
東京に引っ越したら銭湯によく行くようになった。東京にはいっぱい銭湯があってよい。
💼持ち物
タオルやシャンプーなどは置いてないことが多いので、自分のものを持っていく。 ダイソーで買ったビニールバッグにコンビニで買ったシャンプーセットを入れている。
水を持ち込んでる人もたまに見かける。 炭酸水を持ち込んで、飲み水と頭を洗うのに使ってて、なるほど感があった。
👍よい
- 大きい風呂に入れる。
- 10分くらい歩いたとこにあるので便利。ご飯を炊いてる間に風呂、とかをやっている。お風呂の四階に住む。 - 物件ファンほど近くはない。
- 銭湯によっては温泉にはいれるとこもあってお得。
- あまり混んでないので、雑に行ける。
🤔微妙
- 銭湯の料金 > 水道代なので、やりすぎると破産するかもしれない。
- 街中を歩いてた直後、店にはいって数十秒で全裸になるという行為に違和感を覚えてしまう。慣れない。
- 結構話しかけられる。街中で話しかけられることはほぼないのに、全裸だと話しかけられるのおもしろい。
AquaSKK 4.6.0/4.6.1: Mojave対応
Mojave対応を行なったAquaSKKを4.6.0としてリリースした。2018-9-1: HighSierraで一部機能(#84)が動かない不具合が見付かったので、修正して4.6.1としてリリースした。
https://github.com/codefirst/aquaskk/releases/tag/4.6.1
🌓ダークモード
Mojaveではダークモードが導入された。
(WWDC2018 Keynoteより引用)
AquaSKKもダークモードに対応させ、候補ウインドウがダークモードで表示されるようにした。 さらに、アクセントカラーも扱うようにした。
🎨 アイコン色の調整
ダークモードではASCIIモードのモードアイコンの視認性がかなり低くなる。
Mojaveより前から存在していた問題(Issue #75)だが、ダークモードが強化されたMojaveではより顕著になる。
色合いを調整して、視認性を向上させた。
👋32bitサポートの廃止/libc++への切り替え
Xcode10から32ビットアプリケーションのサポートが廃止された。Mojave以降は64ビットアプリケーションのみのサポートとなるので、これを気に32ビットサポートは廃止した。
(WWDC2017 Keynoteより引用)
またlibstdc++のサポートが廃止されたので、libc++に切り替えた。これは目に見える影響はないと思う。
🙊NDAへの配慮
本記事はNDAに配慮し、Xcode 10やmacOS Mojaveのスクショショットは利用していない。Mojaveが正式リリースしてから書けば楽だったが、キリがついたのでリリースした。若干、無理がある。
ダークモード対応中の様子は楽しいので、それはどこかでロンダリングした上で出したい。
🔥Firebase
夏休みなのでFirebaseの練習をした。
ライブフォトをシェアするSNSのプロトタイプを作った。ライブフォト好きだけど、他の人に共有しづらくて困っている。
📝参考にしたもの
基本的にFirebase Guidesを読んで作った。 これでだいたい分かるので、すごい。
ただ、Apple Developer Portalの設定が必要な箇所になると急に説明が雑になるので、そこは茅場町モバイルアプリもくもく会で教えてもらった。
🔥開発の様子: Firebase
ログイン
Firebase Authenticationを使うと、あっさりGoogleアカウントログインが作れる。便利。
Firestore
Firestoreでプロフィール情報を保存するようにした。
コンソールからデータが確認できて便利。
データの保存にはCodableFirebaseが便利だった。 Codable
な構造体を型安全に扱えてよい。
func get<T: Codable>(documentPath: String) -> SignalProducer<T?, NoError> { let ref = Firestore.defaultStore().collection("collectionName") return SignalProducer { observer, _ in ref.document(documentPath).getDocument { document, error in if let data = document?.data(), let entity = try? FirestoreDecoder().decode(T.self, from: data) { observer.send(value: entity) } observer.sendCompleted() } } }
Storage
Cloud Storageで画像を保存できるようにした。
通知
Firebase Cloud MessagingとCloud Functionsを使ってfav通知を作った。
Cloud MessagingでAPNsを使う方法は2通りあるがAPNs authentication keyのほうが、どのアプリでも使えて便利。(茅場町モバイルアプリもくもく会で教えてもらった)
通知のロジックを書く場所が分からなかったが、Cloud FunctionsでDB等の変更を監視して通知を送るのがよいらしい。
(What can I do with Cloud Functions?から引用)
コンソールから通知を送る方法が分からなかったが、Growカテゴリの下にあった。
📱進捗: Firebase以外
差分更新
画面更新にはDifferenceKitを使った差分更新を採用した。reloadData
を読んでた箇所をDifferenceKitに書き換えるだけでよかったので便利。
index c7e6661..ab0d428 100644 --- a/Sources/Controllers/PhotosViewController.swift +++ b/Sources/Controllers/PhotosViewController.swift @@ -5,6 +5,7 @@ // Created by mzp on 2018/08/20. // Copyright © 2018 mzp. All rights reserved. // +import DifferenceKit import MobileCoreServices import NorthLayout import Photos @@ -40,11 +41,15 @@ public class PhotosViewController: UICollectionViewController, UIBarButtonItem(barButtonSystemItem: .add, target: self, action: #s elector(add)) collectionView.register(UICollectionViewCell.self, forCellWithReuseIdentifier: kCell) + + self.dataSource.combinePrevious([]).signal.observeValues { oldValue, newValue in + let changeset = StagedChangeset(source: oldValue, target: newValue) + self.collectionView.reload(using: changeset) { _ in } + } PhotoFetchLatest().call().collect().startWithResult { switch $0 { case .success(let photos): self.dataSource.swap(photos) - self.collectionView.reloadData() case .failure(.firebase(let error)): self.presentError(title: "Firebabse", message: error.message) case .failure(.filesystem(let error)):
ライブフォトの読み書き
ライブフォトの読み書きはアンドキュメントな機能を使う必要がある。
読み込みは、imagePickerController(_:,didFinishPickingMediaWithInfo:)
の info
から取得する。
extension ViewController: UIImagePickerControllerDelegate { public func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey: Any]) { guard let livePhoto = info[UIImagePickerController.InfoKey.livePhoto] as? PHLivePhoto else { return } guard let imageURL = photo.value(forKey: "imageURL") as? URL else { return } guard let videoURL = photo.value(forKey: "videoURL") as? URL else { return } // .... } }
作成は、PHLivePhoto
の request
に、 画像と動画のURLを渡すことで生成できる。 PHLivePhotoInfoCancelledKey
に対応する値が0になるまで待たないと生成されない。
func create(imageURL: URL, videoURL: URL) { PHLivePhoto.request( withResourceFileURLs: [imageURL, videoURL], placeholderImage: nil, targetSize: .zero, contentMode: .aspectFit) { livePhoto, info in if let canceled = info[PHLivePhotoInfoCancelledKey] as? NSNumber, canceled == 0, let livePhoto = livePhoto { // ... } } }
✨その他
GitGuardian
なにも考えずにPublicレポジトリでやっていたら、GitGuardianから警告メールが来た。 あわててrevokeしてprivate レポジトリにした。
夏休み
気分転換のために何度か近所の喫茶店に行ったが、夏休みの宿題をやっているらしき兄弟がいて、夏を感じた。
そうか東京の子どもは喫茶店で宿題やるのか。 そんな文化しらないぞ、という気持ちにもなる。
🐣個人esa
Misocaを退職したらMisocaのesaにアクセスできなくなった。不便だったので個人でesaを契約した。
📝日報
esa標準の日報テンプレートにしたがって「今日の作業」「発生した問題」「明日の予定」「所感」を書いている。
一緒に日報書きましょうと募集したら4人あつまった。毎日、みんな書いてて偉い。
見る人がいるのでローカルのメモとブログの中間の温度感になっている。 たのしい。往年の会員制SNSを彷彿させる。
「個人esaにメモ残すの楽しそう」「実際は4人くらい見てるので個人esaじゃない。会員制ブログみたいなイメージ」「mixiだ」「今はオンラインサロンって言葉があるんですよ」という会話をしてた
— mzp (@mzp) August 7, 2018
🛠各種メモ
esaは次の特徴があるのでメモを書くときにちょうどいい。
- Markdownでサクサク書ける。エディタも使いやすい。
- WIP/ShipItの区別がわかりやすい。
- 記事ごとに共有URLが発行できる
🐞WebKitへのバグレポート - みずぴー日記も最初はesaに書いて、まとまった段階ではてなブログに移した。 WIPではじめれるので、気軽に書きはじめれる。 また早くShipItしたいという気持ちも働く。
引越しに関するメモも置いてある。 これは、かなり頻繁に参照している。
イベントの予定もここに書いている。 共有URLを発行して関係者に共有できて便利。
✨その他
個人esaの契約どうしようかなーと思ってトップページにいったら、 mzpという人がおすすめコメントを寄せてた。 なるほどと納得したので契約した。
日報メンバーのりんすきさんがアプリのドキュメンテーションにesaを使いはじめてた。 esaの輪の広がっている。
🐞WebKitへのバグレポート
Safari 12では、日本語入力中でもTabキーによるフォーカス移動が動作する。
過去のバージョンやChromeなどとは動作が異なるためリグレッションバグと考え、WebKitにバグレポート(#188370)を行なった。 修正コミットがtrunkにされたので、いずれSafariでも修正されるはず。(2018-8-17追記: Version 12.0 (14606.1.36.1.3)で反映された)
💥遭遇
AquaSKKでは入力中にTabキーを押すと入力内容が補完される。 これを使ってTweetしようとした際に、フォーカスが移動してしまって困った。
問題箇所の切り分けのために、いろいろな環境で動作を確認した。その結果、以下のことが分かった。
- 標準の日本語入力でもAquaSKKでも発生する。
- Google Chromeでは発生しない。
- MojaveでもHigh Sierraでも発生する。
- Safari 11では発生しないが、Safari 12で発生する。
- Safari Technology PreviewやWebKit Nightlyでも発生する。
- Keyboard Event Viewerで見るかぎりJSレベルではイベントは変化してない。
どうやらWebKitで問題が発生しているらしい。
🛠WebKitのビルド
trunk(問題のあるビルド)
詳しく調べるためにWebKitを手元でビルドした。 Building WebKit | WebKitにいくつかの方法が記載されているが、今回はデバッガを利用したいのでXcodeによるビルドを選択した。
起動して問題が再現することを確認した。
古いWebKit(問題のないビルド)
比較のために問題が再現しないWebKitをビルドする。WebKit Tracからそれっぽいタグを選ぶ。 Safari 11.1.2 (13605.3.8)が問題ないことが分かっているので、Safari-605.3.8
を選択した。 後半の数字と日付がそれっぽいので選んだが、本当に対応してるかはよく分からない。
svn checkout https://svn.webkit.org/repository/webkit/tags/Safari-605.3.8
あとはtrunkと同様にXcodeのワークスペースを開いてビルドする。
👀スタックトレース
☕️ JavaScriptと入力メソッド - みずぴー日記を参考にコードをみてたらTabキーのハンドラみつけた。 これに辿りつくかどうかが肝では、と予想を立てる。
// Soruce/WebCore/page/EventHandler.cpp void EventHandler::defaultTabEventHandler(KeyboardEvent& event) { Ref<Frame> protectedFrame(m_frame); ASSERT(event.type() == eventNames().keydownEvent); // We should only advance focus on tabs if no special modifier keys are held down. if (event.ctrlKey() || event.metaKey() || event.altGraphKey()) return; Page* page = m_frame.page(); if (!page) return; if (!page->tabKeyCyclesThroughElements()) return; FocusDirection focusDirection = event.shiftKey() ? FocusDirectionBackward : FocusDirectionForward; // Tabs can be used in design mode editing. if (m_frame.document()->inDesignMode()) return; if (page->focusController().advanceFocus(focusDirection, &event)) event.setDefaultHandled(); }
trunkの defaultTabEventHandler
にブレイクポイントを仕掛けて、現在のスタックトレースを入手する。
trunkでのスタックトレース
#0 0x000000059024dbb8 in WebCore::EventHandler::defaultTabEventHandler(WebCore::KeyboardEvent&) at /Volumes/SwiftSSD/webkit/Source/WebCore/page/EventHandler.cpp:3940 #1 0x000000059024da35 in WebCore::EventHandler::defaultKeyboardEventHandler(WebCore::KeyboardEvent&) at /Volumes/SwiftSSD/webkit/Source/WebCore/page/EventHandler.cpp:3470 #2 0x000000058f9b0e24 in WebCore::Node::defaultEventHandler(WebCore::Event&) at /Volumes/SwiftSSD/webkit/Source/WebCore/dom/Node.cpp:2383 #3 0x000000058fc7809d in WebCore::HTMLInputElement::defaultEventHandler(WebCore::Event&) at /Volumes/SwiftSSD/webkit/Source/WebCore/html/HTMLInputElement.cpp:1165 #4 0x000000058f947846 in WebCore::callDefaultEventHandlersInBubblingOrder(WebCore::Event&, WebCore::EventPath const&) at /Volumes/SwiftSSD/webkit/Source/WebCore/dom/EventDispatcher.cpp:61 #5 0x000000058f9472ac in WebCore::EventDispatcher::dispatchEvent(WebCore::Node&, WebCore::Event&) at /Volumes/SwiftSSD/webkit/Source/WebCore/dom/EventDispatcher.cpp:175 #6 0x000000058f9b05cd in WebCore::Node::dispatchEvent(WebCore::Event&) at /Volumes/SwiftSSD/webkit/Source/WebCore/dom/Node.cpp:2329 #7 0x000000059024cbd4 in WebCore::EventHandler::internalKeyEvent(WebCore::PlatformKeyboardEvent const&) at /Volumes/SwiftSSD/webkit/Source/WebCore/page/EventHandler.cpp:3297 #8 0x000000059024c2d9 in WebCore::EventHandler::keyEvent(WebCore::PlatformKeyboardEvent const&) at /Volumes/SwiftSSD/webkit/Source/WebCore/page/EventHandler.cpp:3163 #9 0x0000000590dd531b in WebCore::UserInputBridge::handleKeyEvent(WebCore::PlatformKeyboardEvent const&, WebCore::InputSource) at /Volumes/SwiftSSD/webkit/Source/WebCore/replay/UserInputBridge.cpp:82 #10 0x0000000588bb3899 in WebKit::handleKeyEvent(WebKit::WebKeyboardEvent const&, WebCore::Page*) at /Volumes/SwiftSSD/webkit/Source/WebKit/WebProcess/WebPage/WebPage.cpp:2465 #11 0x0000000588bb3742 in WebKit::WebPage::keyEvent(WebKit::WebKeyboardEvent const&) at /Volumes/SwiftSSD/webkit/Source/WebKit/WebProcess/WebPage/WebPage.cpp:2476 #12 0x0000000588c5299a in void IPC::callMemberFunctionImpl<WebKit::WebPage, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&), std::__1::tuple<WebKit::WebKeyboardEvent>, 0ul>(WebKit::WebPage*, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&), std::__1::tuple<WebKit::WebKeyboardEvent>&&, std::__1::integer_sequence<unsigned long, 0ul>) at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/HandleMessage.h:41 #13 0x0000000588c528f0 in void IPC::callMemberFunction<WebKit::WebPage, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&), std::__1::tuple<WebKit::WebKeyboardEvent>, std::__1::integer_sequence<unsigned long, 0ul> >(std::__1::tuple<WebKit::WebKeyboardEvent>&&, WebKit::WebPage*, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&)) at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/HandleMessage.h:47 #14 0x0000000588c3da7a in void IPC::handleMessage<Messages::WebPage::KeyEvent, WebKit::WebPage, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&)>(IPC::Decoder&, WebKit::WebPage*, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&)) at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/HandleMessage.h:127 #15 0x0000000588c36142 in WebKit::WebPage::didReceiveWebPageMessage(IPC::Connection&, IPC::Decoder&) at /Volumes/SwiftSSD/webkit/WebKitBuild/WebKit/Build/Products/Debug/DerivedSources/WebKit2/WebPageMessageReceiver.cpp:213 #16 0x0000000588bba85e in WebKit::WebPage::didReceiveMessage(IPC::Connection&, IPC::Decoder&) at /Volumes/SwiftSSD/webkit/Source/WebKit/WebProcess/WebPage/WebPage.cpp:4038 #17 0x00000005882863ec in IPC::MessageReceiverMap::dispatchMessage(IPC::Connection&, IPC::Decoder&) at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/MessageReceiverMap.cpp:123 #18 0x0000000588e37e0d in WebKit::WebProcess::didReceiveMessage(IPC::Connection&, IPC::Decoder&) at /Volumes/SwiftSSD/webkit/Source/WebKit/WebProcess/WebProcess.cpp:645 #19 0x000000058816c93c in IPC::Connection::dispatchMessage(IPC::Decoder&) at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/Connection.cpp:940 #20 0x000000058815f998 in IPC::Connection::dispatchMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >) at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/Connection.cpp:967 #21 0x000000058816d4c4 in IPC::Connection::dispatchOneIncomingMessage() at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/Connection.cpp:1036 #22 0x00000005881864a8 in IPC::Connection::enqueueIncomingMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >)::$_14::operator()() at /Volumes/SwiftSSD/webkit/Source/WebKit/Platform/IPC/Connection.cpp:933 #23 0x00000005881863b9 in WTF::Function<void ()>::CallableWrapper<IPC::Connection::enqueueIncomingMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >)::$_14>::call() at /Volumes/SwiftSSD/webkit/WebKitBuild/WebKit/Build/Products/Debug/usr/local/include/wtf/Function.h:101 #24 0x000000059d58b96f in WTF::Function<void ()>::operator()() const at /Volumes/SwiftSSD/webkit/WebKitBuild/WebKit/Build/Products/Debug/usr/local/include/wtf/Function.h:56 #25 0x000000059d5e4103 in WTF::RunLoop::performWork() at /Volumes/SwiftSSD/webkit/Source/WTF/wtf/RunLoop.cpp:106 #26 0x000000059d5e4a94 in WTF::RunLoop::performWork(void*) at /Volumes/SwiftSSD/webkit/Source/WTF/wtf/cf/RunLoopCF.cpp:38
このスタックトレースを参考に、古いWebKitの各所にブレイクポイントを設定して動作を比較する。 その結果、EventDispatcher::dispatchEvent
が WebCore::callDefaultEventHandlersInBubblingOrder
を呼ぶかどうかの差を発見した。
605.3.8のスタックトレース
#0 0x00000003cee0dc25 in WebCore::EventDispatcher::dispatchEvent(WebCore::Node&, WebCore::Event&) at /Users/mzp/Safari-605.3.8/Source/WebCore/dom/EventDispatcher.cpp:176 #1 0x00000003cee6d81d in WebCore::Node::dispatchEvent(WebCore::Event&) at /Users/mzp/Safari-605.3.8/Source/WebCore/dom/Node.cpp:2330 #2 0x00000003cf5bfce5 in WebCore::EventHandler::internalKeyEvent(WebCore::PlatformKeyboardEvent const&) at /Users/mzp/Safari-605.3.8/Source/WebCore/page/EventHandler.cpp:3279 #3 0x00000003cf5bf419 in WebCore::EventHandler::keyEvent(WebCore::PlatformKeyboardEvent const&) at /Users/mzp/Safari-605.3.8/Source/WebCore/page/EventHandler.cpp:3145 #4 0x00000003d00bde4b in WebCore::UserInputBridge::handleKeyEvent(WebCore::PlatformKeyboardEvent const&, WebCore::InputSource) at /Users/mzp/Safari-605.3.8/Source/WebCore/replay/UserInputBridge.cpp:83 #5 0x00000003c8a26289 in WebKit::handleKeyEvent(WebKit::WebKeyboardEvent const&, WebCore::Page*) at /Users/mzp/Safari-605.3.8/Source/WebKit/WebProcess/WebPage/WebPage.cpp:2440 #6 0x00000003c8a26132 in WebKit::WebPage::keyEvent(WebKit::WebKeyboardEvent const&) at /Users/mzp/Safari-605.3.8/Source/WebKit/WebProcess/WebPage/WebPage.cpp:2451 #7 0x00000003c8ab982a in void IPC::callMemberFunctionImpl<WebKit::WebPage, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&), std::__1::tuple<WebKit::WebKeyboardEvent>, 0ul>(WebKit::WebPage*, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&), std::__1::tuple<WebKit::WebKeyboardEvent>&&, std::__1::integer_sequence<unsigned long, 0ul>) at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/HandleMessage.h:40 #8 0x00000003c8ab9780 in void IPC::callMemberFunction<WebKit::WebPage, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&), std::__1::tuple<WebKit::WebKeyboardEvent>, std::__1::integer_sequence<unsigned long, 0ul> >(std::__1::tuple<WebKit::WebKeyboardEvent>&&, WebKit::WebPage*, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&)) at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/HandleMessage.h:46 #9 0x00000003c8aa7f66 in void IPC::handleMessage<Messages::WebPage::KeyEvent, WebKit::WebPage, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&)>(IPC::Decoder&, WebKit::WebPage*, void (WebKit::WebPage::*)(WebKit::WebKeyboardEvent const&)) at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/HandleMessage.h:126 #10 0x00000003c8aa1070 in WebKit::WebPage::didReceiveWebPageMessage(IPC::Connection&, IPC::Decoder&) at /Users/mzp/Safari-605.3.8/WebKitBuild/WebKit/Build/Products/Debug/DerivedSources/WebKit2/WebPageMessageReceiver.cpp:204 #11 0x00000003c8a2cd4e in WebKit::WebPage::didReceiveMessage(IPC::Connection&, IPC::Decoder&) at /Users/mzp/Safari-605.3.8/Source/WebKit/WebProcess/WebPage/WebPage.cpp:3965 #12 0x00000003c824c5e8 in IPC::MessageReceiverMap::dispatchMessage(IPC::Connection&, IPC::Decoder&) at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/MessageReceiverMap.cpp:123 #13 0x00000003c8c6e7fd in WebKit::WebProcess::didReceiveMessage(IPC::Connection&, IPC::Decoder&) at /Users/mzp/Safari-605.3.8/Source/WebKit/WebProcess/WebProcess.cpp:648 #14 0x00000003c8141843 in IPC::Connection::dispatchMessage(IPC::Decoder&) at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/Connection.cpp:907 #15 0x00000003c81369d8 in IPC::Connection::dispatchMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >) at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/Connection.cpp:934 #16 0x00000003c8141e94 in IPC::Connection::dispatchOneMessage() at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/Connection.cpp:965 #17 0x00000003c8159add in IPC::Connection::enqueueIncomingMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >)::$_14::operator()() at /Users/mzp/Safari-605.3.8/Source/WebKit/Platform/IPC/Connection.cpp:901 #18 0x00000003c8159a39 in WTF::Function<void ()>::CallableWrapper<IPC::Connection::enqueueIncomingMessage(std::__1::unique_ptr<IPC::Decoder, std::__1::default_delete<IPC::Decoder> >)::$_14>::call() at /Users/mzp/Safari-605.3.8/WebKitBuild/WebKit/Build/Products/Debug/usr/local/include/wtf/Function.h:101 #19 0x00000003dd2f9a9b in WTF::Function<void ()>::operator()() const at /Users/mzp/Safari-605.3.8/WebKitBuild/WebKit/Build/Products/Debug/usr/local/include/wtf/Function.h:56 #20 0x00000003dd33e5f3 in WTF::RunLoop::performWork() at /Users/mzp/Safari-605.3.8/Source/WTF/wtf/RunLoop.cpp:106
🔍コード確認
EventDispatcher::dispatchEvent
を確認すると、callDefaultEventHandlersInBubblingOrder
にはifによるガードがついている。
// WebCore/dom/EventDispatcher.cpp if (!event.defaultPrevented() && !event.defaultHandled()) { // FIXME: Not clear why we need to reset the target for the default event handlers. // We should research this, and remove this code if possible. auto* finalTarget = event.target(); event.setTarget(EventPath::eventTargetRespectingTargetRules(node)); callDefaultEventHandlersInBubblingOrder(event, eventPath); event.setTarget(finalTarget); }
デバッガを使って条件節の値を確認する。 すると、古いSafariではevent.defaultHandled()
が trueだが、trunkではevent.defaultHandled()
が falseになっている。
コードを目で比較すると、trunkでは resetBeforeDispatch()
の呼び出しがある。
// WebCore/dom/EventDispatcher.cppASSERT_WITH_SECURITY_IMPLICATION(ScriptDisallowedScope::InMainThread::isEventDispatchAllowedInSubtree(node)); LOG(Events, "EventDispatcher::dispatchEvent %s on node %s", event.type().string().utf8().data(), node.nodeName().utf8().data()); auto protectedNode = makeRef(node); auto protectedView = makeRefPtr(node.document().view()); EventPath eventPath { node, event }; ChildNodesLazySnapshot::takeChildNodesLazySnapshot(); event.resetBeforeDispatch(); // <- ??????
コードを確認すると、defaultHandled
を初期化している。
void Event::resetBeforeDispatch() { m_defaultHandled = false; }
なので日本語入力とTabキーの問題を追っていたが、真の問題は入力メソッドが処理したキー入力に対してもデフォルトのハンドラが起動することらしい。
📖コミット調査
trackのblame機能を使って、どのコミットでこれが増えたか特定する。
r228260が原因らしい。 コミットログによると、イベント配信間の影響を切るために導入されているらしい。
(WebCore::Event::resetBeforeDispatch): Added. Clears m_defaultHandled so a value left over from a previous dispatch doesn't affect the next dispatch. ---- (WebCore::Event::resetBeforeDispatch): 追加。以前のイベント配信が次の配信に影響しないようにm_defaultHandledをリセットする。
🗣バグレポート
adhocに直す方法はいくつかあるが、どれを選択するのがいいか分からない。 WebKitとRadarにレポートした。どう書くか迷ったがサマリーでは具体的に困ってることを書いて、真の問題はdetailに書くことにした。 またタイトルにregressionというワードを含めた。
- 188370 – Events handled by input method invoke default event handler[regression]
- rdar://42991694: Safari moves focus by tab key during composing text by input method.
その結果、以下のような修正が適用された。入力メソッドが処理したことを示す isDefaultEventHandlerIgnored()
関数が追加された。
Index: trunk/Source/WebCore/dom/Event.h =================================================================== --- a/trunk/Source/WebCore/dom/Event.h +++ b/trunk/Source/WebCore/dom/Event.h @@ -120,4 +120,7 @@ void setDefaultHandled() { m_defaultHandled = true; } + bool isDefaultEventHandlerIgnored() const { return m_isDefaultEventHandlerIgnored; } + void setIsDefaultEventHandlerIgnored() { m_isDefaultEventHandlerIgnored = true; } + void setInPassiveListener(bool value) { m_isExecutingPassiveEventListener = value; } @@ -157,4 +160,5 @@ bool m_wasCanceled { false }; bool m_defaultHandled { false }; + bool m_isDefaultEventHandlerIgnored { false }; bool m_isTrusted { false }; bool m_isExecutingPassiveEventListener { false }; Index: trunk/Source/WebCore/dom/EventDispatcher.cpp =================================================================== --- a/trunk/Source/WebCore/dom/EventDispatcher.cpp +++ b/trunk/Source/WebCore/dom/EventDispatcher.cpp @@ -168,5 +168,5 @@ // default handling, the detail of which handlers are called is an internal // implementation detail and not part of the DOM. - if (!event.defaultPrevented() && !event.defaultHandled()) { + if (!event.defaultPrevented() && !event.defaultHandled() && !event.isDefaultEventHandlerIgnored()) { // FIXME: Not clear why we need to reset the target for the default event handlers. // We should research this, and remove this code if possible. Index: trunk/Source/WebCore/page/EventHandler.cpp =================================================================== --- a/trunk/Source/WebCore/page/EventHandler.cpp +++ b/trunk/Source/WebCore/page/EventHandler.cpp @@ -3289,5 +3289,5 @@ keydown = KeyboardEvent::create(keyDownEvent, &m_frame.windowProxy()); keydown->setTarget(element); - keydown->setDefaultHandled(); + keydown->setIsDefaultEventHandlerIgnored(); }