[Download] Fixed unexpected result of fixing coverity issues

[Bug] When running on TV device, checking status of cellular network
  returned not supported error, the checking status of cellular needed
  to be moved into correct section instead of running this code in common
  code part.

[Verification] TCT result - 100%

Change-Id: I6cb171034588b2616222bde4101a278c491fdc52
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>
(cherry picked from commit affe078baa94cc800e0eefda9262936826e828d7)

[content] Fixed coding style

Change-Id: I3a4b396e8fba71fbe68763e7128d68334e4de183
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

Merge branch 'tizen_4.0' into tizen

Change-Id: Iba0fed733e5d9c99a37a71ca97c3fe8085e55fe4

Merge "[datacontrol] Fixing coverity issues" into tizen_4.0

[datacontrol] Fixing coverity issues

[Feature] Fixed coverity ids:
109463, 108734

[Verification] 100% passrate on TCT datacontrol tests

Change-Id: I5e563fbd58c814d8d7ca1eefde465303f263c2f3
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

Merge branch 'tizen_3.0' into tizen_4.0

Conflicts:
src/download/download_instance.cc

Change-Id: Ib5546e2489ffdd603e5889e895707b0800209174

[version] 2.19

Change-Id: Ic8848b54eda46b98bd609e0d4254548a8c5beb04
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

Merge "[common] Coverity issues fix" into tizen_3.0

[common] Coverity issues fix

[Feature] Fixed coverity issues with below ids:
  110914, 110006, 108656, 107626, 107387,
  107077, 106006, 108993, 108297, 106912

  Additionally fixed potential crash in download API -
  in case of native function failure, the "success" response
  is not trying to be build.

[Verification] 100% passrate for modules:
  application, bluetooth, download, exif, filesystem, messaging-*

Change-Id: I8d9482501a7375962408186feaa5ecd1f0826b1d
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

[common] Fixing coverity issues

Issues: 110672, 108789, 108137, 107617,
106435, 112265, 112254, 107669, 105923

[Verification] 100% passrate in:
bookmark, calendar, contact, content, datacontrol,
filesystem, keymanager

Change-Id: Id667a94fe653f831a8cfdd70813c8ace91e1cf5d
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

Merge "[common] Fix coverity issue number 112263" into tizen

[common] Fix coverity issue number 112263

[Verification] Build successful

Change-Id: I858c6f6cf34174d1f3efadfbdb688fb0bc04ae8d
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

[Spec][Mediacontroller] Module enabled for tv profile

http://suprem.sec.samsung.net/jira/browse/TWDAPI-189

Change-Id: I5020039fb673ed084904feb7ce7b79b7a79a4447
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

[Push] Prefix string type with namespace std

The patch fixes build break in Tizen:4.0:Unified.
The string type without std::string is not recognized by compiler.

Change-Id: I048bad0a1d71cb841965a84b2d11d2afb9c19eea
Signed-off-by: Lukasz Wojciechowski <l.wojciechow@partner.samsung.com>
(cherry picked from commit a2c7a32a5fbc41e6bf20c891797c786e946628c9)

[Push] Prefix string type with namespace std

The patch fixes build break in Tizen:4.0:Unified.
The string type without std::string is not recognized by compiler.

Change-Id: I048bad0a1d71cb841965a84b2d11d2afb9c19eea
Signed-off-by: Lukasz Wojciechowski <l.wojciechow@partner.samsung.com>

Merge branch 'tizen_4.0' into tizen

Change-Id: I0c449ee349da1f34fbed2243ac0a6616b7b70717

Merge branch 'tizen_3.0' into tizen_4.0

Change-Id: I08833f4e4e820cafcdf5cbb6a291f4c7d78bf05d

[version] 2.18

Change-Id: I171c24fc4170a519fef7fde45b4e223843cf33c0

Merge "[NFC] Fix isActivatedHandlerFunctions return values" into tizen_3.0

[NFC] Fix isActivatedHandlerFunctions return values

Functions IsActivatedHandlerForCategory and IsActivatedHandlerForAID
always return false. After fix they work properly.

TCT tests needs fixing, after fix there will be 100% passrate.
Tested on TW1 tizen3.0

Change-Id: I9a4833d4073e512e9414e63325515ebfa8d6b8a4
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

[Systeminfo] Fix for incorrect USB storage capacity

USB_DEVICE has not been reporting correct capacity values
on TV profile. It was caused by different way of handling
reading storage properties.

[Fix]
Each storage type is treated in the same way when
getting information about storage's space.

[Verification] Tested in Chromium cosole
               tct-systeminfo-tizen-tests passrate: 100%
               tct-filesystem-tizen-tests passrate: 100%

Change-Id: I1d2a8a40a2cac18d5961a301bdd3e586cc3eedad
Signed-off-by: Pawel Kaczmarczyk <p.kaczmarczy@samsung.com>

[NFC] Fix setPreferredApp()

It was unable to use when there was only one app
or app was already set as preferred. Added checking for
already activated app.

[Verification] 100% passrate (auto + manual)_on solis TW2

Change-Id: Ie85d1c09692bebdca7afe5278f63db9071d1adf5
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

[Push] Prefix string type with namespace std

The patch fixes build break in Tizen:Unified.
The string type without std::string is not recognized by compiler.

Change-Id: I048bad0a1d71cb841965a84b2d11d2afb9c19eea
Signed-off-by: Lukasz Wojciechowski <l.wojciechow@partner.samsung.com>

[version] 2.18

Change-Id: Ie8b94c3bb824a8214df2649fdaa196cfd9d8d4e5
Signed-off-by: Lukasz Bardeli <l.bardeli@samsung.com>

[Voice-control] Enabling module for 5.0

[Verification] Code compiles, TCT VC A&M 100%

Change-Id: I6a01eb8548582660da8f09ae1fdc75af04e1da59
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

[Power] Fix for casting void* into int

This bug caused to fail building on 64-bits architecture.

[Verification] Power M&A TCT 100% (TM1).
The code is build-able for 64-bits arch.

Change-Id: I3600a753956f448ca0938655cf1873e9ba057754
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

Merge branch 'tizen_4.0' into tizen

Conflicts:
src/nfc/nfc_message_utils.cc

Change-Id: Iec3145fba1c248841abce2dabbd9c912347aa198

Merge branch 'tizen_3.0' into tizen_4.0

Change-Id: I402b1950719d6b614070be9b7cf2228339e3f70f

[version] 2.17

Change-Id: I3bf56e2b79bff86daefb7b9c3c8d53b41d5ac22c
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

Merge "[NFC] Fix for casting signed value to unsigned int" into tizen

Merge "[NFC] Fix memory management problems" into tizen_3.0

[NFC] Fix memory management problems

This commit prevents crashes, fixing 2 memory management problems:
1. user_data passed to nfc_tag_transceive was deleted twice
2. buffer argument of tagTransceiveCb was deleted inside the function;
   according to Native NFC API reference: "buffer will be automatically
   destroyed when the callback function returns. (Do not release buffer.)"

[Verification] NFCTag.transceive() function was tested in Chrome
               DevTools console and worked fine.

Change-Id: I509c368d817b6663373ba0614af73402f8ad47a6
Signed-off-by: Pawel Wasowski <p.wasowski2@partner.samsung.com>

[NFC] Fix for casting signed value to unsigned int

SVACE ID #310366

[Verification] TW2 NFC M&A TCT pass rate 100%.

Change-Id: Id329ba99d1ae62c6380c672dc3a06d2cd581011b
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

[NFC] Fix for casting signed value to unsigned int

SVACE ID #310366

[Verification] TW2 NFC M&A TCT pass rate 100%.

Change-Id: Id329ba99d1ae62c6380c672dc3a06d2cd581011b
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

Merge "[Archive] Updating cached storages" into tizen

Merge "[MediaController] Fix for bug onChange events" into tizen

[MediaController] Fix for bug onChange events

In server side, all onchange*request events should be fired only after
requesting server state change from client side.
Clients should receive on*changed events only after server state change.

[Verification] TCT passed 100%

Change-Id: I85e5eda12acf6de74bc653a9ccc8a283862eeb16
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

Merge "[MediaController] Refactoring internal implementation into dedicated CAPI" into tizen

[Calendar] Fixing multiple bugs in calendar API

3 bugs has been fixed:
1) User couldn't add new calendar to database
1a) Resolved by sending new object, no calendar through JSON
2) Calendar object was not updated with id got from native layer
2a) Resolved by returning id to js layer and...
2b) Changing setting its as result.id, not just a result
3) Every event has been set to calendar with id 0
3a) Resolved by utilizing not used calendarId attribute

