Commit b5236720 authored by Bailey Berro's avatar Bailey Berro Committed by Commit Bot

[SplitSettings] Call correct function to open Release Notes

- Corrects the About page to calling the correct method on the
AboutPageBrowserProxy.
- Adds a test

Bug: 997155
Change-Id: If39b45274f6d3836b52039c22086b300b5d3f5b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769625Reviewed-by: default avatarSteven Bennetts <stevenjb@chromium.org>
Commit-Queue: Bailey Berro <baileyberro@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690139}
parent 7cf26221
...@@ -253,7 +253,7 @@ Polymer({ ...@@ -253,7 +253,7 @@ Polymer({
/** @private */ /** @private */
onReleaseNotesTap_: function() { onReleaseNotesTap_: function() {
this.aboutBrowserProxy_.showReleaseNotes(); this.aboutBrowserProxy_.launchReleaseNotes();
}, },
/** @private */ /** @private */
......
...@@ -347,6 +347,9 @@ cr.define('settings_about_page', function() { ...@@ -347,6 +347,9 @@ cr.define('settings_about_page', function() {
await initNewPage(); await initNewPage();
await checkReleaseNotesOnline(true); await checkReleaseNotesOnline(true);
await checkReleaseNotesOffline(false); await checkReleaseNotesOffline(false);
page.$$('#releaseNotesOnline').click();
return aboutBrowserProxy.whenCalled('launchReleaseNotes');
}); });
test('RegulatoryInfo', async () => { test('RegulatoryInfo', async () => {
......
...@@ -16,8 +16,8 @@ class TestAboutPageBrowserProxy extends TestBrowserProxy { ...@@ -16,8 +16,8 @@ class TestAboutPageBrowserProxy extends TestBrowserProxy {
methodNames.push( methodNames.push(
'getChannelInfo', 'getVersionInfo', 'getRegulatoryInfo', 'getChannelInfo', 'getVersionInfo', 'getRegulatoryInfo',
'checkInternetConnection', 'getEnabledReleaseNotes', 'checkInternetConnection', 'getEnabledReleaseNotes',
'getHasEndOfLife', 'openOsHelpPage', 'refreshTPMFirmwareUpdateStatus', 'getHasEndOfLife', 'launchReleaseNotes', 'openOsHelpPage',
'setChannel'); 'refreshTPMFirmwareUpdateStatus', 'setChannel');
} }
if (cr.isMac) { if (cr.isMac) {
...@@ -207,4 +207,9 @@ if (cr.isChromeOS) { ...@@ -207,4 +207,9 @@ if (cr.isChromeOS) {
TestAboutPageBrowserProxy.prototype.openOsHelpPage = function() { TestAboutPageBrowserProxy.prototype.openOsHelpPage = function() {
this.methodCalled('openOsHelpPage'); this.methodCalled('openOsHelpPage');
}; };
/** @override */
TestAboutPageBrowserProxy.prototype.launchReleaseNotes = function() {
this.methodCalled('launchReleaseNotes');
};
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment