[Devtools] Added product registry domain pattern test

[Devtools] Added product registry domain pattern test

Added test for product registry to ensure that domains give consistant
results.

R=dgozman,pfeldman
BUG=720133

[allada, 2017-05-09 22:35:15]

PTL

[pfeldman, 2017-05-10 20:51:19]

https://codereview.chromium.org/2869233002/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/devtools/front_end/product_registry_impl/ProductRegistryImpl.js
(right):

https://codereview.chromium.org/2869233002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/product_registry_impl/ProductRegistryImpl.js:95:
ProductRegistryImpl.registerEntryForTest = function(domainPattern, productName,
type) {
This code should move into the test.

[allada, 2017-05-10 23:10:42]

PTaL

https://codereview.chromium.org/2869233002/diff/20001/third_party/WebKit/Sour...
File
third_party/WebKit/Source/devtools/front_end/product_registry_impl/ProductRegistryImpl.js
(right):

https://codereview.chromium.org/2869233002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/product_registry_impl/ProductRegistryImpl.js:95:
ProductRegistryImpl.registerEntryForTest = function(domainPattern, productName,
type) {
On 2017/05/10 20:51:19, pfeldman wrote:
> This code should move into the test.

Done.

[allada, 2017-05-11 02:24:13]

The CQ bit was checked by allada@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-11 02:25:38]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-11 04:38:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-11 04:38:07]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[allada, 2017-05-12 18:41:13]

The CQ bit was checked by allada@chromium.org

[commit-bot: I haz the power, 2017-05-12 18:42:03]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-12 18:42:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-12 18:42:06]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[allada, 2017-05-12 19:13:58]

The CQ bit was checked by allada@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-12 19:14:47]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-12 23:59:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-12 23:59:34]

Dry run: This issue passed the CQ dry run.

[dgozman, 2017-07-15 05:53:39]

Description was changed from

==========
[Devtools] Added product registry domain pattern test

Added test for product registry to ensure that domains give consistant
results.

R=dgozman,pfeldman
BUG=720133
==========

to

==========
[Devtools] Added product registry domain pattern test

Added test for product registry to ensure that domains give consistant
results.

R=dgozman,pfeldman
BUG=720133
==========

[dgozman, 2017-07-15 05:53:39]

dgozman@chromium.org changed reviewers:
- dgozman@chromium.org

[pfeldman, 2017-07-19 05:44:35]

lgtm - i have no memory of this