1554828 - Don't call RefreshViewportSize or RefreshZoomConstraints in the case where added/modified meta tag is not viewport

Status: NEW

(Core :: Layout, defect, P3)

Description

[Hiroyuki Ikezoe (:hiro)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=d86a8631519408bf50fe137388e98bdd4a8fb0c2

Comment 1

[Hiroyuki Ikezoe (:hiro)]

Hey masayuki, calling do_QueryInterface is the right way to get event target element from EventTarget, just like this?
https://hg.mozilla.org/try/rev/0cca3072809c1c767adcdbc8aea4e693647d19c6#l1.19

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to Hiroyuki Ikezoe (:hiro) from comment #1)

Hey masayuki, calling do_QueryInterface is the right way to get event target element from EventTarget, just like this?
https://hg.mozilla.org/try/rev/0cca3072809c1c767adcdbc8aea4e693647d19c6#l1.19

Unfortunately, yes it is for now.

Comment 3

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1554828 - Pass raw pointer to MVM_LOG. r?botond

Comment 4

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1554828 - Don't call RefreshViewportSize or RefreshZoomConstraints in the case where added/modified meta tag is not for viewport. r?botond

Depends on D32786

Comment 5

[Hiroyuki Ikezoe (:hiro)]

(In reply to Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Might be slow response) from comment #2)

(In reply to Hiroyuki Ikezoe (:hiro) from comment #1)

Hey masayuki, calling do_QueryInterface is the right way to get event target element from EventTarget, just like this?
https://hg.mozilla.org/try/rev/0cca3072809c1c767adcdbc8aea4e693647d19c6#l1.19

Unfortunately, yes it is for now.

Thank you!