[Verification]
verified in chrome console
tct-tests 100% passrate

Change-Id: I5edcf33d8dd753fc044d2b61677439145f4a73d1
Signed-off-by: Michal Bistyga <m.bistyga@samsung.com>

Merge "[Systeminfo] Prevention for unnecessary double mutex-locking" into tizen

[Archive] Updating cached storages

Currently the implementation obtains all available storages once, during
launching Archive plugin. The fix is based on solution from Filesystem.

[Verification] Archive 100% pass rate.

Change-Id: If0368d89fef3ac25c95bd7d5cf031d6d635643a4
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

Merge "[MessagePort] Removing unreachable and optimizing code" into tizen

Merge "[Power] Refactoring module" into tizen

[MessagePort] Removing unreachable and optimizing code

SVACE ids: #312935, #312936.

[Verification] MessagePort TCT 100% pass rate.

Change-Id: I1e17e53cef6fa7a5557485e01375c787b8f8c6d7
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

[Systeminfo] Prevention for unnecessary double mutex-locking

The mutex locking may hang the application if all getters fail.

[Verification] Systeminfo TCT 100% pass rate + manually tested in console.

Change-Id: I270c6f5a5cb40f3975104476aa4e7de3179e0e64
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

Merge "[Package] refactor module" into tizen

[VoiceControl] Fixing bug with releasing listeners

Listeners was not removed after calling release() function

[verification] 100% passrate (auto and manual) on TM1

Change-Id: Ie241a7df83a72d0756f51d61b507f60d8b7c5f0f
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

[Package] refactor module

Remove setMessageListener from package_api.js and start using from utils_api.js

[Verification] Code compiles without error. TCT passrate 100%

Change-Id: I5e5f96a4a836dd85e11c3989233092407d681d28

[MediaController] Refactoring internal implementation into dedicated CAPI

[Verification] TCT pass rates: 100% A 100% M

Change-Id: Id01a8988a122d0055f662eabe066a4b3f35bb48b
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

[Power] Refactoring module

Previous implementation depended partly on Deviced functions through the
dbus interface. Currently, the implementation uses only DeviceD Native API
(except the ReleaseBrightness and SetBrightness methods).
TheReleaseBrightness is still not wrapped into the Native C DeviceD API).

[Verification] TCT pass: A/M - 100%.

Change-Id: I03124c0d332bf03cd5481ac2b38c9e5c9a534063
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

Merge "[Filesystem] Fix return on error in FileWriteString and FileWriteBytes" into tizen_3.0

[Filesystem] Fix return on error in FileWriteString and FileWriteBytes

Change-Id: I0377b1a7eaa60a1b5d50c759005d2c6063400332
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

Merge "[Contact] photoURI attribute of Person object is now readonly" into tizen

Merge "[Exif] Fix exception throwing constructor" into tizen

Merge branch 'tizen_3.0' into tizen_4.0

Change-Id: Iaa5ae2327a66726af1922e072dd6b39fc76de3d7

[SecureElement][NFC][Messageport] Fixing casting problems

[Bug] Casting double -> char should not be done directly.

[Verification] Passing values greater than 127 to API makes that
  values are correctly translated.
  TCT passrate 100%.

Change-Id: I786392fd3be2e7d0eb5211e224a25c8078f383e9
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

[Exif] Fix exception throwing constructor

Invalid values error could be thrown by ExifInformation constructor.

[Verification] tct-exif-tizen-tests passrate: 100%

Change-Id: Icee96c47faa4c8327b8dec9c9509bd9a53720e15
Signed-off-by: Pawel Kaczmarczyk <p.kaczmarczy@samsung.com>

Merge branch 'tizen_4.0' into tizen

Change-Id: I688196f478770706efdf5843ed34c6524f11a848

Merge branch 'tizen_3.0' into tizen_4.0

Change-Id: I341501004603b29f58f41320ab1093f09f039af6

[version] 2.16

Change-Id: I7316b54e3c6ab3fe41b5e2e1ac6e4b1a9e7dfa35
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

Merge "[Bluetooth] Fixing undefined behaviour during cast" into tizen_3.0

[NFC] Fixing undefined behaviour during static_cast to unsigned char

Validation:
Automatic tests 100% pass rate
TODO manual tests

