Backed out 1 changeset (bug 1381648) on request from flod for wrong placeholders r=backout Backed out changeset bad9f2a1be21 (bug 1381648)

Bug 1404917 - Move image preview tooltip to file name and remove preview icon. r=Honza MozReview-Commit-ID: 86YPdHZhCmo

Bug 1404917 - Fetch response content only on-demand. r=Honza Response content should only be fetched whenever it is strictly needed as it is the response body. A possibly very large string. So, netmonitor UI should only retrieve it when users select the Response Panel or do any other action that require having access to it (like "Copy response" context menu). MozReview-Commit-ID: CtpJ8PKsCsm

Bug 1404917 - Use request.mimeType instead of request.responseContent.content.mimeType. r=Honza MozReview-Commit-ID: EIaS1CFKzVn

Bug 1416162 - Fix error code for InvalidElementStateException r=jgraham MozReview-Commit-ID: J5nLBJ27xY0

Bug 1390289 - whitelist xpi file r=jmaher MozReview-Commit-ID: 6RBqG8Oob1a

merge mozilla-central to autoland. r=merge a=merge on a CLOSED TREE

merge mozilla-central to autoland. r=merge a=merge on a CLOSED TREE

Bug 1412608 - Fix comment selection color in markup view. r=jdescottes MozReview-Commit-ID: 9G5efQSPHCf

Bug 1411838 - Part 2: Fix non thread-safe objects in VR; r=kip MozReview-Commit-ID: EzbNWU17qAP

Bug 1411838 - Part 1: Enable WebVR reftest for Windows debug; r=kip MozReview-Commit-ID: A8drAvmATUZ

Bug 1392538 - Remove unused onboarding experiments. r=sebastian MozReview-Commit-ID: 5Xjpz7l1IxP

Bug 1382937 - Rewrite Sync's master password functions to use the `nsILoginManagerCrypto` wrappers. r=MattN Using `nsISecretDecoderRing` directly bypasses `nsILoginManagerCrypto.uiBusy` and the observer notifications, so other consumers might not be aware we're already showing the dialog. We also bail early if the UI is busy, to avoid showing multiple dialogs. MozReview-Commit-ID: I7xzUWZkyPH

Bug 1416141 - Remove Telemetry VIDEO_EME_PLAY_SUCCESS. r=kikuo MozReview-Commit-ID: BtiD2tUmbE9

Bug 1415409 - Make == operator of RangeBoundaryBase compare mRef and mOffset more carefully r=catalinb Currently, RangeBoundaryBase can store either only mRef or mOffset. However, == operator of RangeBoudaryBase still compares mRef simply. However, if one has only mRef and the other has only mOffset, it returns false. This patch makes == operator checks if both mOffset have been set. If so, this checks both mOffset.value() and mRef are matched. However, if mRef of only one of them is nullptr, this doesn't make it check mRef because computing mRef needs some cost and initializing mRef from the other fixes the referring child stronger. If the user of the operator sets only mOffset and wait DOM tree changes, computing mRef may break such callers. If one has only mRef and the other has only mOffset, this patch makes it compute mRef. This is not the best behavior, perhaps. However, there is no way to compare these instances. If this becomes a problem, we should make it create temporary instance, but it'll waste runtime cost. So, let's avoid using this approach for now. Finally, making it check both mRef simply. MozReview-Commit-ID: 4nsW5SYDTiZ

Bug 1416065 - Ensure that override dirty rects are properly removed from frames r=mattwoodrow MozReview-Commit-ID: 8uLgDFxl5MV

Bug 1415231 - Add crash tests for a regression of bug 1406482 which has been fixed by bug 1415509 r=m_kato This crash test can cause crash before landing the patches for bug 1415509. So, let's take this for regression test. MozReview-Commit-ID: 652wi49e720

Bug 1415800 - part 5: Redesign HTMLEditRules::FindNearSelectableNode() r=m_kato First, the method name is not correct. It tries to find an editable node near the given DOM point. Therefore, it should be FindNearEditableNode(). Next, the implementation did something odd. E.g., in the first |if| block, when |nearNode| is nullptr, it returns nullptr. However, following |if| block does something only when |nearNode| is nullptr. So, we can get rid of the second |if| block. Then, nobody will change aDirection. So, we can make it not a reference now. Similarly, in |while| block, if |nearNode| becomes nullptr, it returns error. However, following block checks if |nearNode| is NOT nullptr. So, we can get rid of this |if| statement and outdent its block. Additionally, |curNode| isn't necessary. It only increments the refcount redundantly. So, we can get rid of it. Finally, FindNearEditableNode() can return found node directly instead of error code because error code doesn't make sense. Not found an editable node is not illegal. And also it can take EditorRawDOMPoint instead of a set of container, child and offset of the child in the container. MozReview-Commit-ID: CTI581PhJMd

Bug 1415800 - part 4: Redesign HTMLEditor::GetNextHTMLNode() same as similar to EditorBase::GetNext*() r=m_kato HTMLEditor::GetNextHTMLNode() should be redesigned as HTMLEditor::GetNextEditableHTMLNode(nsINode&), HTMLEditor::GetNextEditableHTMLNodeInBlock(nsINode&), HTMLEditor::GetNextEditableHTMLNode(const EditorRawDOMPoint&) and HTMLEditor::GetNextEditableHTMLNodeInBlock(const EditorRawDOMPoint&). Same as GetPreviousEditableHTMLNode*(), we don't need the methods to find non-editable nodes too. MozReview-Commit-ID: JjZauCMblp4

Bug 1415800 - part 3: Redesign HTMLEditor::GetPriorHTMLNode() as similar to EditorBase::GetPrevious*() r=m_kato HTMLEditor::GetPriorHTMLNode() methods are similar to EditorBase::GetPriorNode() which was redesigned with the previous patch. So, it should be redesigned as HTMLEditor::GetPreviousEditableHTMLNode(nsINode&), HTMLEditor::GetPreviousEditableHTMLNode(const EditorRawDOMPoint&), HTMLEditor::GetPreviousEditableHTMLNodeInBlock(nsINode&) and HTMLEditor::GetPreviousEditableHTMLNodeInBlock(const EditorRawDOMPoint&). Note that HTMLEditor::GetPriorHTMLNode() are always return editable node. So, we don't need to create non-editable node methods for them. Although, I don't like the word "HTMLNode" because this can return SVG element or something too. The additional feature of those methods is just checking given node is in active editing host. So, they are for HTML editor, but not returning only HTML nodes. However, I have no better idea with shorter name. MozReview-Commit-ID: 3J4IaBOFjzj