933141 - [NFC] Move NFC files out of system/gonk

Status: RESOLVED FIXED

(Firefox OS Graveyard :: NFC, defect)

Description

[Garner Lee]

NFC is not strictly a FirefoxOS only feature, although it is currently developed as one.

This bug tracks moving NFC out of gecko/dom/system/gonk and into gecko/dom/nfc instead.

Comment 1

[Ken Chang[:kenkai]]

Please this into backlog.

Comment 2

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Attachment: Patch

I'll ask r? for gps once Dimi reviewed this.

Comment 3

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Comment on attachment 8466104 [details] [diff] [review]
Patch

Please note the rename part which isn't showed in Bugzilla

diff --git a/dom/system/gonk/NfcContentHelper.js b/dom/nfc/NfcContentHelper.js
rename from dom/system/gonk/NfcContentHelper.js
rename to dom/nfc/NfcContentHelper.js
diff --git a/dom/system/gonk/Nfc.js b/dom/nfc/gonk/Nfc.js
rename from dom/system/gonk/Nfc.js
rename to dom/nfc/gonk/Nfc.js
diff --git a/dom/system/gonk/Nfc.manifest b/dom/nfc/gonk/Nfc.manifest
rename from dom/system/gonk/Nfc.manifest
rename to dom/nfc/gonk/Nfc.manifest
diff --git a/dom/system/gonk/NfcGonkMessage.h b/dom/nfc/gonk/NfcGonkMessage.h
rename from dom/system/gonk/NfcGonkMessage.h
rename to dom/nfc/gonk/NfcGonkMessage.h
diff --git a/dom/system/gonk/NfcMessageHandler.cpp b/dom/nfc/gonk/NfcMessageHandler.cpp
rename from dom/system/gonk/NfcMessageHandler.cpp
rename to dom/nfc/gonk/NfcMessageHandler.cpp
diff --git a/dom/system/gonk/NfcMessageHandler.h b/dom/nfc/gonk/NfcMessageHandler.h
rename from dom/system/gonk/NfcMessageHandler.h
rename to dom/nfc/gonk/NfcMessageHandler.h
diff --git a/dom/system/gonk/NfcOptions.h b/dom/nfc/gonk/NfcOptions.h
rename from dom/system/gonk/NfcOptions.h
rename to dom/nfc/gonk/NfcOptions.h
diff --git a/dom/system/gonk/NfcService.cpp b/dom/nfc/gonk/NfcService.cpp
rename from dom/system/gonk/NfcService.cpp
rename to dom/nfc/gonk/NfcService.cpp
diff --git a/dom/system/gonk/NfcService.h b/dom/nfc/gonk/NfcService.h
rename from dom/system/gonk/NfcService.h
rename to dom/nfc/gonk/NfcService.h
diff --git a/dom/system/gonk/nfc_consts.js b/dom/nfc/gonk/nfc_consts.js
rename from dom/system/gonk/nfc_consts.js
rename to dom/nfc/gonk/nfc_consts.js
diff --git a/dom/system/gonk/nsINfcService.idl b/dom/nfc/gonk/nsINfcService.idl
rename from dom/system/gonk/nsINfcService.idl
rename to dom/nfc/gonk/nsINfcService.idl
diff --git a/dom/nfc/moz.build b/dom/nfc/moz.build

Comment 4

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

https://tbpl.mozilla.org/?tree=Try&rev=ace3348d1e78

Comment 5

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Comment on attachment 8466104 [details] [diff] [review]
Patch

r? to gps for the moz.build changes.

Comment 6

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

https://hg.mozilla.org/integration/b2g-inbound/rev/0c92a029fc21

Comment 7

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/0c92a029fc21

Comment 8

[Johnny Stenback (:jst)]

For future reference, changes like this do need review from a DOM peer. Retrospective r=jst for this one.

Comment 9

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

(In reply to Johnny Stenback  (:jst, [e-mail]) from comment #8)
> For future reference, changes like this do need review from a DOM peer.
> Retrospective r=jst for this one.

Hi Johnny
Sorry I don't know I have to ask review from a DOM peer for this.
When these files are created they weren't reviewed by a DOM peer either. 
https://hg.mozilla.org/mozilla-central/rev/0e930ae50509

Can you explain more detail why this also needs a DOM peer review?
Is it because this patch creates an extra folder 'dom/nfc/gonk'?
I just want to be sure what kinds of change should be reviewed by DOM peer, and what changes should be reviewed by a Build Peer.

BTW, is my patch causing any trouble?
if so, please tell me and I'll fix it.

Thanks