Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Commit

Permalink
Merge pull request #22863 from KevinGrandon/bug_1053626_search_legazy…
Browse files Browse the repository at this point in the history
…_moz_activity_removal

Bug 1053626 - [Search] Remove legacy MozActivity calls
  • Loading branch information
KevinGrandon committed Aug 14, 2014
2 parents 93de36c + 1bb9b3b commit d8e87a2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 26 deletions.
10 changes: 0 additions & 10 deletions apps/search/js/search.js
Expand Up @@ -308,16 +308,6 @@
*/
navigate: function(url) {
window.open(url, '_blank', 'remote=true');
/*
Bug 1042012: Disabled until we enable registering of the view activity
to the system app.
var activity = new window.MozActivity({name: 'view', data: {
type: 'url',
url: url
}});
// Keep jshint happy
activity.onsuccess = function() {};
*/
},

/**
Expand Down
17 changes: 1 addition & 16 deletions apps/search/test/unit/search_test.js
@@ -1,6 +1,6 @@
'use strict';
/* global MockNavigatormozApps, MockNavigatormozSetMessageHandler,
MockMozActivity, Search, MockProvider, MockasyncStorage, Promise */
Search, MockProvider, MockasyncStorage, Promise */

require('/shared/test/unit/mocks/mock_navigator_moz_apps.js');
require('/shared/test/unit/mocks/mock_navigator_moz_set_message_handler.js');
Expand All @@ -14,7 +14,6 @@ requireApp('search/test/unit/mock_provider.js');
suite('search/search', function() {
var realAsyncStorage;
var realMozApps;
var realMozActivity;
var realSetMessageHandler;
var clock;

Expand All @@ -31,9 +30,6 @@ suite('search/search', function() {
realMozApps = navigator.mozApps;
navigator.mozApps = MockNavigatormozApps;

realMozActivity = window.MozActivity;
window.MozActivity = MockMozActivity;

realAsyncStorage = window.asyncStorage;
window.asyncStorage = MockasyncStorage;

Expand All @@ -57,20 +53,16 @@ suite('search/search', function() {
suiteTeardown(function() {
navigator.mozSetMessageHandler = realSetMessageHandler;
navigator.mozApps = realMozApps;
window.MozActivity = realMozActivity;
window.asyncStorage = realAsyncStorage;
clock.restore();
delete window.SettingsListener;
});

setup(function() {

MockMozActivity.mSetup();
MockNavigatormozSetMessageHandler.mSetup();
});

teardown(function() {
MockMozActivity.mTeardown();
MockNavigatormozSetMessageHandler.mTeardown();
MockNavigatormozApps.mTeardown();
});
Expand Down Expand Up @@ -268,13 +260,6 @@ suite('search/search', function() {
var stub = this.sinon.stub(window, 'open');
Search.navigate(url);
assert.ok(stub.calledOnce);
// Bug 1042012: Disabled until we enable registering of the view activity
/*
var url = 'http://mozilla.org';
assert.equal(MockMozActivity.calls.length, 0);
Search.navigate(url);
assert.equal(MockMozActivity.calls.length, 1);
*/
});
});

Expand Down

0 comments on commit d8e87a2

Please sign in to comment.