Bug 1310796 part 4. Get rid of 'content-targetable' values for the 'type' attribute everywhere except tabbrowser.xml. Just use 'content' instead. r=gijs

This commit is contained in:
Boris Zbarsky 2016-12-08 16:41:45 -10:00
Родитель e0e53c48ae
Коммит 162c8766a4
9 изменённых файлов: 18 добавлений и 19 удалений

Просмотреть файл

@ -51,7 +51,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=990812
var browser = document.createElement("browser");
browser.setAttribute("messagemanagergroup", "test");
browser.setAttribute("src", "about:mozilla");
browser.setAttribute("type", "content-targetable");
browser.setAttribute("type", "content");
document.documentElement.appendChild(browser);
globalMM.removeDelayedFrameScript(FRAME_SCRIPT_GLOBAL);

Просмотреть файл

@ -54,6 +54,6 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=990812
]]></script>
<browser messagemanagergroup="test" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test" type="content" src="about:mozilla" />
</window>

Просмотреть файл

@ -65,7 +65,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=990812
]]></script>
<browser messagemanagergroup="test1" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test1" type="content" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content" src="about:mozilla" />
</window>

Просмотреть файл

@ -62,7 +62,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=990812
]]></script>
<browser messagemanagergroup="test1" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test1" type="content" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content" src="about:mozilla" />
</window>

Просмотреть файл

@ -68,10 +68,10 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=990812
]]></script>
<browser messagemanagergroup="test1" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test1" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test1" type="content" src="about:mozilla" />
<browser messagemanagergroup="test1" type="content" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content-targetable" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content" src="about:mozilla" />
<browser messagemanagergroup="test2" type="content" src="about:mozilla" />
</window>

Просмотреть файл

@ -45,7 +45,7 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=990812
var browser = document.createElement("browser");
browser.setAttribute("messagemanagergroup", "test");
browser.setAttribute("src", "about:mozilla");
browser.setAttribute("type", "content-targetable");
browser.setAttribute("type", "content");
document.documentElement.appendChild(browser);
globalMM.removeDelayedFrameScript(FRAME_SCRIPT_GLOBAL);

Просмотреть файл

@ -9,7 +9,7 @@
windowtype="navigator:browser"
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<browser id="content" type="content-targetable" src="about:blank" flex="1"/>
<browser id="content" type="content" src="about:blank" flex="1"/>
<script type="application/javascript" src="chrome://browser/content/PresentationView.js"/>
</window>

Просмотреть файл

@ -3418,7 +3418,7 @@ Tab.prototype = {
aParams = aParams || {};
this.browser = document.createElement("browser");
this.browser.setAttribute("type", "content-targetable");
this.browser.setAttribute("type", "content");
this.browser.setAttribute("messagemanagergroup", "browsers");
if (Preferences.get("browser.tabs.remote.force-enable", false)) {
@ -3677,7 +3677,7 @@ Tab.prototype = {
Reader.updatePageAction(this);
ExternalApps.updatePageAction(this.browser.currentURI, this.browser.contentDocument);
} else {
this.browser.setAttribute("type", "content-targetable");
this.browser.setAttribute("type", "content");
this.browser.docShellIsActive = false;
this.browser.blur();
}

Просмотреть файл

@ -20,8 +20,7 @@
* sends.
*
* This also means that <xul:browser>'s that hope to use PrintUtils must have
* their type attribute set to either "content", "content-targetable", or
* "content-primary".
* their type attribute set to either "content" or "content-primary".
*
* PrintUtils sends messages at different points in its implementation, but
* their documentation is consolidated here for ease-of-access.
@ -166,13 +165,13 @@ var PrintUtils = {
*
* getPrintPreviewBrowser:
* Returns the <xul:browser> to display the print preview in. This
* <xul:browser> must have its type attribute set to "content",
* "content-targetable", or "content-primary".
* <xul:browser> must have its type attribute set to "content" or
* "content-primary".
*
* getSourceBrowser:
* Returns the <xul:browser> that contains the document being
* printed. This <xul:browser> must have its type attribute set to
* "content", "content-targetable", or "content-primary".
* "content" or "content-primary".
*
* getNavToolbox:
* Returns the primary toolbox for this window.