Change-Id: I9c15efc77c82f0f7e4019913b2bd02d0f8047bb1
Signed-off-by: Michal Bistyga <m.bistyga@samsung.com>

[Bluetooth] Fixing undefined behaviour during cast

Char is unsigned by default on ARM architecture. static_cast from double to unsigned char is undefined behaviour and complier overwritten any negative value with 0.

[Validation]
tests 100% pass rate

Change-Id: Iec39a3c17b18ec7aa4f020fe5de709d66fa426fd
Signed-off-by: Michal Bistyga <m.bistyga@samsung.com>

[Common] Adding macro to check file access

[Verification] The macro works as intended on applications with required
version 4.0 and 5.0.

Change-Id: I2d530cd6c2447616bb963fb3640bd16b8b1b8fd9
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

[Contact] photoURI attribute of Person object is now readonly

[Verification] 1 tc fails, DPTTIZEN needs to be requested

Change-Id: I038662501bdc5b0409945e3da4a65bab961a4e60
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

Merge branch 'tizen_3.0' into tizen_4.0

Conflicts:
src/download/download_instance.cc

Change-Id: I58ad10dac86b0a93074499fc839b30dd365b0f59

[version] 2.15

- version increased
- fixed minor style issue

Change-Id: I70a371ce7e6834bf73c413e19de37111168c9147
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

[Download] refactoring module

Remove setMessageListener from download_api.js and start using from utils_api.js
Remove method declarations from .cc file getDownloadRequest and setListener
Variable name changes
Delete element from callbacks after finished, cancel or failed download

[Verification] Code compiles without error. TCT passrate 100%

Change-Id: Idcca394f4ec6bda7ea3097248a9c900b2a542f7a
Signed-off-by: Lukasz Bardeli <l.bardeli@samsung.com>

Merge "[Utils] Fix Long conversion (allow hex string value)" into tizen_3.0

[Utils] Fix Long conversion (allow hex string value)

Until now string "0x15" would be converted 0. This change affects
all numeric conversions which use _toLong.

According to https://www.w3.org/TR/WebIDL-1/#es-long value
should be initialized by ToNumber, which is defined in
https://tc39.github.io/ecma262/#sec-tonumber-applied-to-the-string-type
and allows value to be in form of HexIntegerLiteral.

Change-Id: Ib719ce8fd5beccc5947b761dc905c49ac0469490

[Filesystem] Add support for ISO-8859-1

Latin-1 encoding in FileReadString and FileWriteString

Change-Id: I2cd7942481619989891c3019d3f7d4325b962892
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

Merge "[Filesystem] Remove special case for zero in readBytes/writeBytes" into tizen_4.0

Merge "[Download] Fix preventing crash. Prevent call callback twice" into tizen_4.0

[Download] Fix preventing crash. Prevent call callback twice

Add condition to prevent call some callback twice and modify body of methods
OnFinished, OnCanceled and OnFailed to prevent crash.

[Verification] Code compiles without error. TCT passrate 100%

Change-Id: I7d194b4d762a52f0281e22555bfd54caa3cb4579
Signed-off-by: Lukasz Bardeli <l.bardeli@samsung.com>

[Download] Fix preventing crash. Prevent call callback twice

Add condition to prevent call some callback twice and modify body of methods
OnFinished, OnCanceled and OnFailed to prevent crash.

[Verification] Code compiles without error. TCT passrate 100%

Change-Id: I7d194b4d762a52f0281e22555bfd54caa3cb4579
Signed-off-by: Lukasz Bardeli <l.bardeli@samsung.com>

Merge "[Application] Removing useless comment" into tizen

Merge "[Common] Improve performance of picojson::serialize_str" into tizen_4.0

[humanactivitymonitor] Fix crash when WRIST_UP event was called

[Verification] Tested with custom app - event fires properly
Auto test on TW2 - 100% passrate

Change-Id: I762a2a0950c4b17333f78b11eb320680f8ec1128
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

[Filesystem] Remove special case for zero in readBytes/writeBytes

Null character in Unicode sequence is correctly interpreted
by picojson and crosswalk, so remove mapping 0 to 0x100.

Change-Id: I61988a1089ecdab006ce5cc2492960d526fb9455
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

[Common] Remove unused member and parameters from ScopeLogger

Change-Id: I1ce2a5747c31e7a751f75005b0574e983e860fb1
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

[Common] Improve performance of picojson::serialize_str

Remove snprintf by mapping all C0 control characters
in a static array.

Change-Id: I3c801551845523b8c33d74426cdddc30103d426d
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

[Sound] Using new native API

[Feature] Old native API was removed with ACR-1128, so there is need to use
  new API instead.

[Verification] TCT passrate 100%

Change-Id: I9cd4343294f10a7d56fea597adda0cf1153d1300
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

Merge branch 'tizen_4.0' into tizen

Change-Id: Icf932b14b5aecf7a4e381a464f4fb5a81f64e6fc

Merge branch 'tizen_3.0' into tizen_4.0

Conflicts:
src/filesystem/filesystem_file.cc

Change-Id: I0360e670aefd456dd14b93948c1f2e3e4640e88e

[version] 2.14

Change-Id: I7b9924c14471bce00e2e5eaf607080c869ad92b1
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

Merge "[Filesystem] Remove unused Base64 and UTF-8 functions" into tizen_3.0

Merge "[Filesystem] Speed up conversion to octet in writeBytes" into tizen_3.0

Merge "[Filesystem] Write file performace improvement (binary data through UTF-8)" into tizen_3.0

Merge "[Filesystem] Read file performace improvement (binary data through UTF-8)" into tizen_3.0

[Application] Removing useless comment

The Context-History related features are available on Mobile&Wearables.

Change-Id: I1c232b55e80760dbbe08b8b69be2121e9cd000b4
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>

[Filesystem] Remove unused Base64 and UTF-8 functions

removed: encodeString, decode, decodeString, getCodePoint,
_utf8_encode, _utf8_decode

Change-Id: I269cd71eb252e2f6ea9b266a50acde3f769db50e
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

[Filesystem] Speed up conversion to octet in writeBytes

Do not use validateArgs to validate each value in array

Change-Id: I9ac59f59187c7f57a0213778d0f2552cb9de4538
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

[Filesystem] Write file performace improvement (binary data through UTF-8)

Improve performance of write, writeBytes, writeBase64 by
pushing binary data through UTF-8 null-terminated string.
Byte 0x00 is encoded as U+0100.

Removed class FilesystemFile.

[Verification] TCT pass rate is 100%
    tct-filesystem-tizen-tests

Change-Id: Iee5983a429ed484eb0fd6a6d35ea794d292de8de
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

[Filesystem] Read file performace improvement (binary data through UTF-8)

Improve performance of readBytes, readBase64 by pushing binary data
through UTF-8 null-terminated string.
Byte 0x00 is encoded as U+0100.

Removed not used functions in FilesystemFile. Improved performance of
readAsText.

[Verification] TCT pass rate is 100%
    tct-filesystem-tizen-tests

Change-Id: I41fe2f9c7855721b38ad4bbf268e8eb7851b10eb
Signed-off-by: Jakub Skowron <j.skowron@samsung.com>

[humanactivitymonitor] Fix timestamp in HumanActivityRecognitionData

[Verification] auto and manual tests which use this data pass

Change-Id: I8f174925147c561cfcf22590cc5d4b5abbbba506
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>
(cherry picked from commit f3123977c14bad5410ee36b520f07c4061d4fc3a)

Merge "[humanactivitymonitor] Fix timestamp in HumanActivityRecognitionData" into tizen_4.0

[HAM] Added missing conversion of alwaysOn parameter

[Verification] TCT passrate on Solis binary increased, some TCT issues left,
but webapi code seems to be valid now.

Change-Id: I7f5e1cd4ff7ea8f3a9f141f46794afc3a4789f06
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>

[humanactivitymonitor] Fix timestamp in HumanActivityRecognitionData

[Verification] auto and manual tests which use this data pass

Change-Id: I8f174925147c561cfcf22590cc5d4b5abbbba506
Signed-off-by: Rafal Walczyna <r.walczyna@partner.samsung.com>

Merge branch 'tizen_3.0' into tizen_4.0

Conflicts:
src/filesystem/filesystem_instance.cc
src/filesystem/js/file.js

Change-Id: I355ce9bb6d27d1990267994f0cfec5c7eaa07792

[version] 2.13

Code format was also fixed

Change-Id: Iae0d214d3caeeb67cdbfe1938718863d70a15789
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>